-
Notifications
You must be signed in to change notification settings - Fork 100
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
[REARCH] Container orchestrator #207
Merged
Merged
Commits on Feb 15, 2018
-
Revert to DeploymentConfig and remove server PV
This makes the entire app act as one server. The GUID is generated using a template parameter and is provided to the server pod using an environment variable. We ignore logs because we will be printing our logs to STDOUT which will be processed by the EFK log aggregation stack. We also remove certs because they will be handled by the httpd pod or by the route itself. This allows us to remove the "server" PV entirely.
Configuration menu - View commit details
-
Copy full SHA for 406ce3b - Browse repository at this point
Copy the full SHA 406ce3bView commit details -
Add images and alter template for containerized workers
Now we have a base image which contains the manageiq code base then we build the orchestrator image and worker images from there. The only image that contains the comiled assets is the UI image. Every image that runs a web server exposes only port 3000, so we make apache listen on that port and the UI worker binds to port 3001 A common liveness check script (manageiq_liveness_check) is used for all images which don't have a service. Each web server worker has it's own service which is pointed to by httpd directly. This means that we can also remove the "manageiq" service.
Configuration menu - View commit details
-
Copy full SHA for f1aeaef - Browse repository at this point
Copy the full SHA f1aeaefView commit details -
Configuration menu - View commit details
-
Copy full SHA for 2205ac8 - Browse repository at this point
Copy the full SHA 2205ac8View commit details -
Configuration menu - View commit details
-
Copy full SHA for 156710e - Browse repository at this point
Copy the full SHA 156710eView commit details -
Configuration menu - View commit details
-
Copy full SHA for f95df14 - Browse repository at this point
Copy the full SHA f95df14View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2ede1ba - Browse repository at this point
Copy the full SHA 2ede1baView commit details
Commits on Feb 16, 2018
-
We don't have a PV for the server anymore
Configuration menu - View commit details
-
Copy full SHA for 3e9acaa - Browse repository at this point
Copy the full SHA 3e9acaaView commit details -
Configuration menu - View commit details
-
Copy full SHA for 435e07b - Browse repository at this point
Copy the full SHA 435e07bView commit details
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.