-
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
Add support for automatic publishing and distributing #1938
Conversation
Attached issue: https://pulp.plan.io/issues/7622 |
d2ea502
to
e7c3450
Compare
bc6e5d5
to
0497c43
Compare
338bbb8
to
d71e00e
Compare
e0c89d6
to
bf9ae43
Compare
bf9ae43
to
a072e12
Compare
80cfa20
to
5029773
Compare
38abc38
to
0df4619
Compare
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.
Looks really good! Thanks!
def delete_remaining_old_master_model_entries(apps, schema_editor): | ||
with connection.cursor() as cursor: | ||
for pk in pks_to_delete: | ||
cursor.execute("DELETE from core_basedistribution WHERE pulp_id = %s", [pk]) |
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.
Why are we deleting one by one? is it faster/better?
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.
Yeah I don't really get it either, but this was what Brian ended up doing. There was some discussion about it on the original file PR.
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.
At this point I'm not going to diverge from what was already tested / merged
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.
There's probably not enough distributions for it to make a lot of difference.
@@ -205,6 +205,7 @@ class RpmRepository(Repository): | |||
ModulemdDefaults, | |||
] | |||
REMOTE_TYPES = [RpmRemote] | |||
GPGCHECK_CHOICES = [(0, 0), (1, 1)] |
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.
👍
e4130e2
to
80ddfe9
Compare
80ddfe9
to
75eb93d
Compare
closes: #7622
https://pulp.plan.io/issues/7622