Conversation
…errors - Update actions/upload-artifact from v3 to v4 (deprecated warning fix) - Update actions/setup-python from v4 to v5 (latest version) - Update actions/cache from v3 to v4 (latest version) - Fix integration-status.yml syntax error with secret access - Replace invalid secrets[variable] syntax with proper secrets.VARIABLE_NAME
- Quote ansible version strings to prevent shell expansion (ansible>=8.0.0,<9.0.0) - Fix docs-build.yml to look for HTML files in correct location (docs/ not docs/html/) - Update artifact upload path to match actual build output location
- Fix ansible-test compatibility issues by simplifying pytest requirements - Add missing __init__.py files for proper Python package structure - Fix tests/config.yml syntax errors - Add 2.0.0 branch to unit test workflow triggers - Update unit test workflow to use pytest as primary test runner with ansible-test fallback - Clean up test requirements to avoid pytest plugin conflicts - Remove .DS_Store files from test directories - Fix integration test tasks syntax in exec_and_wait target
- Change from cross-product matrix to include matrix for better control - Remove incompatible combinations (Python 3.9 + Ansible 10+) - Ansible 10+ requires Python 3.10+ according to pip error - Python 3.9: supports Ansible 6-8 - Python 3.10-3.11: supports all Ansible versions 6-10 - Python 3.12: supports Ansible 8-10 - Reduces matrix from 16 to 14 combinations, all compatible
- Add dynamic import detection for both local and collection environments - Use collection-aware module paths for unittest.mock patches - Fix exception type matching between test and runtime environments - Tests now pass in both local development and GitHub Actions collection structure - All 18 unit tests now pass consistently across environments Technical details: - Import fallback: tries collection imports first, falls back to direct imports - Dynamic CLIENT_MODULE_PATH variable for proper patch targeting - Resolves 'No module named client' errors in collection environment - Fixes exception type mismatches between test assertions and raised exceptions
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
1 participant
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.
No description provided.