test: fix syntax for ansible lint

.forgejo/workflows/molecule.yml

@@ -0,0 +1,18 @@
+---
+on: [push]
+jobs:
+  lint:
+    runs-on: docker
+    container:
+      image: code.waks.be/nishiki/molecule:docker
+    steps:
+      - uses: actions/checkout@v3
+      - run: ansible-lint .
+      - run: yamllint .
+  molecule:
+    runs-on: docker
+    container:
+      image: code.waks.be/nishiki/molecule:docker
+    steps:
+      - uses: actions/checkout@v3
+      - run: molecule test

.gitlab-ci.yml

@@ -0,0 +1,10 @@
+---
+image: nishiki/molecule:docker
+
+before_script:
+  - molecule --version
+
+molecule:
+  stage: test
+  script:
+    - molecule test

.yamllint

@@ -2,11 +2,11 @@
 extends: default
 
 ignore: |
-  .kitchen/*
+  .kitchen*
   vendor/
+  .forgejo/
 
 rules:
   line-length:
     max: 120
     level: warning
-  truthy: false

CHANGELOG.md

@@ -5,6 +5,10 @@ Which is based on [Keep A Changelog](http://keepachangelog.com/)
 
 ## Unreleased
 
+### Fixed
+
+- fix: change gpg key
+
 ## v2.1.0 - 2021-08-22
 
 ### Added
@@ -19,6 +23,7 @@ Which is based on [Keep A Changelog](http://keepachangelog.com/)
 
 - chore: use FQCN for module name
 - fix: replace no_log to loop label
+- test: use personal docker registry
 
 ## v2.0.1 - 2020-03-28
 

README.md

@@ -1,22 +1,22 @@
 # Ansible role: MariaDB
 
-[![Version](https://img.shields.io/badge/latest_version-2.1.0-green.svg)](https://git.yaegashi.fr/nishiki/ansible-role-mariadb/releases)
+[![Version](https://img.shields.io/badge/latest_version-2.1.0-green.svg)](https://code.waks.be/nishiki/ansible-role-mariadb/releases)
-[![Build Status](https://travis-ci.org/nishiki/ansible-role-mariadb.svg?branch=master)](https://travis-ci.org/nishiki/ansible-role-mariadb)
+[![License](https://img.shields.io/badge/license-Apache--2.0-blue.svg)](https://code.waks.be/nishiki/ansible-role-mariadb/src/branch/master/LICENSE)
-[![License](https://img.shields.io/badge/license-Apache--2.0-blue.svg)](https://git.yaegashi.fr/nishiki/ansible-role-mariadb/src/branch/master/LICENSE)
+[![Build](https://code.waks.be/nishiki/ansible-role-mariadb/actions/workflows/molecule.yml/badge.svg?branch=main)](https://code.waks.be/nishiki/ansible-role-mariadb/actions?workflow=molecule.yml)
 
 Install and configure MariaDB
 
 ## Requirements
 
-* Ansible >= 2.9
+- Ansible >= 2.9
-* Debian
+- Debian
-  * Buster
+  - Bullseye
-  * Bullseye
+  - Bookworm
 
 ## Role variables
 
 | Name                            | Type  | Required | Default | Comment                                |
-|---------------------------------|-------|----------|--------|----------------------------------------|
+| ------------------------------- | ----- | -------- | ------- | -------------------------------------- |
 | mariadb_use_official_repository | bool  | no       | true    | use the official repository            |
 | mariadb_branch                  | str   | no       | 10.3    | the branch version to install          |
 | mariadb_user                    | str   | no       | root    | login to connect on mariadb            |
@@ -30,7 +30,7 @@ Install and configure MariaDB
 ### mariadb_users
 
 | Name       | Type  | Required | Default | Comment                                                   |
-|------------|-------|----------|---------|-----------------------------------------------------------|
+| ---------- | ----- | -------- | ------- | --------------------------------------------------------- |
 | name       | str   | yes      |         | the username                                              |
 | host       | str   | yes      |         | the mysql user host                                       |
 | password   | str   | yes      |         | the user password                                         |
@@ -51,7 +51,7 @@ Example:
 ### mariadb_databases
 
 | Name  | Type | Required | Default | Comment                                      |
-|------------|-------|----------|---------|-----------------------------------------------------------|
+| ----- | ---- | -------- | ------- | -------------------------------------------- |
 | name  | str  | yes      |         | the dabase name                              |
 | state | str  | no       | present | if state is `absent` the database is deleted |
 
@@ -97,10 +97,10 @@ Example:
 
 ### Test with molecule and docker
 
-* install [docker](https://docs.docker.com/engine/installation/)
+- install [docker](https://docs.docker.com/engine/installation/)
-* install `python3` and `python3-pip`
+- install `python3` and `python3-pip`
-* install molecule and dependencies `pip3 install molecule molecule-docker docker ansible-lint pytest-testinfra yamllint`
+- install molecule and dependencies `pip3 install molecule molecule-docker docker ansible-lint pytest-testinfra yamllint`
-* run `molecule test`
+- run `molecule test`
 
 ## License
 

defaults/main.yml

@@ -1,13 +1,13 @@
 ---
-mariadb_use_official_repository: yes
+mariadb_use_official_repository: true
 mariadb_branch: 10.6
 mariadb_repository: |
   deb http://mariadb.mirrors.ovh.net/MariaDB/repo/{{ mariadb_branch }}/debian {{ ansible_distribution_release }} main
 mariadb_repository_key_id: '0xF1656F24C74CD1D8'
-mariadb_repository_key_server: keyserver.ubuntu.com
+mariadb_repository_key_server: hkp://keyserver.ubuntu.com:80
 mariadb_user: root
-mariadb_master: no
+mariadb_master: false
-mariadb_autorestart: no
+mariadb_autorestart: false
 mariadb_users: []
 mariadb_databases: []
 mariadb_config: {}

handlers/main.yml

@@ -1,5 +1,5 @@
 ---
-- name: restart mariadb
+- name: Restart mariadb
   ansible.builtin.service:
     name: mysql
     state: restarted

meta/main.yml

@@ -1,17 +1,18 @@
 ---
 galaxy_info:
   role_name: mariadb
+  namespace: nishiki
   author: Adrien Waksberg
   company: Adrien Waksberg
   description: Install and configure mariadb
   license: Apache2
-  min_ansible_version: 2.9
+  min_ansible_version: "2.9"
 
   platforms:
     - name: Debian
       versions:
-        - buster
         - bullseye
+        - bookworm
 
   galaxy_tags:
     - database

molecule/default/molecule.yml

@@ -2,19 +2,20 @@
 driver:
   name: docker
 platforms:
-  - name: debian11
+  - name: debian12
-    image: nishiki/debian11:molecule
+    image: code.waks.be/nishiki/molecule:debian12
     privileged: true
     volumes:
-      - /sys/fs/cgroup:/sys/fs/cgroup:ro
+      - /sys/fs/cgroup:/sys/fs/cgroup:rw
+    cgroupns_mode: host
     command: /bin/systemd
     capabilities:
       - SYS_ADMIN
 provisioner:
   inventory:
     host_vars:
-      debian11:
+      debian12:
-        mariadb_use_official_repository: yes
+        mariadb_use_official_repository: false
 lint: |
   set -e
   yamllint .

tasks/initdb.yml

@@ -1,5 +1,5 @@
 ---
-- name: create the data directory
+- name: Create the data directory
   ansible.builtin.file:
     path: '{{ mariadb_full_config.mysqld.datadir }}'
     owner: '{{ mariadb_full_config.mysqld.user }}'
@@ -8,7 +8,7 @@
     state: directory
   tags: mariadb
 
-- name: initialize database
+- name: Initialize database  # noqa no-changed-when
   ansible.builtin.command:
   args:
     argv:
@@ -17,35 +17,33 @@
       - '--user={{ mariadb_full_config.mysqld.user }}'
       - '--skip-name-resolve'
       - '--no-defaults'
-  when: True
   tags: mariadb
 
-- name: start mariadb
+- name: Start mariadb
   ansible.builtin.service:
     name: mysql
     state: started
   tags: mariadb
 
-- name: wait mysql start
+- name: Wait mysql start
   ansible.builtin.pause:
     seconds: 5
   tags: mariadb
 
-- name: set root password
+- name: Set root password  # noqa no-changed-when
   ansible.builtin.command: 'mysqladmin -u root password {{ mariadb_password }}'
-  when: True
   tags: mariadb
 
-- name: remove anonymous users
+- name: Remove anonymous users
   community.mysql.mysql_user:
     name: ''
-    host_all: yes
+    host_all: true
     state: absent
     login_user: '{{ mariadb_user }}'
     login_password: '{{ mariadb_password }}'
   tags: mariadb
 
-- name: remove root user without password
+- name: Remove root user without password
   community.mysql.mysql_user:
     name: root
     host: '{{ item }}'

tasks/main.yml

@@ -1,5 +1,5 @@
 ---
-- name: install dependencies packages
+- name: Install dependencies packages
   ansible.builtin.package:
     name:
       - gpg
@@ -9,10 +9,11 @@
   until: result is succeeded
   tags: mariadb
 
-- import_tasks: repo.yml
+- name: Install official repository
+  ansible.builtin.import_tasks: repo.yml
   when: mariadb_use_official_repository|bool
 
-- name: install package
+- name: Install package
   ansible.builtin.package:
     name:
       - mariadb-server
@@ -21,57 +22,58 @@
   until: result is succeeded
   tags: mariadb
 
-- name: copy configuration file
+- name: Copy configuration file
   ansible.builtin.template:
     src: my.cnf.j2
     dest: /etc/mysql/my.cnf
     owner: root
     group: root
     mode: 0644
-  notify: restart mariadb
+  notify: Restart mariadb
   tags: mariadb
 
-- name: check if datadir exist
+- name: Check if datadir exist
   ansible.builtin.stat:
-    path: '{{ mariadb_full_config.mysqld.datadir }}'
+    path: "{{ mariadb_full_config.mysqld.datadir }}"
   register: st
   tags: mariadb
 
-- import_tasks: initdb.yml
+- name: Initialize the database
+  ansible.builtin.import_tasks: initdb.yml
   when: not st.stat.exists
 
-- name: enable and start the service
+- name: Enable and start the service
   ansible.builtin.service:
     name: mysql
     state: started
     enabled: true
   tags: mariadb
 
-- name: manage users
+- name: Manage users
   community.mysql.mysql_user:
-    name: '{{ item.name }}'
+    name: "{{ item.name }}"
-    host: '{{ item.host }}'
+    host: "{{ item.host }}"
-    password: '{{ item.password }}'
+    password: "{{ item.password }}"
-    priv: '{{ item.privileges|default("")|join("/") }}'
+    priv: "{{ item.privileges | default('') | join('/') }}"
-    state: '{{ item.state|default("present") }}'
+    state: "{{ item.state | default('present') }}"
-    login_user: '{{ mariadb_user }}'
+    login_user: "{{ mariadb_user }}"
-    login_password: '{{ mariadb_password }}'
+    login_password: "{{ mariadb_password }}"
-  loop: '{{ mariadb_users }}'
+  loop: "{{ mariadb_users }}"
   loop_control:
-    label: '{{ item.name }}@{{ item.host }}'
+    label: "{{ item.name }}@{{ item.host }}"
   run_once: true
   when: mariadb_master|bool
   tags: mariadb
 
-- name: manage databases
+- name: Manage databases
   community.mysql.mysql_db:
-    name: '{{ item.name }}'
+    name: "{{ item.name }}"
-    state: '{{ item.state|default("present") }}'
+    state: "{{ item.state | default('present') }}"
-    login_user: '{{ mariadb_user }}'
+    login_user: "{{ mariadb_user }}"
-    login_password: '{{ mariadb_password }}'
+    login_password: "{{ mariadb_password }}"
-  loop: '{{ mariadb_databases }}'
+  loop: "{{ mariadb_databases }}"
   loop_control:
-    label: '{{ item.name }}'
+    label: "{{ item.name }}"
   run_once: true
   when: mariadb_master|bool
   tags: mariadb

tasks/repo.yml

@@ -1,14 +1,10 @@
 ---
-- name: add gpg key for mariadb repository
+- name: Add gpg key for mariadb repository
   ansible.builtin.apt_key:
-    keyserver: '{{ mariadb_repository_key_server }}'
+    url: https://supplychain.mariadb.com/MariaDB-Server-GPG-KEY
-    id: '{{ mariadb_repository_key_id }}'
-  retries: 2
-  register: result
-  until: result is succeeded
   tags: mariadb
 
-- name: add official repo
+- name: Add official repo
   ansible.builtin.apt_repository:
-    repo: '{{ mariadb_repository }}'
+    repo: "{{ mariadb_repository }}"
   tags: mariadb