Repository navigation
CI: run sanity tests on ansible-core 2.15 to 2.22 and devel - #95
Merged
Merged
Conversation
Sanity was tested only on ansible-core 2.15 and 2.16, which are end of life, while the collection declares requires_ansible >= 2.15.0. - Test every stable branch from 2.15 to 2.22, each installed on a Python version that ansible-core release supports (2.18 and later no longer install on Python 3.10). - Test devel as an early warning; a devel failure does not fail the run. - Disable fail-fast, so a failure on one version does not cancel the others. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
ansible-core 2.21 validate-modules rejects return value keys that cannot be used with dot notation in Jinja (bad-return-value-key). The keys were also not the ones the modules return: - cp_gaia_simulate_packet: the result is returned in simulate_packet, not "packet rulebase execution result". - cp_gaia_vsnext_state_facts: the API response keys are converted to underscores, so the facts are session_virtual_system_id and member_id. - cp_gaia_virtual_systems_facts: document the show-virtual-systems response (objects, from, to, total) instead of the VSNext state facts that were copied from cp_gaia_vsnext_state_facts. Regenerate the docs of the three modules. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
chkp-majds
approved these changes
Nov 3, 2026
chkp-ameera
added a commit
to chkp-ameera/CheckPointAnsibleGAIACollection
that referenced
this pull request
Nov 3, 2026
Record the RETURN documentation fixes of cp_gaia_simulate_packet, cp_gaia_virtual_systems_facts and cp_gaia_vsnext_state_facts from CheckPointSW#95 in the 7.1.0 bugfixes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
chkp-majds
pushed a commit
that referenced
this pull request
Nov 3, 2026
Record the RETURN documentation fixes of cp_gaia_simulate_packet, cp_gaia_virtual_systems_facts and cp_gaia_vsnext_state_facts from #95 in the 7.1.0 bugfixes. Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Sanity was tested only on ansible-core 2.15 and 2.16, which are end of life, although the collection declares
requires_ansible: '>=2.15.0'. The Ansible community review of the collection's inclusion request asks for at least 2.18, and 2.19 or later.Changes to
.github/workflows/ansible-test.yml:develas an early warning. Adevelfailure does not fail the run.Any sanity failures that the newer ansible-core versions report will be fixed in this PR.
🤖 Generated with Claude Code