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

Improve test suite #55

Merged
merged 4 commits into from
Apr 24, 2024
Merged

Improve test suite #55

merged 4 commits into from
Apr 24, 2024

Conversation

lkubb
Copy link
Member

@lkubb lkubb commented Apr 24, 2024

What does this PR do?

  • Drops long EOL Vault versions from test suite, adds 1.14.8 as static reference point (it's the last version with the previous license)
  • Uses upstream containers instead of SaltStack mirror
  • Raises pytest output verbosity
  • removes hard dependency on Docker as container orchestrator
  • tries to autodetect a rootless Podman socket if not running in CI to be able to run the functional/integration tests

What issues does this PR fix or reference?

Fixes: #51

Previous Behavior

  • tests run also against laughably outdated Vault versions
  • trying to run the tests with Podman does not work

New Behavior

  • tests against Vault latest and 14.1.8
  • Podman rootless works for local testing

Merge requirements satisfied?

Commits signed with GPG?

Yes

@lkubb lkubb marked this pull request as ready for review April 24, 2024 18:00
@lkubb lkubb merged commit 19439f1 into salt-extensions:main Apr 24, 2024
17 checks passed
@lkubb lkubb deleted the update-containers branch April 24, 2024 18:19
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.

[TECH DEBT] Update CI containers
1 participant