Merge pull request #3 from NUZAT-TABASSUM/dev

Enable automatic linting/molecule tests and Debian support (relates to #1,#2 )

Author: wsmirnow

.ansible-lint

@@ -0,0 +1,7 @@
+---
+# .ansible-lint
+exclude_paths:
+  - .cache/
+  - .github/
+  - venv/
+  - .ansible/

.dev_requirements.txt

@@ -0,0 +1,9 @@
+ansible
+ansible-core>=2.11
+ansible-lint
+dnspython
+yamllint
+molecule
+molecule-plugins
+molecule-plugins[docker]
+molecule-plugins[podman]

.github/workflows/lint.yml

@@ -0,0 +1,21 @@
+---
+name: lint
+
+on:  # yamllint disable-line rule:truthy
+  push:
+  pull_request:
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: install dependencies
+        run: >
+          pip install -r .dev_requirements.txt
+
+      - run: yamllint --strict -c .yamllint .
+
+      - run: ansible-lint

.github/workflows/molecule.yml

@@ -0,0 +1,22 @@
+---
+
+name: molecule
+
+on:  # yamllint disable-line rule:truthy
+  - push
+  - pull_request
+
+jobs:
+  molecule:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Install dependencies
+        run: pip3 install -r .dev_requirements.txt
+
+      - name: Test playbook
+        run: molecule test -- -e opencast_postgresql_password=123
+        env:
+          PY_COLORS: '1'

.gitignore

@@ -0,0 +1,2 @@
+venv/
+.ansible/

.yamllint

@@ -0,0 +1,36 @@
+---
+# Yamllint configuration should be compatible with Ansible,
+# see https://ansible.readthedocs.io/projects/lint/rules/yaml/#yamllint-configuration
+
+extends: default
+
+rules:
+  comments:
+    # https://github.com/prettier/prettier/issues/6780
+    min-spaces-from-content: 1
+  # https://github.com/adrienverge/yamllint/issues/384
+  comments-indentation: false
+  document-start: disable
+  # 160 chars was the default used by old E204 rule, but
+  # you can easily change it or disable in your .yamllint file.
+  line-length:
+    max: 200
+  # We are adding an extra space inside braces as that's how prettier does it
+  # and we are trying not to fight other linters.
+  braces:
+    min-spaces-inside: 0 # yamllint defaults to 0
+    max-spaces-inside: 1 # yamllint defaults to 0
+  # key-duplicates:
+  #   forbid-duplicated-merge-keys: true # not enabled by default
+  octal-values:
+    forbid-implicit-octal: true # yamllint defaults to false
+    forbid-explicit-octal: true # yamllint defaults to false
+  # quoted-strings:
+  #   quote-type: double
+  #   required: only-when-needed
+
+
+ignore: |
+  .cache/
+  .github/
+  venv/

defaults/main.yml

@@ -1,6 +1,6 @@
 ---
 
-opencast_postgresql_version: 12
+opencast_postgresql_version: 16
 opencast_postgresql_user: opencast
 opencast_postgresql_database: opencast
 opencast_postgresql_connection_hosts:

handlers/main.yml

@@ -1,5 +1,12 @@
 ---
-- name: restart postgresql
-  service:
-    name: postgresql
-    state: restarted
+- name: Restart Postgresql On CentOS
+  ansible.builtin.service:
+   name: "postgresql-{{ opencast_postgresql_version }}"
+   state: restarted
+  when: ansible_os_family == "RedHat"
+
+- name: Restart Postgresql On Debian/Ubuntu
+  ansible.builtin.service:
+   name: postgresql
+   state: restarted
+  when: ansible_os_family == "Debian"

meta/main.yml

@@ -1,16 +1,29 @@
+---
 galaxy_info:
   author: Lars Kiesow
   role_name: opencast_postgresql
   namespace: elan
   company: ELAN e.V.
   description: Install and prepare PostgreSQL for Opencast
   license: BSD-3-Clause
-  min_ansible_version: 2.9
+  min_ansible_version: "2.9"
+
   galaxy_tags:
     - opencast
+
   platforms:
     - name: EL
       versions:
-        - 8
+        - "8"
+        - "9"
+    - name: Debian
+      versions:
+        - "bullseye"
+        - "bookworm"
+    - name: Ubuntu
+      versions:
+        - "focal"
+        - "jammy"
+        - "noble"
 
 dependencies: []

molecule/default/Dockerfile.j2

@@ -0,0 +1,8 @@
+FROM {{ item.image }}
+
+# Install dependencies
+{% if "centos" in item.name %}
+RUN dnf install --refresh -y sudo systemd bash ca-certificates iproute pam passwd shadow-utils python39 python3-libselinux 
+{% else %}
+RUN apt-get update && apt-get install -y sudo init bash ca-certificates iproute2 python3 && apt-get clean
+{% endif %}