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

Note that context is not present in local Kubeconfig #1147

Merged
merged 2 commits into from
Nov 3, 2020

Conversation

snitkdan
Copy link
Contributor

@snitkdan snitkdan commented Nov 3, 2020

Signed-off-by: Daniel Snitkovskiy thelastdan1@gmail.com

Fixes #1017

Proposed changes

  • Log the original error message (with the full error attached)
  • Note to the client that $CONTEXT not found in $KUBECONFIG

image

cc @rberrelleza

Signed-off-by: Daniel Snitkovskiy <thelastdan1@gmail.com>
@derek
Copy link

derek bot commented Nov 3, 2020

Thank you for your contribution. I've just checked and your commit doesn't appear to be signed-off. That's something we need before your Pull Request can be merged. Please see our contributing guide.
Tip: if you only have one commit so far then run: git commit --amend --signoff and then git push --force.

Signed-off-by: Daniel Snitkovskiy <thelastdan1@gmail.com>
@derek derek bot removed the no-dco label Nov 3, 2020
Copy link
Member

@rberrelleza rberrelleza left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

@rberrelleza rberrelleza merged commit 691fde8 into okteto:master Nov 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

User friendly message when the context is not found
2 participants