-
Notifications
You must be signed in to change notification settings - Fork 123
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Advisory conflict resolution #1583
Advisory conflict resolution #1583
Conversation
Hello @pavelpicka! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2020-02-11 14:52:11 UTC |
f55e77c
to
292fa89
Compare
CHANGES/5924.misc
Outdated
@@ -0,0 +1 @@ | |||
Fix advisory conflict resolution to check current version first and enabled SyncMutatedUpdateRecordTestCase. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the patch to enable the test was lost, because I don't believe it's enabled?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was not lost, there was intention to fix/enable it too but will need refactor of test itself (issue #6015 on pulp.plan.io) before just enable it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pavelpicka Please re-enable the test
pulp_rpm/pulp_rpm/tests/functional/api/test_sync.py
Lines 409 to 412 in 292fa89
@unittest.skip( | |
'FIXME: Enable this test after we can throw out duplicate UpdateRecords' | |
) | |
class SyncMutatedUpdateRecordTestCase(unittest.TestCase): |
Code changes look good, thanks. |
292fa89
to
ba778dd
Compare
Advisory conflict resolution now check current version first if advisory resolution is needed. Required PR: pulp#1568 closes: #5924 https://pulp.plan.io/issues/5924
ba778dd
to
33761b7
Compare
…rror Advisory conflict resolution
…rror Advisory conflict resolution
Advisory conflict resolution now check current version first if advisory
resolution is needed.
Required PR: #1568
closes: #5924
https://pulp.plan.io/issues/5924