-
Notifications
You must be signed in to change notification settings - Fork 13
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
Adds code to build and upload Community Execution Environment via Github Action #76
Conversation
Build succeeded. ✔️ community-images-ansible-test-build SUCCESS in 26m 32s |
Build succeeded. ✔️ community-images-ansible-test-build SUCCESS in 24m 55s |
Build succeeded. ✔️ community-images-ansible-test-build SUCCESS in 24m 04s |
Build failed. ❌ community-images-ansible-test-build FAILURE in 17m 26s |
Build failed. ❌ community-images-ansible-test-build POST_FAILURE in 18m 05s |
Update to images/execution-environments/community-ee-base in the build step. Updates to ~/eercheck in the Test step.
Build succeeded. ✔️ community-images-ansible-test-build SUCCESS in 26m 29s |
3894a49
to
b3f8d5c
Compare
Build succeeded. ✔️ community-images-ansible-test-build SUCCESS in 26m 22s |
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.
Hi, thanks for working on this!
I did not look closely at every detail, but I did comment on a few things I noticed.
Also, should the docs be updated as well? :https://github.com/ansible-community/images/tree/main/docs/community-ee
I haven't looked at the steps which should be performed or not in the workflow in the first place (e.g., eertest), as I believe there is a RH policy on that. But IMO the tests and the entire flow could be simplified a bit more.
Thanks 😃
Yes, I will work on the documentation once this PR is merged. |
Build succeeded. ✔️ community-images-ansible-test-build SUCCESS in 21m 26s |
Build succeeded. ✔️ community-images-ansible-test-build SUCCESS in 17m 56s |
Build succeeded. ✔️ community-images-ansible-test-build SUCCESS in 18m 51s |
Build succeeded. ✔️ community-images-ansible-test-build SUCCESS in 25m 03s |
Build succeeded. ✔️ community-images-ansible-test-build SUCCESS in 26m 36s |
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 are also some more unresolved comments above.
Build failed. ❌ community-images-ansible-test-build FAILURE in 8m 19s |
Build failed. ❌ community-images-ansible-test-build FAILURE in 14m 45s |
Build failed. ❌ community-images-ansible-test-build FAILURE in 14m 16s |
Co-authored-by: Felix Fontein <felix@fontein.de>
Co-authored-by: Felix Fontein <felix@fontein.de>
Co-authored-by: Felix Fontein <felix@fontein.de>
Build succeeded. ✔️ community-images-ansible-test-build SUCCESS in 20m 01s |
@anweshadas thanks for working on this! |
Automates Community Execution Environment release process. This applies to both community-ee-minimal and community-ee-base.