🚨 Fix linter warnings of type 'name[casing]'
ansible-lint reads like this: name[casing]: All names should start with an uppercase letter. While at it: Some task/handler names were slightly adapted to better match what should be done.
This commit is contained in:
parent
266ba39473
commit
8b2ab7753d
21 changed files with 42 additions and 42 deletions
host-beaker.ymlhost-platon.ymlhost-pottwal.ymlhost-radon.ymlsetup-ssh.yml
roles
apache
apache_letsencrypt
dd24_dyndns_cron
desec_dyndns_cron
nfs_host
nginx_https_ingress
setup_http_site_forward
setup_http_site_proxy
|
@ -9,7 +9,7 @@
|
|||
|
||||
tasks:
|
||||
|
||||
- name: enable proxmox gui login for admin users
|
||||
- name: Enable proxmox gui login for admin users
|
||||
ansible.builtin.lineinfile:
|
||||
path: /etc/pve/user.cfg
|
||||
regexp: "^user:{{ item.logname }}@pam"
|
||||
|
@ -18,7 +18,7 @@
|
|||
state: present
|
||||
loop: "{{ users }}"
|
||||
|
||||
- name: configure proxmox admin group
|
||||
- name: Configure proxmox admin group
|
||||
ansible.builtin.lineinfile:
|
||||
path: /etc/pve/user.cfg
|
||||
regexp: "^group:Admins:"
|
||||
|
|
|
@ -63,7 +63,7 @@
|
|||
owner: root
|
||||
group: root
|
||||
mode: '0644'
|
||||
notify: restart mosquitto
|
||||
notify: Restart mosquitto service
|
||||
|
||||
|
||||
### Sesam for SSH access
|
||||
|
@ -245,7 +245,7 @@
|
|||
owner: root
|
||||
group: root
|
||||
mode: "0644"
|
||||
notify: restart rsyslog
|
||||
notify: Restart rsyslog
|
||||
|
||||
|
||||
### Asterisk
|
||||
|
@ -258,7 +258,7 @@
|
|||
owner: root
|
||||
group: root
|
||||
mode: "0644"
|
||||
notify: restart asterisk
|
||||
notify: Restart asterisk
|
||||
|
||||
- name: Set up extensions for asterisk
|
||||
# This uses the variables gatekeeper_user and door_open_command
|
||||
|
@ -268,14 +268,14 @@
|
|||
owner: root
|
||||
group: root
|
||||
mode: "0644"
|
||||
notify: restart asterisk
|
||||
notify: Restart asterisk
|
||||
|
||||
- name: Ensure asterisk is in the right groups
|
||||
ansible.builtin.user:
|
||||
name: asterisk
|
||||
groups: audio,i2c,gpio
|
||||
append: yes
|
||||
notify: restart asterisk
|
||||
notify: Restart asterisk
|
||||
|
||||
# Asterisk now executes shell scripts with reduced privileges, so we need to
|
||||
# use sudo for I2C access.
|
||||
|
@ -304,19 +304,19 @@
|
|||
|
||||
|
||||
handlers:
|
||||
- name: restart mosquitto
|
||||
- name: Restart mosquitto service
|
||||
service:
|
||||
name: mosquitto
|
||||
state: restarted
|
||||
enabled: yes
|
||||
|
||||
- name: restart rsyslog
|
||||
- name: Restart rsyslog
|
||||
service:
|
||||
name: rsyslog
|
||||
state: restarted
|
||||
enabled: yes
|
||||
|
||||
- name: restart asterisk
|
||||
- name: Restart asterisk
|
||||
service:
|
||||
name: asterisk
|
||||
state: restarted
|
||||
|
|
|
@ -642,7 +642,7 @@
|
|||
src: templates/pottwal/renovate-cron.j2
|
||||
dest: /etc/cron.hourly/renovate-bot
|
||||
mode: "0700"
|
||||
notify: reload cron
|
||||
notify: Reload cron
|
||||
tags:
|
||||
- renovate
|
||||
|
||||
|
@ -658,7 +658,7 @@
|
|||
name: apache2
|
||||
state: restarted
|
||||
|
||||
- name: reload cron
|
||||
- name: Reload cron
|
||||
ansible.builtin.shell:
|
||||
cmd: service cron reload
|
||||
# Use the shell call because the task sometimes has problems finding the service state
|
||||
|
|
|
@ -53,7 +53,7 @@
|
|||
src: "templates/mosquitto.conf.j2"
|
||||
dest: "{{ mosquitto_data }}/config/mosquitto.conf"
|
||||
mode: 0644
|
||||
notify: restart mosquitto
|
||||
notify: Restart mosquitto container
|
||||
tags:
|
||||
- mosquitto
|
||||
|
||||
|
@ -226,7 +226,7 @@
|
|||
- grafana-screenshot
|
||||
|
||||
handlers:
|
||||
- name: restart mosquitto
|
||||
- name: Restart mosquitto container
|
||||
docker_container:
|
||||
name: mosquitto
|
||||
state: started
|
||||
|
|
|
@ -1,6 +1,6 @@
|
|||
# Handlers for role apache
|
||||
---
|
||||
- name: restart apache2
|
||||
- name: Restart apache2
|
||||
service:
|
||||
name: apache2
|
||||
state: restarted
|
||||
|
|
|
@ -23,7 +23,7 @@
|
|||
mode: "0644"
|
||||
owner: root
|
||||
group: root
|
||||
notify: restart apache2
|
||||
notify: Restart apache2
|
||||
|
||||
- name: Add symlink to enable configuration
|
||||
ansible.builtin.file:
|
||||
|
@ -32,4 +32,4 @@
|
|||
state: link
|
||||
owner: root
|
||||
group: root
|
||||
notify: restart apache2
|
||||
notify: Restart apache2
|
||||
|
|
|
@ -1,6 +1,6 @@
|
|||
# Handlers for role apache_letsencrypt
|
||||
---
|
||||
- name: restart apache2
|
||||
- name: Restart apache2
|
||||
service:
|
||||
name: apache2
|
||||
state: restarted
|
||||
|
|
|
@ -7,7 +7,7 @@
|
|||
mode: "0644"
|
||||
owner: root
|
||||
group: root
|
||||
notify: restart apache2
|
||||
notify: Restart apache2
|
||||
|
||||
- name: Add symlink to enable configuration
|
||||
ansible.builtin.file:
|
||||
|
@ -17,4 +17,4 @@
|
|||
mode: "0644"
|
||||
owner: root
|
||||
group: root
|
||||
notify: restart apache2
|
||||
notify: Restart apache2
|
||||
|
|
|
@ -1,6 +1,6 @@
|
|||
# handlers file for cron-dd24-dyndns
|
||||
---
|
||||
- name: reload cron
|
||||
- name: Reload cron
|
||||
ansible.builtin.shell:
|
||||
cmd: service cron reload
|
||||
warn: no
|
||||
|
|
|
@ -13,6 +13,6 @@
|
|||
owner: root
|
||||
group: root
|
||||
mode: "0644"
|
||||
notify: reload cron
|
||||
notify: Reload cron
|
||||
# There is ansible.builtin.cron, but this makes configuration much
|
||||
# more complicated, so we stick to the template.
|
||||
|
|
|
@ -1,6 +1,6 @@
|
|||
# handlers file for desec_dyndns_cron
|
||||
---
|
||||
- name: reload cron
|
||||
- name: Reload cron
|
||||
ansible.builtin.shell:
|
||||
cmd: service cron reload
|
||||
warn: no
|
||||
|
|
|
@ -13,6 +13,6 @@
|
|||
owner: root
|
||||
group: root
|
||||
mode: "0644"
|
||||
notify: reload cron
|
||||
notify: Reload cron
|
||||
# There is ansible.builtin.cron, but this makes configuration much
|
||||
# more complicated, so we stick to the template.
|
||||
|
|
|
@ -1,3 +1,3 @@
|
|||
---
|
||||
- name: reload nfs
|
||||
- name: Reload nfs
|
||||
command: 'exportfs -ra'
|
||||
|
|
|
@ -14,7 +14,7 @@
|
|||
state: present
|
||||
fs_type: ext4
|
||||
|
||||
- name: ensure nfs mountpoints exist
|
||||
- name: Ensure nfs mountpoints exist
|
||||
ansible.builtin.file:
|
||||
path: "{{ item.directory }}"
|
||||
state: directory
|
||||
|
@ -30,11 +30,11 @@
|
|||
fstype: ext4
|
||||
state: present
|
||||
|
||||
- name: template /etc/exports
|
||||
- name: Put /etc/exports in place from template
|
||||
ansible.builtin.template:
|
||||
src: templates/exports.j2
|
||||
dest: "/etc/exports"
|
||||
notify: reload nfs
|
||||
notify: Reload nfs
|
||||
|
||||
- name: Ensure nfs is running.
|
||||
ansible.builtin.service: "name=nfs-kernel-server state=started enabled=yes"
|
||||
|
|
|
@ -1,6 +1,6 @@
|
|||
# Handlers für nginx-https-proxy
|
||||
---
|
||||
- name: restart nginx
|
||||
- name: Restart nginx
|
||||
service:
|
||||
name: nginx
|
||||
state: restarted
|
||||
|
|
|
@ -56,7 +56,7 @@
|
|||
owner: root
|
||||
group: root
|
||||
mode: '0644'
|
||||
notify: restart nginx
|
||||
notify: Restart nginx
|
||||
|
||||
- name: Create directory for dehydrated forwardings
|
||||
ansible.builtin.file:
|
||||
|
@ -74,7 +74,7 @@
|
|||
group: root
|
||||
mode: '0644'
|
||||
loop: "{{ ingress }}"
|
||||
notify: restart nginx
|
||||
notify: Restart nginx
|
||||
|
||||
- name: Setup nginx configuration
|
||||
# Note the order here: The nginx configuration _needs_ he dehydrated-hosts
|
||||
|
@ -86,4 +86,4 @@
|
|||
owner: root
|
||||
group: root
|
||||
mode: '0644'
|
||||
notify: restart nginx
|
||||
notify: Restart nginx
|
||||
|
|
|
@ -1,5 +1,5 @@
|
|||
---
|
||||
- name: restart apache2
|
||||
- name: Restart apache2
|
||||
service:
|
||||
name: apache2
|
||||
state: restarted
|
||||
|
|
|
@ -3,10 +3,10 @@
|
|||
template:
|
||||
src: templates/apache-docker-forward-site.j2
|
||||
dest: /etc/apache2/sites-available/{{ site_name }}.conf
|
||||
notify: restart apache2
|
||||
notify: Restart apache2
|
||||
|
||||
- name: Activate Apache2 site
|
||||
command: a2ensite {{ site_name }}
|
||||
args:
|
||||
creates: /etc/apache2/sites-enabled/{{ site_name }}.conf
|
||||
notify: restart apache2
|
||||
notify: Restart apache2
|
||||
|
|
|
@ -1,5 +1,5 @@
|
|||
---
|
||||
- name: restart apache2
|
||||
- name: Restart apache2
|
||||
service:
|
||||
name: apache2
|
||||
state: restarted
|
||||
|
|
|
@ -4,10 +4,10 @@
|
|||
src: templates/apache-docker-proxy-site.j2
|
||||
dest: /etc/apache2/sites-available/{{ site_name }}.conf
|
||||
mode: "0644"
|
||||
notify: restart apache2
|
||||
notify: Restart apache2
|
||||
|
||||
- name: Activate Apache2 site
|
||||
command: a2ensite {{ site_name }}
|
||||
args:
|
||||
creates: /etc/apache2/sites-enabled/{{ site_name }}.conf
|
||||
notify: restart apache2
|
||||
notify: Restart apache2
|
||||
|
|
|
@ -1,21 +1,21 @@
|
|||
---
|
||||
- name: configure local ssh to access n39 hosts
|
||||
- name: Configure local ssh to access n39 hosts
|
||||
hosts: localhost
|
||||
|
||||
tasks:
|
||||
- name: ensure {{ lookup('env', 'HOME') }}/.ssh/config.d/ dir is present
|
||||
- name: Ensure {{ lookup('env', 'HOME') }}/.ssh/config.d/ dir is present
|
||||
ansible.builtin.file:
|
||||
path: "{{ lookup('env', 'HOME') }}/.ssh/config.d/"
|
||||
state: directory
|
||||
delegate_to: localhost
|
||||
|
||||
- name: template ssh config for access to internal systems
|
||||
- name: Put ssh config for access to n39 internal systems in place
|
||||
ansible.builtin.template:
|
||||
src: templates/ssh_config.j2
|
||||
dest: "{{ lookup('env', 'HOME') }}/.ssh/config.d/n39_config"
|
||||
delegate_to: localhost
|
||||
|
||||
- name: ensure that n39 access config is included
|
||||
- name: Ensure that n39 access config is included
|
||||
ansible.builtin.lineinfile:
|
||||
path: ~/.ssh/config
|
||||
insertbefore: BOF
|
||||
|
|
Loading…
Add table
Reference in a new issue