Skip to content
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

0.7 sync not excluding v3/excludes #959

Closed
awcrosby opened this issue May 11, 2022 · 1 comment · Fixed by #975
Closed

0.7 sync not excluding v3/excludes #959

awcrosby opened this issue May 11, 2022 · 1 comment · Fixed by #975
Assignees
Labels

Comments

@awcrosby
Copy link
Contributor

Version
galaxy_ng dev environment running:
galaxy_ng==4.3.5
pulpcore==3.11.2
pulp-ansible==0.7.4

Describe the bug

  • Performing a sync with galaxy_ng 4.5.0 with pulp-ansible==0.13.0 against console.stage.redhat.com yields the expected result: collections in the v3/excludes endpoint are not sync’d.
  • Doing the same with galaxy_ng 4.3.5 with pulp-ansible==0.7.4 does not exclude the collections at v3/excludes, this is expected since there is a backport of this: 41f79c6

To Reproduce
Sync against https://console.stage.redhat.com/api/automation-hub/content/6089719-synclist/

Expected behavior
Collections sync'd should not include https://console.stage.redhat.com/api/automation-hub/content/6089719-synclist/v3/excludes/

Additional context
This is mentioned and tracked in https://issues.redhat.com/browse/AAH-1583

@gerrod3
Copy link
Contributor

gerrod3 commented May 13, 2022

I figured out the issue. An early version of the exclude sync was backported. This version required setting the excluded collections on the sync task, but since excludes was put into an endpoint it was changed later and never backported (again).

gerrod3 added a commit to gerrod3/pulp_ansible that referenced this issue May 13, 2022
gerrod3 added a commit to gerrod3/pulp_ansible that referenced this issue May 13, 2022
gerrod3 added a commit to gerrod3/pulp_ansible that referenced this issue May 13, 2022
gerrod3 added a commit to gerrod3/pulp_ansible that referenced this issue May 13, 2022
gerrod3 added a commit to gerrod3/pulp_ansible that referenced this issue May 19, 2022
gerrod3 added a commit to gerrod3/pulp_ansible that referenced this issue May 19, 2022
@gerrod3 gerrod3 closed this as completed May 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants