Integrate dispatcher
Merge request reports
Activity
added configdb dispatcher compose.yaml labels
assigned to @bert.palm
added 3 commits
requested review from @bert.palm
removed review request for @bert.palm
requested review from @joost.hemmen1
added 3 commits
mentioned in merge request !237 (closed)
mentioned in merge request !251 (merged)
- Resolved by Joost Hemmen
Yup, this is a bit of a big one
I am a bit torn on putting all source code into one directory and copying/mounting that into the cron, configdb and dispatcher images.
On one hand it is really helpful not having to define libraries like crypto or the journal in several places but to easily share them between images.
On the other hand i liked having only the dedicated source files and python packages in the respective images/containers. But maybe that is a solution that we can works towards in the next step.
Edited by Joost Hemmen - Last reply by Bert Palm
- Resolved by Joost Hemmen
- Resolved by Joost Hemmen
tbh I don't exactly know how to review this fully.
I tested some general workflows, checked critical steps that I could think of. It works flawlessly.
As this is mostly integrating or moving established cron/dispatcher/configdb-updater code, I assume this is enough.
Beore deploying on prod we should check the above mentionend images/services a bit more in depth by testing some setup and sync jobs
Edited by Joost Hemmen - Last reply by Bert Palm
mentioned in commit 3be0eabe