Jose/ansible_role_proxmox_provision
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

style 💎: Fix table formatting and add code block for list_banned task #46

Merged
Jose merged 8 commits from dev into main 2026-03-01 10:55:12 +01:00
Conversation 1 Commits 8 Files Changed 4 +12 -4
2 changed files with 12 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit bc91383094 - Show all commits

6
defaults/main.yml
View File

@@ -49,10 +49,10 @@ vm_dirty_background_ratio: 5
vm_swappiness: "{{ proxmox_swapiness }}"
# Fail2ban settings
f2b_bantime: 1800 # 30 minutes
f2b_findtime: 600
f2b_bantime: 600 # 10 minutes
f2b_findtime: 1200 # 20 minutes
f2b_maxretry: 5
f2b_recidive_bantime: 86400 # 24 hours
f2b_recidive_bantime: 3600 # 1 hours
f2b_recidive_findtime: 86400 # 24 hours
f2b_recidive_maxretry: 3
f2b_ipset_name: f2b-blacklist

10
handlers/main.yml
View File

@@ -32,10 +32,18 @@
ansible.builtin.systemd:
daemon_reload: true
- name: Restart fail2ban
- name: Reload fail2ban
ansible.builtin.systemd:
name: fail2ban
state: reloaded
enabled: true
- name: Restart fail2ban
ansible.builtin.systemd:
name: fail2ban
state: restarted
enabled: true
gitea-actions commented 2026-03-01 10:54:12 +01:00
Review
Copy Link

[Lines 35-47] [Score: 2] Duplicating steps to restart and enable fail2ban, which may lead to confusion or inconsistencies in the system behavior. Consider consolidating these into a single task or renaming both tasks to be more descriptive (e.g., "Restart and Enable fail2ban").

[Lines 35-47] [Score: 2] Duplicating steps to restart and enable fail2ban, which may lead to confusion or inconsistencies in the system behavior. Consider consolidating these into a single task or renaming both tasks to be more descriptive (e.g., "Restart and Enable fail2ban").
- name: Reload pve firewall
ansible.builtin.command: pve-firewall reload