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

support kubeconfig options when using kubectl command #1057

Merged
merged 1 commit into from
Mar 27, 2023

Conversation

zxdvd
Copy link
Contributor

@zxdvd zxdvd commented Mar 15, 2023

support kubeconfig options when using kubectl command

Summary: When deploy using px --kubeconfig THE_KUBE_FILE deploy,
it failed with error ✕ Kubectl > 1.10.0 is present ERR: exit status 1.

And I found the px cli has the --kubeconfig option, but the internal
kubectl command didn't respect it.

Relevant Issues: N/A

Type of change: /kind bug

Test Plan: Run px deploy with --kubeconfig option should work without
problem.

@pixie-io-buildbot
Copy link
Member

Can one of the admins verify this patch?

@JamesMBartlett
Copy link
Member

@pixie-io-buildbot test this please

@JamesMBartlett
Copy link
Member

Thanks for your contribution!
Please use our PR template to fill out your PR description so that we can keep our commit messages consistent.

@zxdvd
Copy link
Contributor Author

zxdvd commented Mar 16, 2023

@JamesMBartlett I've updated the commit message.

@JamesMBartlett
Copy link
Member

@pixie-io-buildbot test this please

Copy link
Member

@JamesMBartlett JamesMBartlett left a comment

Choose a reason for hiding this comment

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

LGTM just a small nit on using an environment variable instead of a flag

src/utils/shared/k8s/kubectl.go Outdated Show resolved Hide resolved
@JamesMBartlett
Copy link
Member

@pixie-io-buildbot test this please

2 similar comments
@JamesMBartlett
Copy link
Member

@pixie-io-buildbot test this please

@JamesMBartlett
Copy link
Member

@pixie-io-buildbot test this please

@JamesMBartlett
Copy link
Member

Summary: When deploy using  `px --kubeconfig THE_KUBE_FILE deploy`,
it failed with error `✕    Kubectl > 1.10.0 is present  ERR: exit status 1`.

And I found the px cli has the `--kubeconfig` option, but the internal
kubectl command didn't respect it.

Relevant Issues: N/A

Type of change: /kind bug

Test Plan: Run `px deploy` with `--kubeconfig` option should work without
problem.

Signed-off-by: Xudong Zhang <felixmelon@gmail.com>
@zxdvd
Copy link
Contributor Author

zxdvd commented Mar 24, 2023

Can you please GPG sign your commit? See https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

I've signed the commit and rebased.

@JamesMBartlett
Copy link
Member

@pixie-io-buildbot test this please

@JamesMBartlett JamesMBartlett merged commit ecf9b5c into pixie-io:main Mar 27, 2023
@JamesMBartlett
Copy link
Member

Thanks for your contribution!

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