-
Notifications
You must be signed in to change notification settings - Fork 0
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
Moving HTTPD configuration to SSM parameters #2
Conversation
Will be testing with a proper management console deployment over the next few days. |
Kicking all of the new ssm parameters to their own TF file for sanity Pulling the virtualhost definitions out of the startup py and into actual ssm definitions, which should allow us to add additional virtualhosts down the line, if necessary
b3015c7
to
d85bc77
Compare
All right, I think this is finally tested and generally ready to merge in. Just to note, with all of the changes involved here, this line will need to be changed to whatever (actual) version number this will be built as. unity-proxy/terraform-unity/variables.tf Line 38 in 8cafef3
Right now, it's just pointing at a copy of the image I manually built and pushed. Unfortunately the management console doesn't respect the variables provided by the marketplace entry, so this line has to be hardcoded, and is the only control (until such time as the management console parameterizes it properly). |
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.
If we are just getting the proxy working and saving mgmt /location changes for another PR, then we need to make sure we're opening a ticket for that.
Also replacing every instance of deployment_name and install_prefix with project-venue
@mike-gangl @galenatjpl ready for actual merge, I think. |
Moving the httpd configuration for the unity-proxy container to SSM parameters, to let us fetch them on container startup.
Should ease the heartburn of unity-sds/unity-cs#403.