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
Deprecate resources argument of dispatch #1556
Conversation
At some point we can even write a migration to split the lists of resources in the two list fields in the database. |
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 only reviewed the 2nd commit because I've reviewed the first PR and I assume they're the same. I think the explicitness is helpful, definitely the right way to go. Thanks.
e88a452
to
f4df722
Compare
Attached issue: https://pulp.plan.io/issues/9257 |
85281bb
to
2eb94ca
Compare
pulpcore/app/viewsets/reclaim.py
Outdated
@@ -37,7 +37,7 @@ def reclaim(self, request): | |||
|
|||
task = dispatch( | |||
reclaim_space, | |||
repos, | |||
shared_resources=repos, |
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.
This seems to be out of scope of this PR. I think it has nothing to do with the deprecation but it changes the current behaviour. correct?
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.
Right, i'll make it a separate PR.
edit: #1574
pulpcore/app/viewsets/repository.py
Outdated
@@ -195,7 +197,7 @@ def repair(self, request, repository_pk, number): | |||
|
|||
task = dispatch( | |||
tasks.repository.repair_version, | |||
[version.repository], | |||
shared_resources=[version.repository], |
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.
same here
fixes #9257
2eb94ca
to
3e10dd5
Compare
@dralley This is based on #1526 but i'll open it for early review now.
If you think this is the way to go i'm happy to assign a ticket to it.