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: make tests backwards compatible with nerdctl v1 #212

Merged

Conversation

austinvazquez
Copy link
Member

Issue #, if available:
nerdctl v2 rollout is quick for macOS and Windows, but slow for Linux (~weeks).

Description of changes:
This change enables test runners to specify the nerdctl version depending on the runtime for that platform and tests will run accordingly.

Testing done:
Ran in runfinch/finch#1237

  • I've reviewed the guidance in CONTRIBUTING.md

License Acceptance

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@austinvazquez austinvazquez force-pushed the make-tests-nerdctlv1-compatible branch from 47faa7c to 5a4bd56 Compare January 3, 2025 16:50
@austinvazquez austinvazquez force-pushed the make-tests-nerdctlv1-compatible branch from 5a4bd56 to 43a5702 Compare January 3, 2025 17:07
@austinvazquez austinvazquez marked this pull request as ready for review January 3, 2025 22:09
Copy link
Member

@pendo324 pendo324 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@austinvazquez
Copy link
Member Author

Going to pull this in. Happy to go comeback and iterate on this more if needed.

@austinvazquez austinvazquez merged commit 9f5e817 into runfinch:main Jan 4, 2025
3 checks passed
@austinvazquez austinvazquez deleted the make-tests-nerdctlv1-compatible branch January 4, 2025 00:57
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.

3 participants