gerrit: don't chown mariadb container directory

We are currently re-chowning the running db directories back to root,
causing havoc for the db.  Drop the explicit permissions to avoid
this.

Change-Id: I8d8ce5c62c660875d5c6eed54c686996576ec9df
This commit is contained in:
Ian Wienand 2021-11-05 08:18:27 +11:00
parent 4e84d29e38
commit 5f1c603b3d

View File

@ -285,16 +285,14 @@
when: gerrit_redirect_vhost is defined when: gerrit_redirect_vhost is defined
notify: gerrit Reload apache2 notify: gerrit Reload apache2
- name: Start gerrit # NOTE(ianw) This deliberately does not set owner/group/mode, as the
include_tasks: start.yaml # mariadb container chowns this directory to be owned by a
# container-internal user and drops root privileges. We don't want to
# reset this from outside the container.
- name: Setup reviewdb directory for mariadb - name: Setup reviewdb directory for mariadb
file: file:
state: directory state: directory
path: /home/gerrit2/reviewdb path: /home/gerrit2/reviewdb
owner: root
group: root
mode: 0755
- name: Set up root mariadb conf file - name: Set up root mariadb conf file
template: template:
@ -302,6 +300,9 @@
dest: /root/.gerrit_db.cnf dest: /root/.gerrit_db.cnf
mode: 0400 mode: 0400
- name: Start gerrit
include_tasks: start.yaml
- name: Set up cron job to optmize git repos - name: Set up cron job to optmize git repos
cron: cron:
name: optmize-git-repos name: optmize-git-repos