From d9d57cbeeb94b4e71ea2259a506c6877491e215f Mon Sep 17 00:00:00 2001
From: Chris Hawk <hawk@google.com>
Date: Wed, 24 Jul 2024 16:15:24 -0700
Subject: [PATCH 01/13] Remove jinja vars from conditional statements in
 integration test asserts

---
 .../targets/gcp_container_cluster/tasks/autogen.yml           | 4 ++--
 .../targets/gcp_container_node_pool/tasks/autogen.yml         | 4 ++--
 .../targets/gcp_dns_resource_record_set/tasks/autogen.yml     | 4 ++--
 .../targets/gcp_pubsub_subscription/tasks/autogen.yml         | 4 ++--
 tests/integration/targets/gcp_pubsub_topic/tasks/autogen.yml  | 4 ++--
 tests/integration/targets/gcp_sql_database/tasks/autogen.yml  | 4 ++--
 tests/integration/targets/gcp_sql_user/tasks/autogen.yml      | 4 ++--
 7 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/tests/integration/targets/gcp_container_cluster/tasks/autogen.yml b/tests/integration/targets/gcp_container_cluster/tasks/autogen.yml
index 4a845eac0..f3c1d1273 100644
--- a/tests/integration/targets/gcp_container_cluster/tasks/autogen.yml
+++ b/tests/integration/targets/gcp_container_cluster/tasks/autogen.yml
@@ -55,7 +55,7 @@
 - name: Verify that command succeeded
   ansible.builtin.assert:
     that:
-      - "'my-cluster' in \"{{ results['resources'] | map(attribute='name') | list }}\""
+      - results['resources'] | map(attribute='name') | select("match", ".*my-cluster.*") | list | length == 1
 # ----------------------------------------------------------------------------
 - name: Create a cluster that already exists
   google.cloud.gcp_container_cluster:
@@ -104,7 +104,7 @@
 - name: Verify that command succeeded
   ansible.builtin.assert:
     that:
-      - "'my-cluster' not in \"{{ results['resources'] | map(attribute='name') | list }}\""
+      - results['resources'] | map(attribute='name') | select("match", ".*my-cluster.*") | list | length == 0
 # ----------------------------------------------------------------------------
 - name: Delete a cluster that does not exist
   google.cloud.gcp_container_cluster:
diff --git a/tests/integration/targets/gcp_container_node_pool/tasks/autogen.yml b/tests/integration/targets/gcp_container_node_pool/tasks/autogen.yml
index ae04f3049..84fe978a7 100644
--- a/tests/integration/targets/gcp_container_node_pool/tasks/autogen.yml
+++ b/tests/integration/targets/gcp_container_node_pool/tasks/autogen.yml
@@ -62,7 +62,7 @@
 - name: Verify that command succeeded
   ansible.builtin.assert:
     that:
-      - "'my-pool' in \"{{ results['resources'] | map(attribute='name') | list }}\""
+      - results['resources'] | map(attribute='name') | select("match", ".*my-pool.*") | list | length == 1
 # ----------------------------------------------------------------------------
 - name: Create a node pool that already exists
   google.cloud.gcp_container_node_pool:
@@ -108,7 +108,7 @@
 - name: Verify that command succeeded
   ansible.builtin.assert:
     that:
-      - "'my-pool' not in \"{{ results['resources'] | map(attribute='name') | list }}\""
+      - results['resources'] | map(attribute='name') | select("match", ".*my-pool.*") | list | length == 0
 # ----------------------------------------------------------------------------
 - name: Delete a node pool that does not exist
   google.cloud.gcp_container_node_pool:
diff --git a/tests/integration/targets/gcp_dns_resource_record_set/tasks/autogen.yml b/tests/integration/targets/gcp_dns_resource_record_set/tasks/autogen.yml
index 67fe3e328..c8ca5354c 100644
--- a/tests/integration/targets/gcp_dns_resource_record_set/tasks/autogen.yml
+++ b/tests/integration/targets/gcp_dns_resource_record_set/tasks/autogen.yml
@@ -67,7 +67,7 @@
 - name: Verify that command succeeded
   ansible.builtin.assert:
     that:
