-
Notifications
You must be signed in to change notification settings - Fork 19
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
Converge backbeat images #2600
base: development/9.0
Are you sure you want to change the base?
Converge backbeat images #2600
Conversation
Benefit is to avoid duplicating image, make sure we actually run the build we tested, and reduce dependency on federation. Additionally, switched to `ochinchina/supervisord`, so we can fully remove the python dependency and reduce image size. Differences are very few: - Install ballot and supervisord - Run as `scality` user - Run supervisord as entrypoint Issue: BB-633
Remove redundant shell command (implicit from using shell form of `CMD`) and use exec to run supervisord. Issue: BB-633
Hello francoisferrand,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files
@@ Coverage Diff @@
## development/9.0 #2600 +/- ##
================================================
Coverage 54.35% 54.35%
================================================
Files 201 201
Lines 13336 13336
================================================
Hits 7249 7249
Misses 6077 6077
Partials 10 10
Flags with carried forward coverage won't be shown. Click here to find out more. |
Build federation image as additional layers on top of "standard" backbeat image, to avoid duplication and ensure the image we tested is used in production.
There are still some differences at the moment (e.g. name of user, path) which should be further converged to minimize differences, may be done over time after considering all things.
This allows us to fully control (and minimize) what gets into the image, relying on any official node image. Also, now switch from python-based supervisord to a golang reimplementation [1], to minimize dependencies and CVEs.
Finally, took the change to fix startup command of the image, to ensure it properly propagates OS signals.
[1] https://github.com/ochinchina/supervisord
Issue: BB-633