data.coop/ansible
8
3
Fork 7
Code Issues 52 Pull requests 10 Projects 1 Releases Wiki Activity

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
Conversation 7 Commits 8 Files changed 45 +641 -462
valberg commented 2022-11-25 13:01:02 +00:00
Owner
Copy link

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).
👍 2
valberg added 1 commit 2022-11-25 13:01:03 +00:00
benjaoming commented 2022-11-25 13:20:41 +00:00
Owner
Copy link

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
valberg commented 2022-11-25 13:56:52 +00:00
Author
Owner
Copy link

@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.
benjaoming commented 2022-11-25 14:14:36 +00:00
Owner
Copy link

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.
reynir commented 2022-11-25 14:16:20 +00:00
Owner
Copy link

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.
valberg commented 2022-11-25 14:37:56 +00:00
Author
Owner
Copy link

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.
valberg commented 2022-11-25 14:39:59 +00:00
Author
Owner
Copy link

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.
👍 1
valberg added 1 commit 2022-11-25 14:57:50 +00:00
samsapti commented 2022-11-25 21:16:34 +00:00
Owner
Copy link

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
roles/docker/tasks/services/postfix.yml Outdated
@ -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:
samsapti commented 2022-11-25 21:28:48 +00:00
Owner
Copy link

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.
valberg commented 2022-11-25 21:34:41 +00:00
Author
Owner
Copy link

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.
samsapti commented 2022-11-25 21:36:08 +00:00
Owner
Copy link

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
roles/docker/files/configs/thelounge.js Outdated
@ -405,3 +405,3 @@
// @type string
//
url: "ldap://{{ ldap.domain }}",
url: "ldap://{{ services.openldap.domain }}",
samsapti commented 2022-11-25 21:38:28 +00:00
Owner
Copy link

a6420830e4

a6420830e4a3c78664c3a8849df723e7ee4ac795
valberg commented 2022-11-26 07:34:29 +00:00
Author
Owner
Copy link

@samsapti jeg skal bruge lidt mere info ;)

@samsapti jeg skal bruge lidt mere info ;)
samsapti commented 2022-11-26 15:20:35 +00:00
Owner
Copy link

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
samsapti commented 2022-11-25 23:22:32 +00:00
Owner
Copy link

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.
Reviewers
No reviewers
Labels
Clear labels   
Blocked

Something is blocking progress

  
Existing Service

Issues/improvements regarding existing services

  
Infrastructure Issue

Infrastructure issue that needs investigation or time to fix

  
Refactor

Refactoring of files / file structure or other improvements

  
Security Hardening

Non-verified / non-critical security improvements

  
Security Issue

Urgent security issue that needs to be fixed ASAP

  
Service Idea

Ideas for new services

  
Service Removal

Removal of services for one reason or another

  
Upgrade service
No labels
Blocked
Existing Service
Infrastructure Issue
Refactor
Security Hardening
Security Issue
Service Idea
Service Removal
Upgrade service
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
4 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: data.coop/ansible#125
No description provided.
Delete branch "unify_configurations"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?