-      - "'www.testzone-4.com.'in \"{{ results['resources'] | map(attribute='name') | list }}\""
+      - results['resources'] | map(attribute='name') | select("match", ".*www\\.testzone-4\\.com.*") | list | length == 1
 # ----------------------------------------------------------------------------
 - name: Create a resource record set that already exists
   google.cloud.gcp_dns_resource_record_set:
@@ -118,7 +118,7 @@
 - name: Verify that command succeeded
   ansible.builtin.assert:
     that:
-      - "'www.testzone-4.com.'not in \"{{ results['resources'] | map(attribute='name') | list }}\""
+      - results['resources'] | map(attribute='name') | select("match", ".*www\\.testzone-4\\.com.*") | list | length == 0
 # ----------------------------------------------------------------------------
 - name: Delete a resource record set that does not exist
   google.cloud.gcp_dns_resource_record_set:
diff --git a/tests/integration/targets/gcp_pubsub_subscription/tasks/autogen.yml b/tests/integration/targets/gcp_pubsub_subscription/tasks/autogen.yml
index ca78eb436..6aeaa58dd 100644
--- a/tests/integration/targets/gcp_pubsub_subscription/tasks/autogen.yml
+++ b/tests/integration/targets/gcp_pubsub_subscription/tasks/autogen.yml
@@ -56,7 +56,7 @@
 - name: Verify that command succeeded
   ansible.builtin.assert:
     that:
-      - "\"{{ resource_name }}\" in \"{{ results['resources'] | map(attribute='name') | list }}\""
+      - results['resources'] | map(attribute='name') | select("match", ".*{{ resource_name }}.*") | list | length == 1
 # ----------------------------------------------------------------------------
 - name: Create a subscription that already exists
   google.cloud.gcp_pubsub_subscription:
@@ -98,7 +98,7 @@
 - name: Verify that command succeeded
   ansible.builtin.assert:
     that:
-      - "\"{{ resource_name }}\" not in \"{{ results['resources'] | map(attribute='name') | list }}\""
+      - results['resources'] | map(attribute='name') | select("match", ".*{{ resource_name }}.*") | list | length == 0
 # ----------------------------------------------------------------------------
 - name: Delete a subscription that does not exist
   google.cloud.gcp_pubsub_subscription:
diff --git a/tests/integration/targets/gcp_pubsub_topic/tasks/autogen.yml b/tests/integration/targets/gcp_pubsub_topic/tasks/autogen.yml
index 0c0dbdc96..d9f3cd3da 100644
--- a/tests/integration/targets/gcp_pubsub_topic/tasks/autogen.yml
+++ b/tests/integration/targets/gcp_pubsub_topic/tasks/autogen.yml
@@ -44,7 +44,7 @@
 - name: Verify that command succeeded
   ansible.builtin.assert:
     that:
-      - "'test-topic1' in \"{{ results['resources'] | map(attribute='name') | list }}\""
+      - results['resources'] | map(attribute='name') | select("match", ".*test-topic1.*") | list | length == 1
 # ----------------------------------------------------------------------------
 - name: Create a topic that already exists
   google.cloud.gcp_pubsub_topic:
@@ -82,7 +82,7 @@
 - name: Verify that command succeeded
   ansible.builtin.assert:
     that:
-      - "'test-topic1' not in \"{{ results['resources'] | map(attribute='name') | list }}\""
+      - results['resources'] | map(attribute='name') | select("match", ".*test-topic1.*") | list | length == 0
 # ----------------------------------------------------------------------------
 - name: Delete a topic that does not exist
   google.cloud.gcp_pubsub_topic:
