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

Fixes #31197 - ansible galaxy URL updates #9387

Merged
merged 1 commit into from Jun 14, 2021

Conversation

jjeffers
Copy link
Member

No description provided.

@theforeman-bot
Copy link

Issues: #31197

@jjeffers jjeffers changed the title [WIP] Fixes #31197 - ansible galaxy URL updates Fixes #31197 - ansible galaxy URL updates Jun 9, 2021
@jeremylenz jeremylenz mentioned this pull request Jun 9, 2021
7 tasks
"<b>2. For specific collection with URL filtering:</b><br/>" +
"https://galaxy.ansible.com/api/v2/collections/testing/k8s_demo_collection <br/>" +
"<b>3. For specific collections with Requirements.yml:</b><br/>" +
"https://galaxy.ansible.com/api/<br/>" +
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Think https://galaxy.ansible.com/ is used for mirroring although the last I tried to mirror, it failed on some collection in galaxy..Not sure we want to advertise mirroring?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sjha4 you mean mirroring all of Galaxy? Do we know why this was presented as a tool tip in the first place?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ya..That was supposed to be a use case for collections support..Mirroring all of galaxy. I don't mind leaving that as advertised. Only issue was last I tried it, it failed on some collection in the galaxy. Think we'll have to test it better so it doesn't stay broken.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sjha4 @jlsherrill aside from the comments re the galaxy url being listed... any other items we need to adjust? I think @jeremylenz is waiting on this for 4.1 picks.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe we should discuss it more with the pulp-ansible team and merge this as is. ACK from me

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack +1

@jlsherrill
Copy link
Member

Tested capsule sync with no issues! so just some small text updates needed

@jjeffers jjeffers force-pushed the 31197_ansible_repo_path_changes branch from 89b0da5 to a20b5b8 Compare June 14, 2021 10:31
@jjeffers jjeffers force-pushed the 31197_ansible_repo_path_changes branch from a20b5b8 to d6dd2a6 Compare June 14, 2021 16:56
@jjeffers jjeffers merged commit 427b67e into Katello:master Jun 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants