Merge pull request #414 from andygrunwald/upgrade-ansible-version-range

minor: bump support for ansible from v2.16.* to v2.17.*
prometheus-community · Sep 30, 2024 · 79fbda7 · 79fbda7
2 parents 15369b7 + e0d0aa4
commit 79fbda7
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 2 deletions.
diff --git a/.config/molecule/config.yml b/.config/molecule/config.yml
@@ -88,3 +88,6 @@ provisioner:
           - "2.9"
           - "2.10"
           - "2.11"
+      almalinux-8:
+        exclude_ansible_vers:
+          - "2.17"
diff --git a/meta/runtime.yml b/meta/runtime.yml
@@ -1,2 +1,2 @@
 ---
-requires_ansible: ">=2.9.0,<=2.16.99"
+requires_ansible: ">=2.9.0,<=2.17.99"
diff --git a/tests/integration/molecule.sh b/tests/integration/molecule.sh
@@ -51,5 +51,5 @@ unset _ANSIBLE_COVERAGE_CONFIG
 unset ANSIBLE_PYTHON_INTERPRETER
 
 # Run molecule test
-cd "$role_root"
+cd "$role_root" || { echo "Fail to change directory into $role_root"; exit 1; }
 molecule -c "$collection_root/.config/molecule/config.yml" test -s "$scenario"
diff --git a/tests/sanity/ignore-2.17.txt b/tests/sanity/ignore-2.17.txt
@@ -0,0 +1 @@
+.github/scripts/collection_version_parser.py shebang  # https://github.com/ansible/ansible/issues/78346
-Original file line number
+Diff line change
@@ Expand Up / @@ -88,3 +88,6 @@ provisioner: @@
               - "2.9"
               - "2.10"
               - "2.11"
+          almalinux-8:
+            exclude_ansible_vers:
+              - "2.17"
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		.github/scripts/collection_version_parser.py shebang # https://github.com/ansible/ansible/issues/78346