diff --git a/tests/integration/targets/gcp_sql_database/tasks/autogen.yml b/tests/integration/targets/gcp_sql_database/tasks/autogen.yml
index dd87faa83..a84096fbd 100644
--- a/tests/integration/targets/gcp_sql_database/tasks/autogen.yml
+++ b/tests/integration/targets/gcp_sql_database/tasks/autogen.yml
@@ -64,7 +64,7 @@
 - name: Verify that command succeeded
   ansible.builtin.assert:
     that:
-      - "\"{{ resource_name }}\" in \"{{ results['resources'] | map(attribute='name') | list }}\""
+      - results['resources'] | map(attribute='name') | select("match", ".*{{ resource_name }}.*") | list | length == 1
 # ----------------------------------------------------------------------------
 - name: Create a database that already exists
   google.cloud.gcp_sql_database:
@@ -107,7 +107,7 @@
 - name: Verify that command succeeded
   ansible.builtin.assert:
     that:
-      - "\"{{ resource_name }}\" not in \"{{ results['resources'] | map(attribute='name') | list }}\""
+      - results['resources'] | map(attribute='name') | select("match", ".*{{ resource_name }}.*") | list | length == 0
 # ----------------------------------------------------------------------------
 - name: Delete a database that does not exist
   google.cloud.gcp_sql_database:
diff --git a/tests/integration/targets/gcp_sql_user/tasks/autogen.yml b/tests/integration/targets/gcp_sql_user/tasks/autogen.yml
index 20ddbea25..0a820cf5d 100644
--- a/tests/integration/targets/gcp_sql_user/tasks/autogen.yml
+++ b/tests/integration/targets/gcp_sql_user/tasks/autogen.yml
@@ -66,7 +66,7 @@
 - name: Verify that command succeeded
   ansible.builtin.assert:
     that:
-      - "'test-user' in \"{{ results['resources'] | map(attribute='name') | list }}\""
+      - results['resources'] | map(attribute='name') | select("match", ".*test-user.*") | list | length == 1
 # ----------------------------------------------------------------------------
 - name: Create a user that already exists
   google.cloud.gcp_sql_user:
@@ -111,7 +111,7 @@
 - name: Verify that command succeeded
   ansible.builtin.assert:
     that:
-      - "'test-user' not in \"{{ results['resources'] | map(attribute='name') | list }}\""
+      - results['resources'] | map(attribute='name') | select("match", ".*test-user.*") | list | length == 0
 # ----------------------------------------------------------------------------
 - name: Delete a user that does not exist
   google.cloud.gcp_sql_user:

From e79c751f3affb50e1b2443e29c3848c276f37bc5 Mon Sep 17 00:00:00 2001
From: Chris Hawk <hawk@google.com>
Date: Fri, 26 Jul 2024 13:21:00 -0700
Subject: [PATCH 02/13] Fix assertNotEquals: deprecated in 3.2, removed in 3.12

---
 tests/unit/plugins/test_gcp_utils.py | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/tests/unit/plugins/test_gcp_utils.py b/tests/unit/plugins/test_gcp_utils.py
index 4956fe513..804bbbb6f 100644
--- a/tests/unit/plugins/test_gcp_utils.py
+++ b/tests/unit/plugins/test_gcp_utils.py
@@ -85,7 +85,7 @@ def test_simple_different(self):
         difference = {"test": "original"}
         request1 = GcpRequest(value1)
         request2 = GcpRequest(value2)
-        self.assertNotEquals(request1, request2)
+        self.assertNotEqual(request1, request2)
         self.assertEqual(request1.difference(request2), difference)
 
     def test_nested_dictionaries_no_difference(self):
@@ -99,7 +99,7 @@ def test_nested_dictionaries_with_difference(self):
         difference = {"foo": {"quiet": {"tree": "test"}, "bar": "baz"}}
         request1 = GcpRequest(value1)
         request2 = GcpRequest(value2)
-        self.assertNotEquals(request1, request2)
+        self.assertNotEqual(request1, request2)
         self.assertEqual(request1.difference(request2), difference)
 
     def test_arrays_strings_no_difference(self):
