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

Install awx collection from branch for operator ci #15616

Conversation

chrismeyersfsu
Copy link
Member

@chrismeyersfsu chrismeyersfsu commented Nov 5, 2024

SUMMARY

Install awx collection from current branch for operator ci test

ISSUE TYPE
  • Bug, Docs Fix or other nominal change
COMPONENT NAME
  • API
AWX VERSION
devel
ADDITIONAL INFORMATION

How do I know this is installing the branch version of the awx collection?
https://github.com/ansible/awx/actions/runs/11695232281/job/32570297454?pr=15616 <-- here is a run where I put an error in awx_collection/galaxy.yml file to make the awx collection install fail. This shows that the collection install is running against the branch.
image

A successful run with the error in awx_collection/galaxy.yml removed https://github.com/ansible/awx/actions/runs/11695303693/job/32570502882?pr=15616

@chrismeyersfsu chrismeyersfsu force-pushed the feature-install-collection-from-source branch 8 times, most recently from 7f5eeb6 to de46dc3 Compare November 6, 2024 01:01
@github-actions github-actions bot added the component:awx_collection issues related to the collection for controlling AWX label Nov 6, 2024
@chrismeyersfsu chrismeyersfsu force-pushed the feature-install-collection-from-source branch from de46dc3 to 4b7e00c Compare November 6, 2024 01:08
@github-actions github-actions bot removed the component:awx_collection issues related to the collection for controlling AWX label Nov 6, 2024
Copy link
Contributor

@pb82 pb82 left a comment

Choose a reason for hiding this comment

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

LGTM, good idea to rewrite requirements.yml

Copy link

sonarcloud bot commented Nov 7, 2024

@chrismeyersfsu chrismeyersfsu merged commit 51b1fa4 into ansible:devel Nov 7, 2024
39 of 46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants