Skip to content

Conversation

@ykim-akamai
Copy link
Contributor

@ykim-akamai ykim-akamai commented Nov 1, 2024

📝 Description

This PR adds workflow_dispatch input to allow e2e tests run against EOL python version as an option. Also remove 3.8 from unit test matrix as it is no longer supported.

e.g.
image

✔️ How to Test

Forked e2e run with EOL:

📷 Preview

If applicable, include a screenshot or code snippet of this change. Otherwise, please remove this section.

@ykim-akamai ykim-akamai added the testing for updates to the testing suite in the changelog. label Nov 1, 2024
@ykim-akamai ykim-akamai requested a review from a team as a code owner November 1, 2024 20:18
@ykim-akamai ykim-akamai requested review from ezilber-akamai and yec-akamai and removed request for a team November 1, 2024 20:18
description: 'The hash value of the commit'
required: false
default: ''
run-eol-python-version:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Would it make sense to include another variable here for us to specify which python version we want to run against? E.g. 3.12

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, I think that's a good idea! It will be really nice to be able to run the tests against all supported python versions easily.

@ykim-akamai ykim-akamai changed the title test: Allow e2e and unit tests to run against EOL python version test: Allow integration and unit tests to run against EOL and desired python version Nov 5, 2024
@ykim-akamai ykim-akamai changed the title test: Allow integration and unit tests to run against EOL and desired python version test: Allow integration and unit tests to run against EOL or desired python version Nov 5, 2024
Copy link
Contributor

@yec-akamai yec-akamai left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ezilber-akamai ezilber-akamai left a comment

Choose a reason for hiding this comment

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

Great work!

@ykim-akamai ykim-akamai merged commit 9ca4937 into linode:dev Nov 7, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing for updates to the testing suite in the changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants