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

ci(e2e-test): add test for k8s 1.30.5 and 1.31.1 #134

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

g-iannelli
Copy link
Contributor

@g-iannelli g-iannelli commented Oct 25, 2024

Changes:

  • updated pluto target version to 1.31
  • removed e2e test for version 1.26.6 and 1.27.3
  • updated kind to 0.24
  • updated sighup/e2e-testing image using the new one with bats, kubectl, customize and yq
  • add e2e test for version 1.30.5 and 1.31.1

Depends on:

@g-iannelli
Copy link
Contributor Author

I am not able to test locally because I do not have a linux amd64 machine to run the drone pipeline locally.

Copy link
Member

@ralgozino ralgozino left a comment

Choose a reason for hiding this comment

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

you can test these changes by tagging a release candidate on this branch, that should trigger the e2e tests.

@g-iannelli g-iannelli force-pushed the feat/sighupio/product-management#443/update-e2e-test-for-1.30-1.31 branch 2 times, most recently from b2fc94e to d719c94 Compare October 29, 2024 08:15
@ralgozino ralgozino changed the base branch from main to develop October 29, 2024 08:18
@ralgozino ralgozino changed the base branch from develop to main October 29, 2024 08:18
@g-iannelli g-iannelli force-pushed the feat/sighupio/product-management#443/update-e2e-test-for-1.30-1.31 branch from d719c94 to b8c1a30 Compare October 29, 2024 08:34
@g-iannelli
Copy link
Contributor Author

you can test these changes by tagging a release candidate on this branch, that should trigger the e2e tests.

It seems work. The pre-release step failed due to dependency from #135

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