Skip to content

Commit

Permalink
public_facing: skip no-tabs linter rule on single task
Browse files Browse the repository at this point in the history
Instead of skipping ansible-lint's no-tabs rule globally, apply it on
this single task that uses a tab (\t) character.

Longer-term, we could replace this tab with a space because /etc/hosts
can use either whitespace character. I'm taking a cautious approach
today for simplicity.
  • Loading branch information
ktdreyer committed Apr 24, 2023
1 parent ded2bb2 commit c20b642
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion .ansible-lint
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ skip_list:
- no-handler
- no-jinja-when
- no-relative-paths
- no-tabs
- package-latest
- risky-file-permissions
- risky-shell-pipe
Expand Down
2 changes: 1 addition & 1 deletion roles/public_facing/tasks/letsencrypt_nginx.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@
state: present

# 'letsencrypt renew' fails because it can't reach the letsencrypt authority server using IPv6
- name: Create cron entry to force IPv4 connectivity to letsencrypt authority server
- name: Create cron entry to force IPv4 connectivity to letsencrypt authority server # noqa no-tabs
cron:
name: "Forces letsencrypt to use IPv4 when accessing acme-v01.api.letsencrypt.org"
hour: "0"
Expand Down

0 comments on commit c20b642

Please sign in to comment.