Collect versions and service information in docker/defaults/main.yml #125

Merged
valberg merged 8 commits from unify_configurations into main 2022-11-26 08:15:19 +00:00
Owner

Idéen her er at samle metadata om services i docker/default/main.yml.

Lige nu er det kun "version" der er trukket ud fra de enkelte service filer, men planen er også at inkludere beskrivelser, "vores badges", evt. health check endpoint, etc.

Målet er at når der pushes til ansible repoet at der bliver bygget et docker image der bliver deployet på services.data.coop med en oversigt over services, deres version, status (hvis servicen har noget det ligner et health endpoint) og hvilke af "vores badges" der gælder for dem. Dette gør at vi kan præsentere vores services på en måde der svarer til hvad der er deployet (eller i hvert fald skubbet til ansible).

Idéen her er at samle metadata om services i docker/default/main.yml. Lige nu er det kun "version" der er trukket ud fra de enkelte service filer, men planen er også at inkludere beskrivelser, "vores badges", evt. health check endpoint, etc. Målet er at når der pushes til ansible repoet at der bliver bygget et docker image der bliver deployet på services.data.coop med en oversigt over services, deres version, status (hvis servicen har noget det ligner et health endpoint) og hvilke af "vores badges" der gælder for dem. Dette gør at vi kan præsentere vores services på en måde der svarer til hvad der er deployet (eller i hvert fald skubbet til ansible).
valberg added 1 commit 2022-11-25 13:01:03 +00:00
Owner

God idé at gøre opmærksom på det gældende badge! Evt. med direkte link til manifestet i website repo. P.t. er det i new branch, men det bliver flyttet.

https://git.data.coop/data.coop/website/src/branch/new/content/services

God idé at gøre opmærksom på det gældende badge! Evt. med direkte link til manifestet i `website` repo. P.t. er det i `new` branch, men det bliver flyttet. https://git.data.coop/data.coop/website/src/branch/new/content/services
valberg force-pushed unify_configurations from 4f7eab9d8d to 6708653c94 2022-11-25 13:54:41 +00:00 Compare
Author
Owner

@benjaoming Mit mål er at gøre "services" siden på websitet overflødigt - eller måske endda bare gøre det til en iframe der inkluderer services.data.coop

Netop fordi vi har ansible som "single source of truth" og vi kan undgå dobbeltarbejde.

@benjaoming Mit mål er at gøre "services" siden på websitet overflødigt - eller måske endda bare gøre det til en iframe der inkluderer services.data.coop Netop fordi vi har ansible som "single source of truth" og vi kan undgå dobbeltarbejde.
Owner

Bare kommunikationen på hjemmesiden er første prio :) men ja, hvis det er yaml eller JSON kan vi nemt importere det via Hugo. Hugo kan faktisk kalde APIer build-time. Vi kan også importere et repo build time og læse YAMLen.

Bare kommunikationen på hjemmesiden er første prio :) men ja, hvis det er yaml eller JSON kan vi nemt importere det via Hugo. Hugo kan faktisk kalde APIer build-time. Vi kan også importere et repo build time og læse YAMLen.
Owner

Måske værd at kigge på data.coop/dokumenter#19 til ansible repoet også :) jeg har ikke kigget på hvordan vi inkluderer vedtægter og aup endnu.

Måske værd at kigge på https://git.data.coop/data.coop/dokumenter/pulls/19 til ansible repoet også :) jeg har ikke kigget på hvordan vi inkluderer vedtægter og aup endnu.
Author
Owner

Bare kommunikationen på hjemmesiden er første prio :) men ja, hvis det er yaml eller JSON kan vi nemt importere det via Hugo. Hugo kan faktisk kalde APIer build-time. Vi kan også importere et repo build time og læse YAMLen.

Jeg tror jeg er uenig i at kommunikationen af hvilke services vi har og deres status er nødvendig at have på hjemmesiden - jeg synes at et link er nok, hvis det betyder at vi kan lave en bedre og mere overskuelig oversigt over vores services.

Min plan er at bygge et site som er seperat fra hjemmesiden - først og fremmest for at vise status over vores services. På sigt håber jeg at det kan blive erstatningen for hjemmesidens "tjenester" side.

