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

Install kubectl binary in e2e run suite GH workflow #667

Closed

Conversation

furkatgofurov7
Copy link
Contributor

@furkatgofurov7 furkatgofurov7 commented Aug 16, 2024

What this PR does / why we need it:
Install kubectl binary with specific version v1.30.0 in e2e run suite GH workflow. e2e nightly runs started failing after #659 where we set MINIMUM_KUBECTL_VERSION to v1.30.0. However, version of kubectl used in GH env (not coming from our codebase) seems to be 1.28.4 which is < v1.30.0 MINIMUM_KUBECTL_VERSION, hence, they started failing https://github.com/rancher/turtles/actions/runs/10419056888/job/28856363769.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

Special notes for your reviewer:
Tested this patch here: https://github.com/rancher/turtles/actions/runs/10421045742/job/28862355979

Checklist:

  • squashed commits into logical changes
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests

@furkatgofurov7 furkatgofurov7 requested a review from a team as a code owner August 16, 2024 13:35
Signed-off-by: Furkat Gofurov <furkat.gofurov@suse.com>
@furkatgofurov7 furkatgofurov7 changed the title Use older version of kubectl as minimum binary version Install kubectl binary in e2e run suite GH workflow Aug 16, 2024
Copy link
Contributor

@yiannistri yiannistri left a comment

Choose a reason for hiding this comment

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

As an fyi there is another PR in-flight from Danil that bumps the kind/kubectl version to 1.30.4, these should probably be in sync. https://github.com/rancher/turtles/pull/668/files

Copy link
Contributor

@Danil-Grigorev Danil-Grigorev left a comment

Choose a reason for hiding this comment

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

#668 is similar but they are not blocking each other

@furkatgofurov7 furkatgofurov7 mentioned this pull request Aug 16, 2024
4 tasks
@furkatgofurov7
Copy link
Contributor Author

furkatgofurov7 commented Aug 16, 2024

#668 is similar but they are not blocking each other

I would not merge this, since #668 already fixes e2e nightly without introducing new dependency, so waiting on that to merge and will close this.

@furkatgofurov7 furkatgofurov7 deleted the set-min-kubectl-version branch August 18, 2024 17:37
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.

4 participants