Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix molecule tests for spm-toolbox #10

Merged
merged 2 commits into from
Oct 23, 2024
Merged

Fix molecule tests for spm-toolbox #10

merged 2 commits into from
Oct 23, 2024

Conversation

romakarol
Copy link
Contributor

@romakarol romakarol commented Oct 23, 2024

Description

All molecule tests are failing for spm-toolbox, blocking proper code delivery.

spm-middleware molecule seems to be passing, so issue is unique to spm-toolbox:
https://github.com/merative/spm-middleware/actions/runs/11049713204/job/31060554861

How

Branch from main
Commit any change (such as a readme change)
All tests will fail in the 'main' gh workflow which performs the molecule testing. Molecule converge (for all tests) fails with a rather generic "The following modules failed to execute: ansible.legacy.setup".

Fix

Issue seem related to older versions of ansible running on python 3.7+ ansible/ansible#82068. Update our requirements to use the same ci-requirements as the spm-middleware repo with explicit versions for ansible and the rest.

@romakarol romakarol changed the title use only requirements.txt with explciit versions Fix molecule tests for spm-toolbox Oct 23, 2024
@romakarol romakarol changed the title Fix molecule tests for spm-toolbox SPM-???? Fix molecule tests for spm-toolbox Oct 23, 2024
@salob salob self-requested a review October 23, 2024 23:09
@salob salob changed the title SPM-???? Fix molecule tests for spm-toolbox Fix molecule tests for spm-toolbox Oct 23, 2024
@salob salob merged commit 700ce23 into main Oct 23, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants