Commit graph

17 commits

Author SHA1 Message Date
2b9b1d1c39 🚨 host: krypton: Fix fqcn linter warnings
fqcn[action-core]: Use FQCN for builtin module actions (file).
    host-krypton.yml:41 Use `ansible.builtin.file` or `ansible.legacy.file` instead.

    fqcn[action]: Use FQCN for module actions, such `community.docker.docker_container`.
    host-krypton.yml:52 Action `docker_container` is not FQCN.

    fqcn[action]: Use FQCN for module actions, such `community.docker.docker_container`.
    host-krypton.yml:117 Action `docker_container` is not FQCN.

    fqcn[action-core]: Use FQCN for builtin module actions (include_role).
    host-krypton.yml:130 Use `ansible.builtin.include_role` or `ansible.legacy.include_role` instead.
2025-03-09 19:00:21 +01:00
de7d285ee4 Merge pull request 'Fix linter warnings of type 'name'' () from alex/netz39-infra-ansible:lint into master
Reviewed-on: 
Reviewed-by: Stefan Haun <tux@netz39.de>
2025-03-05 18:13:58 +01:00
bbfff8f481 Update netz39/entities_validation_svc Docker tag to v1.0.4 2025-03-02 17:18:08 +00:00
bc43af38d9 🚨 Add names to plays to fix name[play] linter warnings
Adds basic descriptions on what those host are supposed to do.
2025-03-02 09:22:05 +01:00
b22c86758e 🚚 Migrate docker_host role to external project
We used a similar local role here, in the Freifunk Magdeburg project,
and in personal playbooks.  That was moved to an external project,
unified, and reworked, so the external role can act as replacement for
the distributed, redundant copies.

Link: https://github.com/netz39/ansible-role-host-docker
2025-02-23 19:18:21 +01:00
41ab4622b3 🚨 Fix trivial jinja spacing warnings
Fixes the following ansible-lint warnings:

    jinja[spacing]: Jinja2 spacing could be improved: group:Admins:{{ users | map(attribute = 'logname') | join("@pam,") }}@pam:: -> group:Admins:{{ users | map(attribute='logname') | join("@pam,") }}@pam:: (warning)
    host-beaker.yml:23 Jinja2 template rewrite recommendation: `group:Admins:{{ users | map(attribute='logname') | join("@pam,") }}@pam::`.

    jinja[spacing]: Jinja2 spacing could be improved: {{  docker_ip_ranges  }} -> {{  docker_ip_ranges }} (warning)
    host-krypton.yml:95 Jinja2 template rewrite recommendation: `{{  docker_ip_ranges }}`.

    jinja[spacing]: Jinja2 spacing could be improved: {{  item  }} -> {{  item }} (warning)
    host-krypton.yml:98 Jinja2 template rewrite recommendation: `{{  item }}`.

    jinja[spacing]: Jinja2 spacing could be improved: {{  docker_ip_ranges  }} -> {{  docker_ip_ranges }} (warning)
    host-krypton.yml:105 Jinja2 template rewrite recommendation: `{{  docker_ip_ranges }}`.

    jinja[spacing]: Jinja2 spacing could be improved: {{  item  }} -> {{  item }} (warning)
    host-krypton.yml:108 Jinja2 template rewrite recommendation: `{{  item }}`.

    jinja[spacing]: Jinja2 spacing could be improved: {{ cleanuri_amqp_user  }} -> {{ cleanuri_amqp_user }} (warning)
    roles/cleanuri/tasks/main.yml:36 Jinja2 template rewrite recommendation: `{{ cleanuri_amqp_user }}`.

    jinja[spacing]: Jinja2 spacing could be improved: {{ cleanuri_amqp_user  }} -> {{ cleanuri_amqp_user }} (warning)
    roles/cleanuri/tasks/main.yml:53 Jinja2 template rewrite recommendation: `{{ cleanuri_amqp_user }}`.

    jinja[spacing]: Jinja2 spacing could be improved: {{ cleanuri_amqp_user  }} -> {{ cleanuri_amqp_user }} (warning)
    roles/cleanuri/tasks/main.yml:70 Jinja2 template rewrite recommendation: `{{ cleanuri_amqp_user }}`.
2025-02-20 06:43:59 +01:00
605eca4c38 🚚 roles: dehydrated_cron: Use namespaced name
Removes the redundant words "ansible" and "role" from the role name
originating from the Git repo name, and uses the author's name as
namespace instead.  This makes it easier to recognize as external role.

Note: the host-wittgenstein recipe already used that new name, but we
did not set it up in requirements, yet.  (How did that ever work?)

Link: https://docs.ansible.com/ansible/latest/galaxy/user_guide.html#installing-multiple-roles-from-a-file
Fixes: f4db6fa395 ("Add Ansible setup for wittgenstein")
2024-12-28 12:16:21 +01:00
832094843a 🚚 roles: dehydrated: Use namespaced role name
In "ansible-role-dehydrated" the words "ansible" and "role" are
redundant, because implicit anyway.  What actually helps here is using a
namespaced name in the format galaxy would use, to make clear it's an
external role, and where it comes from.

Link: https://docs.ansible.com/ansible/latest/galaxy/user_guide.html#installing-multiple-roles-from-a-file
2024-12-28 12:07:20 +01:00
7e91c59631 chore(deps): update netz39/entities_validation_svc docker tag to v1.0.3 2024-10-23 15:30:16 +02:00
4e418220c2 chore(deps): update netz39/entities_validation_svc docker tag to v1.0.2 2024-08-07 14:18:27 +00:00
0f5c908997 chore(deps): update netz39/entities_validation_svc docker tag to v1.0.1 2024-02-26 22:37:01 +01:00
0afe932aa5 Change dehydrated_cron name for git dependency specification 2023-11-17 19:53:47 +01:00
74e0b9112d chore: cleanup docker image vars to enable renovate matching 2023-07-28 21:52:44 +02:00
ae7b65cc5f 🔧 Move docker_setup role application to group playbook
To install docker on a host you have to put it into that group in
inventory now, instead of adding the role to each host playbook.  Idea
is to extend the group docker_host playbook by more docker related
things as for example metrics and monitoring.
2022-12-21 19:00:22 +01:00
69d98b461d 🚨 Fix "wrong indentation" warnings
yamllint warned:

host-krypton.yml
  32:9      error    wrong indentation: expected 10 but found 8 (indentation)
2022-11-18 08:44:54 +01:00
6796e2f273 🔧 Add timezone information to docker containers
Docker does not take the tzdata from host systems, so the
timezone needs to be set individually.

Otherwise dates without explicit timezone information or Z will
be generated or interpreted in the wrong timezone
(such as dokuwiki signatures)
2022-11-10 13:05:18 +01:00
f11ddd9778 🚚 Rename host playbooks to host-* 2022-11-04 22:34:37 +01:00
Renamed from krypton.yml (Browse further)