@@ -123,7 +123,7 @@ def test_arrays_strings_with_difference(self):
         }
         request1 = GcpRequest(value1)
         request2 = GcpRequest(value2)
-        self.assertNotEquals(request1, request2)
+        self.assertNotEqual(request1, request2)
         self.assertEqual(request1.difference(request2), difference)
 
     def test_arrays_dicts_with_no_difference(self):
@@ -141,7 +141,7 @@ def test_arrays_dicts_with_difference(self):
         difference = {"foo": [{"test": "value", "foo": "bar"}]}
         request1 = GcpRequest(value1)
         request2 = GcpRequest(value2)
-        self.assertNotEquals(request1, request2)
+        self.assertNotEqual(request1, request2)
         self.assertEqual(request1.difference(request2), difference)
 
     def test_dicts_boolean_with_difference(self):
@@ -165,5 +165,5 @@ def test_dicts_boolean_with_difference(self):
         }
         request1 = GcpRequest(value1)
         request2 = GcpRequest(value2)
-        self.assertNotEquals(request1, request2)
+        self.assertNotEqual(request1, request2)
         self.assertEqual(request1.difference(request2), difference)

From c76063342ebaf9833bd9b128ff2d322d164387c5 Mon Sep 17 00:00:00 2001
From: Chris Hawk <hawk@google.com>
Date: Fri, 26 Jul 2024 13:21:21 -0700
Subject: [PATCH 03/13] Include container cluster in the cleanup script

---
 scripts/cleanup-project.sh | 1 +
 1 file changed, 1 insertion(+)