> Bare kommunikationen på hjemmesiden er første prio :) men ja, hvis det er yaml eller JSON kan vi nemt importere det via Hugo. Hugo kan faktisk kalde APIer build-time. Vi kan også importere et repo build time og læse YAMLen. Jeg tror jeg er uenig i at kommunikationen af hvilke services vi har og deres status er nødvendig at have på hjemmesiden - jeg synes at et link er nok, hvis det betyder at vi kan lave en bedre og mere overskuelig oversigt over vores services. Min plan er at bygge et site som er seperat fra hjemmesiden - først og fremmest for at vise status over vores services. På sigt håber jeg at det kan blive erstatningen for hjemmesidens "tjenester" side.
Author
Owner

Hvis det skal være rigtig vildt så hoster vi services|status.data.coop på en helt anden maskine end den hvor vi kører alt andet fra, så den er tilgængelig selvom at serveren(e) er nede.

Hvis det skal være rigtig vildt så hoster vi `services|status.data.coop` på en helt anden maskine end den hvor vi kører alt andet fra, så den er tilgængelig selvom at serveren(e) er nede.
valberg added 1 commit 2022-11-25 14:57:50 +00:00
Owner

Jeg kan især godt lide at vi har versionsnumre stående samlet, da det vil give et bedre overblik ift. når vi skal opgradere services (#123).

Jeg kan især godt lide at vi har versionsnumre stående samlet, da det vil give et bedre overblik ift. når vi skal opgradere services (#123).
samsapti requested changes 2022-11-25 21:24:53 +00:00
Dismissed
samsapti dismissed samsapti's review 2022-11-25 21:27:00 +00:00
samsapti reviewed 2022-11-25 21:27:20 +00:00
samsapti reviewed 2022-11-25 21:28:48 +00:00
@ -19,1 +18,4 @@
ALLOWED_SENDER_DOMAINS: "{{ allowed_sender_domains|join(' ') }}"
HOSTNAME: "smtp.data.coop" # the name the smtp server will identify itself as
vars:
allowed_sender_domains:
Owner

Jeg synes det her bør flyttes tilbage til defaults/main.yml, så vi har alle vars samlet ét sted.

Jeg synes det her bør flyttes tilbage til `defaults/main.yml`, så vi har alle vars samlet ét sted.
Author
Owner

Det er fordi det ikke kan defineres i selve services dictionary'en da der refereres til services dictionary'en før den en færdig defineret. Jeg har muligvis en anden løsning coming up.

Det er fordi det ikke kan defineres i selve services dictionary'en da der refereres til services dictionary'en før den en færdig defineret. Jeg har muligvis en anden løsning coming up.
Owner

Aaah, det giver selvfølgelig mening.

Aaah, det giver selvfølgelig mening.
valberg marked this conversation as resolved
samsapti reviewed 2022-11-25 21:38:28 +00:00
@ -405,3 +405,3 @@
// @type string
//
url: "ldap://{{ ldap.domain }}",
url: "ldap://{{ services.openldap.domain }}",
Owner
a6420830e4a3c78664c3a8849df723e7ee4ac795
Author
Owner

@samsapti jeg skal bruge lidt mere info ;)

@samsapti jeg skal bruge lidt mere info ;)
Owner

Vi har ikke thelounge mere, så hele den fil bør fjernes :)

Vi har ikke thelounge mere, så hele den fil bør fjernes :)
samsapti added the
Refactor
label 2022-11-25 21:49:28 +00:00
valberg added 1 commit 2022-11-25 23:12:03 +00:00
samsapti reviewed 2022-11-25 23:22:32 +00:00
playbook.yml Outdated
@ -15,2 +16,4 @@
enabled_services:
- nginx_proxy
- postfix
- openldap
Owner

Hvis den her er tilbage, hvad er så forskellen på det, og det vi har på main? Det her virker meget mere komplekst.

Hvis den her er tilbage, hvad er så forskellen på det, og det vi har på `main`? Det her virker meget mere komplekst.
samsapti marked this conversation as resolved
valberg force-pushed unify_configurations from d372a6ece6 to 55c9a346b1 2022-11-25 23:29:53 +00:00 Compare
valberg added 1 commit 2022-11-25 23:31:34 +00:00
samsapti added 2 commits 2022-11-25 23:37:04 +00:00
samsapti added 1 commit 2022-11-25 23:52:19 +00:00
samsapti force-pushed unify_configurations from cfb78eac12 to 2603852f1c 2022-11-25 23:53:34 +00:00 Compare
valberg merged commit d15e7e562f into main 2022-11-26 08:15:19 +00:00
Sign in to join this conversation.
No description provided.