From: Ken Dreyer <kdreyer@redhat.com>
Date: Fri, 21 Apr 2023 14:57:15 +0000 (-0400)
Subject: public_facing: skip no-tabs linter rule on single task
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F733%2Fhead;p=ceph-cm-ansible.git

public_facing: skip no-tabs linter rule on single task

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.
---

diff --git a/.ansible-lint b/.ansible-lint
index 6c902028..578703e5 100644
--- a/.ansible-lint
+++ b/.ansible-lint
@@ -18,7 +18,6 @@ skip_list:
   - no-handler
   - no-jinja-when
   - no-relative-paths
-  - no-tabs
   - package-latest
   - risky-file-permissions
   - risky-shell-pipe
diff --git a/roles/public_facing/tasks/letsencrypt_nginx.yml b/roles/public_facing/tasks/letsencrypt_nginx.yml
index f1370a43..3dcd31e7 100644
--- a/roles/public_facing/tasks/letsencrypt_nginx.yml
+++ b/roles/public_facing/tasks/letsencrypt_nginx.yml
@@ -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"