diff --git a/scripts/cleanup-project.sh b/scripts/cleanup-project.sh
index b05a2f15f..76f9714b3 100755
--- a/scripts/cleanup-project.sh
+++ b/scripts/cleanup-project.sh
@@ -16,6 +16,7 @@ ZONE="us-central1-a"
 main() {
     # note: the ordering here is deliberate, to start with
     # leaf resources and work upwards to parent resources.
+    cleanup_resource "container" "clusters" "" "--zone=$ZONE"
     cleanup_resource_per_region "compute" "vpn-tunnels"
     cleanup_resource "compute" "instances" "" "--zone=$ZONE"
     cleanup_resource_per_region "compute" "addresses"

From 5d28cf0bdeb7e6ddce64ca0fcad2edd6f480c135 Mon Sep 17 00:00:00 2001
From: Chris Hawk <hawk@google.com>
Date: Fri, 26 Jul 2024 15:12:45 -0700
Subject: [PATCH 04/13] Update ansible-lint; exclude the submodule and stop
 excluding others

---
 .ansible-lint | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/.ansible-lint b/.ansible-lint
index c77c6a873..00c8c9605 100644
--- a/.ansible-lint
+++ b/.ansible-lint
@@ -1,10 +1,10 @@
 ---
+profile: production
 parseable: true
 skip_list:
   - ANSIBLE0010
 use_default_rules: true
 verbosity: 1
 exclude_paths:
-  - ./roles/gcp_http_lb/
-  - ./tests/
-  - ./plugins
+  # Ignore submodule https://github.com/GoogleCloudPlatform/google-cloud-ops-agents-ansible
+  - roles/google-cloud-ops-agents-ansible/

From 3010182ade956bf9ddfa248dd9dc355ca401c748 Mon Sep 17 00:00:00 2001
From: Chris Hawk <hawk@google.com>
Date: Fri, 26 Jul 2024 15:17:30 -0700
Subject: [PATCH 05/13] Change RHEL yum references to dnf

---
 roles/gcloud/tasks/archive/command_completion.yml |  2 +-
 roles/gcloud/tasks/package/redhat.yml             | 12 ++++++------
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/roles/gcloud/tasks/archive/command_completion.yml b/roles/gcloud/tasks/archive/command_completion.yml
index 277210125..a130e3397 100644
--- a/roles/gcloud/tasks/archive/command_completion.yml
+++ b/roles/gcloud/tasks/archive/command_completion.yml
@@ -10,7 +10,7 @@
   when: ansible_os_family == "Debian"
 
 - name: Gcloud | Archive | RedHat | Ensure bash completion is installed
-  ansible.builtin.yum:
+  ansible.builtin.dnf:
     name:
       - bash-completion
   register: task_result
diff --git a/roles/gcloud/tasks/package/redhat.yml b/roles/gcloud/tasks/package/redhat.yml
index 8917561a2..156d54563 100644
--- a/roles/gcloud/tasks/package/redhat.yml
+++ b/roles/gcloud/tasks/package/redhat.yml
@@ -1,18 +1,18 @@
 ---
-- name: Gcloud | RHEL | Add an Apt signing key, uses whichever key is at the URL
+# https://cloud.google.com/sdk/docs/install#rpm
+- name: Gcloud | RHEL | Add a dnf signing key, uses whichever key is at the URL
   ansible.builtin.yum_repository:
     name: google-cloud-sdk
     description: Google Cloud SDK
-    baseurl: https://packages.cloud.google.com/yum/repos/cloud-sdk-el7-x86_64
+    baseurl: https://packages.cloud.google.com/yum/repos/cloud-sdk-el9-x86_64
     enabled: true
     gpgcheck: true
-    repo_gpgcheck: true
+    repo_gpgcheck: false
     gpgkey:
-      - https://packages.cloud.google.com/yum/doc/yum-key.gpg
       - https://packages.cloud.google.com/yum/doc/rpm-package-key.gpg
 
 - name: Gcloud | RHEL | Install the google-cloud-sdk package
-  ansible.builtin.yum:
+  ansible.builtin.dnf:
     name: google-cloud-sdk
     update_cache: "yes"
   register: task_result
@@ -21,7 +21,7 @@
   delay: 2
 
 - name: Gcloud | Debian | Install the google-cloud-sdk additional components
-  ansible.builtin.yum:
+  ansible.builtin.dnf:
     name: google-cloud-sdk-{{ item }}
     update_cache: "yes"
   register: task_result

From c630ef996a5949976792419e1534852d98f07021 Mon Sep 17 00:00:00 2001
From: Chris Hawk <hawk@google.com>
Date: Fri, 26 Jul 2024 15:24:26 -0700
Subject: [PATCH 06/13] Fix the path to google-cloud-ops-agents-ansible

---
 .ansible-lint | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.ansible-lint b/.ansible-lint
index 00c8c9605..1f5f25a97 100644
--- a/.ansible-lint
+++ b/.ansible-lint
@@ -7,4 +7,4 @@ use_default_rules: true
 verbosity: 1
 exclude_paths:
   # Ignore submodule https://github.com/GoogleCloudPlatform/google-cloud-ops-agents-ansible
-  - roles/google-cloud-ops-agents-ansible/
+  - ./roles/google-cloud-ops-agents-ansible/

From c9b7cf5a2157441dd73bd33ef083df40ba7a41b4 Mon Sep 17 00:00:00 2001
From: Chris Hawk <hawk@google.com>
Date: Fri, 26 Jul 2024 15:42:19 -0700
Subject: [PATCH 07/13] Try passing a full path to the ansible-lint config file

---
 .ansible-lint                      | 2 +-
 .github/workflows/ansible-test.yml | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/.ansible-lint b/.ansible-lint
index 1f5f25a97..00c8c9605 100644
--- a/.ansible-lint
+++ b/.ansible-lint
@@ -7,4 +7,4 @@ use_default_rules: true
 verbosity: 1
 exclude_paths:
   # Ignore submodule https://github.com/GoogleCloudPlatform/google-cloud-ops-agents-ansible
-  - ./roles/google-cloud-ops-agents-ansible/
+  - roles/google-cloud-ops-agents-ansible/
diff --git a/.github/workflows/ansible-test.yml b/.github/workflows/ansible-test.yml
index 8d576043c..cc2df802d 100644
--- a/.github/workflows/ansible-test.yml
+++ b/.github/workflows/ansible-test.yml
@@ -37,7 +37,7 @@ jobs:
       - name: Install ansible-lint
         run: pip install ansible-lint==24.7.0
       - name: Run ansible-lint
-        run: ansible-lint
+        run: ansible-lint -c ${GITHUB_WORKSPACE}/ansible_collections/google/cloud/.ansible-lint
   unit:
     runs-on: ubuntu-latest
     defaults:

From 6a078d2ddbffcf54d8108d402d2cee21fc0797ac Mon Sep 17 00:00:00 2001
From: Chris Hawk <hawk@google.com>
Date: Fri, 26 Jul 2024 15:49:09 -0700
Subject: [PATCH 08/13] Make the path to google-cloud-ops-agents-ansible
 prefixed with ./

---
 .ansible-lint | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.ansible-lint b/.ansible-lint
index 00c8c9605..1f5f25a97 100644
--- a/.ansible-lint
+++ b/.ansible-lint
@@ -7,4 +7,4 @@ use_default_rules: true
 verbosity: 1
 exclude_paths:
   # Ignore submodule https://github.com/GoogleCloudPlatform/google-cloud-ops-agents-ansible
-  - roles/google-cloud-ops-agents-ansible/
+  - ./roles/google-cloud-ops-agents-ansible/

From 811901f82c1325118a952abfe112eac2c7538dd3 Mon Sep 17 00:00:00 2001
From: Chris Hawk <hawk@google.com>
Date: Fri, 26 Jul 2024 16:02:15 -0700
Subject: [PATCH 09/13] Add some debugging statements

---
 .github/workflows/ansible-test.yml | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/ansible-test.yml b/.github/workflows/ansible-test.yml
index cc2df802d..7206aa210 100644
--- a/.github/workflows/ansible-test.yml
+++ b/.github/workflows/ansible-test.yml
@@ -36,8 +36,12 @@ jobs:
         run: ansible-test sanity -v --color --skip validate-modules
       - name: Install ansible-lint
         run: pip install ansible-lint==24.7.0
+      - name: Debug the ansible-lint path
+        run: |
+          pwd
+          ls -al
       - name: Run ansible-lint
-        run: ansible-lint -c ${GITHUB_WORKSPACE}/ansible_collections/google/cloud/.ansible-lint
+        run: ansible-lint --show-relpath --exclude ./roles/google-cloud-ops-agents-ansible/
   unit:
     runs-on: ubuntu-latest
     defaults:

From 12d500f044f2f6717979c1f88357234910dda75b Mon Sep 17 00:00:00 2001
From: Chris Hawk <hawk@google.com>
Date: Fri, 26 Jul 2024 16:08:18 -0700
Subject: [PATCH 10/13] Use the submodule path, not the name of the repo

---
 .ansible-lint                      | 2 +-
 .github/workflows/ansible-test.yml | 6 +-----
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/.ansible-lint b/.ansible-lint
index 1f5f25a97..19247a58d 100644
--- a/.ansible-lint
+++ b/.ansible-lint
@@ -7,4 +7,4 @@ use_default_rules: true
 verbosity: 1
 exclude_paths:
   # Ignore submodule https://github.com/GoogleCloudPlatform/google-cloud-ops-agents-ansible
-  - ./roles/google-cloud-ops-agents-ansible/
+  - ./roles/google_cloud_ops_agents/
diff --git a/.github/workflows/ansible-test.yml b/.github/workflows/ansible-test.yml
index 7206aa210..8d576043c 100644
--- a/.github/workflows/ansible-test.yml
+++ b/.github/workflows/ansible-test.yml
@@ -36,12 +36,8 @@ jobs:
         run: ansible-test sanity -v --color --skip validate-modules
       - name: Install ansible-lint
         run: pip install ansible-lint==24.7.0
-      - name: Debug the ansible-lint path
-        run: |
-          pwd
-          ls -al
       - name: Run ansible-lint
-        run: ansible-lint --show-relpath --exclude ./roles/google-cloud-ops-agents-ansible/
+        run: ansible-lint
   unit:
     runs-on: ubuntu-latest
     defaults:

From dfe85d24a05f1022984a855828e66edb1ad785ce Mon Sep 17 00:00:00 2001
From: Chris Hawk <hawk@google.com>
Date: Fri, 26 Jul 2024 16:18:55 -0700
Subject: [PATCH 11/13] ansible-lint truly hates not running in the workspace
 root directory

---
 .github/workflows/ansible-test.yml | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/ansible-test.yml b/.github/workflows/ansible-test.yml
index 8d576043c..2bef6469e 100644
--- a/.github/workflows/ansible-test.yml
+++ b/.github/workflows/ansible-test.yml
@@ -36,8 +36,12 @@ jobs:
         run: ansible-test sanity -v --color --skip validate-modules
       - name: Install ansible-lint
         run: pip install ansible-lint==24.7.0
+      - name: Debug the ansible-lint path
+        run: |
+          pwd
+          ls -al
       - name: Run ansible-lint
-        run: ansible-lint
+        run: ansible-lint --exclude ./roles/google_cloud_ops_agents/
   unit:
     runs-on: ubuntu-latest
     defaults:

From 7ae46673ed6a7d283143355ef26a5259b949e698 Mon Sep 17 00:00:00 2001
From: Chris Hawk <hawk@google.com>
Date: Fri, 26 Jul 2024 16:33:09 -0700
Subject: [PATCH 12/13] Use the path that ansible-lint is reporting for
 exclusion

---
 .ansible-lint                      | 2 +-
 .github/workflows/ansible-test.yml | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/.ansible-lint b/.ansible-lint
index 19247a58d..0c5e66897 100644
--- a/.ansible-lint
+++ b/.ansible-lint
@@ -7,4 +7,4 @@ use_default_rules: true
 verbosity: 1
 exclude_paths:
   # Ignore submodule https://github.com/GoogleCloudPlatform/google-cloud-ops-agents-ansible
-  - ./roles/google_cloud_ops_agents/
+  - roles/google_cloud_ops_agents/
diff --git a/.github/workflows/ansible-test.yml b/.github/workflows/ansible-test.yml
index 2bef6469e..6a2e1fcf4 100644
--- a/.github/workflows/ansible-test.yml
+++ b/.github/workflows/ansible-test.yml
@@ -41,7 +41,7 @@ jobs:
           pwd
           ls -al
       - name: Run ansible-lint
-        run: ansible-lint --exclude ./roles/google_cloud_ops_agents/
+        run: ansible-lint --exclude roles/google_cloud_ops_agents/
   unit:
     runs-on: ubuntu-latest
     defaults:

From 7a95362e0305884545466bfd5d2501e5cd234bdf Mon Sep 17 00:00:00 2001
From: Chris Hawk <hawk@google.com>
Date: Fri, 26 Jul 2024 16:44:29 -0700
Subject: [PATCH 13/13] Remove debugging lines

---
 .github/workflows/ansible-test.yml | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/.github/workflows/ansible-test.yml b/.github/workflows/ansible-test.yml
index 6a2e1fcf4..e9fea6a37 100644
--- a/.github/workflows/ansible-test.yml
+++ b/.github/workflows/ansible-test.yml
@@ -36,10 +36,6 @@ jobs:
         run: ansible-test sanity -v --color --skip validate-modules
       - name: Install ansible-lint
         run: pip install ansible-lint==24.7.0
-      - name: Debug the ansible-lint path
-        run: |
-          pwd
-          ls -al
       - name: Run ansible-lint
         run: ansible-lint --exclude roles/google_cloud_ops_agents/
   unit: