feat/nextcloud_compose #2
No reviewers
bobbyblues
Labels
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
bbl-ansible/services!2
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "ezwen/services:feat/nextcloud_compose"
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?
WIP: feat/nextcloud_composeto feat/nextcloud_compose@ -12,7 +12,7 @@bbl_use_systemd: falsebbl_prevent_reboot: truebbl_extra_packages:- python-pexpectAh bah oui, mais ça casse la compatibilité avec arch, qui est l'image molecule sur ma collection ;)
Oui au final j'ai clairement volontairement anticipé Mandragot… inconsciemment j'étais déjà dans le scénario "tout basculer dans la collection Mandragot" 😄
@ -0,0 +22,4 @@platforms:- name: molecule_mgt_deploy_nextcloud# Image to be used as a baseimage: forge.mandragot.org/mandragot-docker/debian:bookwormComme dit plus haut, ça c'est pour mandragot, mais ici j'utilise mon image arch (qui correspond à mon infra).
Idem
@ -2,12 +2,14 @@bbl_admin_username: adminbbl_cache_docker_tag: 7-alpinebbl_cache_image: docker.io/redisIci ça devrait être
docker.io/library/redis.Oui on peut pas de pb, mais je ne me souviens plus pourquoi c'était important de mettre explicitement le
library? Pour le proxy ?@ -4,3 +4,3 @@bbl_cache_image: docker.io/redisbbl_db_docker_tag: 10.5.9bbl_db_docker_tag: 11.4bbl_db_image: docker.io/mariadbdocker.io/library/mariadbIdem
@ -7,3 +7,3 @@bbl_default_phone_region: FRbbl_nextcloud_docker_tag: "27"bbl_nextcloud_docker_tag: "31"bbl_nextcloud_image: docker.io/nextclouddocker.io/library/nextcloudIdem
@ -31,3 +19,1 @@- "{{ bbl_volume_directory }}/{{ bbl_service_name }}/data-cache"- "{{ bbl_volume_directory }}/{{ bbl_service_name }}/data-db"- "{{ bbl_volume_directory }}/{{ bbl_service_name }}/data-nextcloud"- path: "{{ bbl_volume_directory }}/{{ bbl_service_name }}"Je trouve plus pratique de créer une variable
_bbl_volume_directory: "{{ bbl_volume_directory }}/{{ bbl_service_name }}dansvars/main.ymlet de l'utiliser partout. Ça simplifie un peu le code.Oui je suis d'accord. J'avais gardé comme c'était dans l'immédiat (objectif compose surtout), mais autant améliorer ça.
@ -34,0 +23,4 @@owner: "999"group: "999"- path: "{{ bbl_volume_directory }}/{{ bbl_service_name }}/data-db"owner: "{{ ansible_facts.user_uid }}"Ici il faudrait l'utilisateur
33et le groupe33.Pourtant Redis alpine est bien sur un user 999 ?
Cf
redis/docker-library-redis@8338d86bc3/7.4/alpine/Dockerfile (L13)(avec group 1000 d'ailleurs, by bad)
Je ne parle pas de redis, je parle de mariadb :)
@ -34,0 +26,4 @@owner: "{{ ansible_facts.user_uid }}"group: "{{ ansible_facts.user_gid }}"- path: "{{ bbl_volume_directory }}/{{ bbl_service_name }}/data-nextcloud"owner: "{{ ansible_facts.user_uid }}"Ici il faudrait l'utilisateur
999et le groupe999.Oui en effet !
@ -19,6 +19,21 @@changed_when: falsewhen: _bbl_new_install- name: Database is configured in config.phpJ'imagine que ça c'est pour palier un souci de db. Le
Initializing finishedde la tâche au dessus fonctionne bien même si la db est mal configurée ?Oui, en fait c'est justement au premier lancement qu'il configure la connexion à la db, puis n'y retouchera jamais pas la suite. Donc on peut considérer qu'au premier lancement c'est à la charge de l'image de faire ce boulot, et par la suite nous on a cette nouvelle tâche qui s'assure que c'est bon et qui met à jour au besoin.
@ -9,3 +6,1 @@- container_cache- container_nextcloud- container_cron_bbl_volume_directory: "{{ bbl_volume_directory }}/{{ bbl_service_name }}"Ah bah il est bien là ^^ Pourquoi ne pas l'utiliser dans
main.ymletcompose.yml.j2?indeed!
Pour les droits sur les répertoires, après vérif à coup de
cat /etc/passwddans chaque image utilisée, je pars sur ceci :Normalement tout est fixed, et re-testé dans molecule avec succès ✅
Donc le plan :
bblen `mgtmgt.servicesà coup de copier-coller de cette brancheView command line instructions
Checkout
From your project repository, check out a new branch and test the changes.Merge
Merge the changes and update on Forgejo.Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.