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: use absolute path to rke2's kubectl binary #404

Merged
merged 1 commit into from
Aug 14, 2024

Conversation

simplysoft
Copy link
Contributor

ensures that cluster-etcd secret can be created if host does not have kubectl installed

fixes #403

kind/bug

Checklist:

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

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.

Thanks, lgtm. PTAL @alexander-demicev @furkatgofurov7

@Danil-Grigorev
Copy link
Contributor

@simplysoft we require commits to be signed, this blocks the PR from being merged. Can you fix that?

@simplysoft
Copy link
Contributor Author

@simplysoft we require commits to be signed, this blocks the PR from being merged. Can you fix that?

Sure, didn't find contribution guidelines on first sight, so missed that sorry

@simplysoft simplysoft force-pushed the fix/kubectl-not-found branch from 8dd5625 to cf224b9 Compare August 13, 2024 09:06
Copy link
Contributor

@furkatgofurov7 furkatgofurov7 left a comment

Choose a reason for hiding this comment

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

Thanks for PR @simplysoft.

@furkatgofurov7
Copy link
Contributor

furkatgofurov7 commented Aug 13, 2024

I re-triggered e2e tests and they will hopefully pass, if not, please rebase the branch with main to include the latest e2e fixes and we will re-run them again.

@furkatgofurov7
Copy link
Contributor

I re-triggered e2e tests and they will hopefully pass, if not, please rebase the branch with main to include the latest e2e fixes and we will re-run them again.

This will need a rebase

ensures that cluster-etcd secret can be created if host does not have kubectl installed

fixes rancher#403

Signed-off-by: Simplysoft GmbH <1588210+simplysoft@users.noreply.github.com>
auto-merge was automatically disabled August 13, 2024 14:11

Head branch was pushed to by a user without write access

@simplysoft simplysoft force-pushed the fix/kubectl-not-found branch from cf224b9 to 1412bb8 Compare August 13, 2024 14:12
@furkatgofurov7 furkatgofurov7 merged commit d660678 into rancher:main Aug 14, 2024
7 of 8 checks passed
@furkatgofurov7 furkatgofurov7 added this to the v0.6.0 milestone Aug 14, 2024
@furkatgofurov7 furkatgofurov7 added the kind/bug Something isn't working label Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cluster-etcd secret is not created during cloud-init setup
4 participants