Customization of the init_projects_container_image is now possible #1248
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
3 tasks
rooftopcellist
approved these changes
Feb 22, 2023
|
Thank you for the PR @mbutton77 ! This looks like it has everything needed. Once CI finishes, we can merge this. |
|
Can I know what exactly is project container be used for? Can I use it to store backups created by playbook ran via awx tower template? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
SUMMARY
When working in a closed (with no direct internet access) environment, you can override some Docker images, but there was still a hardcoded reference (init_projects_container_image which was referencing a centos9 image on quay.io).
I simply extended the existing mechanism for init_container_image var to the init_projects_container_image var.
fixes #1162
ISSUE TYPE
ADDITIONAL INFORMATION
Should be clear with the summary.