-
Notifications
You must be signed in to change notification settings - Fork 41
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
Add update_container_documentation script and monitoring profile #1589
base: main
Are you sure you want to change the base?
Add update_container_documentation script and monitoring profile #1589
Conversation
bwbroersma
commented
Dec 9, 2024
•
edited
Loading
edited
- Fixes Options to only create main part of internet.nl Batch API (Django) #1588
The number of profiles seems to be rapidly expanding. We've had a dev misconfiguration for them already. I think we need one place in the deployment docs where they are all listed, along with defaults and recommendations. Particularly as misconfiguration may not be immediately obvious. |
Agree, also wonder if all services with the It seems after this addition there are these profiles defined in the compose files:
I see Line 5 in 0e5a6d1
Internet.nl/docker/batch-test.env Line 11 in 0e5a6d1
However, its unused in all compose files. |
@aequitas I think we discusses this also, and some compose profiles ( |
Idea:
|
$ yq -r '"container|profiles|description","-|-|-",.services|to_entries|map([.key,(.value.profiles //[]|join(", "),(.key|head_comment))]|join("|"))[]' compose.yaml|sed 's/$/|/'|column -ts\| -o' | '|sed 's/^/| /;s/ $//;2{s/ /-/g}' using $ yq -r '"container|profiles|description",.services|to_entries|map([.key,(.value.profiles //[]|join(", "),(.key|head_comment))]|join("|"))[]' compose.yaml|markdown-table -s\|
Questions:
Todo:
|
affbb0a
to
120c9f9
Compare
feb43b9
to
6a06207
Compare
6a06207
to
667d7da
Compare
I've also added an auto-make workflow for PRs in: |
Internet.nl/documentation/Docker-deployment.md Lines 387 to 393 in 583c017
Should be updated too. |