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

print error on cli connection problems #1594

Merged
merged 1 commit into from
Mar 18, 2024

Conversation

eumel8
Copy link
Contributor

@eumel8 eumel8 commented Mar 15, 2024

/kind bugfix

What does this pull request do? Which issues does it resolve?

I had problems to find out the root cause for connection issues. The related error message was dropped. Let's print out this to make debuggung easier

resolves #1584

Please provide a short message that should be published in the vcluster release notes

print error on cli connection problems

What else do we need to know?

Signed-off-by: Frank Kloeker <f.kloeker@telekom.de>
Copy link

netlify bot commented Mar 15, 2024

Deploy Preview for vcluster-docs canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit 4d3fed5
🔍 Latest deploy log https://app.netlify.com/sites/vcluster-docs/deploys/65f3d87a30fa8b0008e37bd2

@FabianKramm
Copy link
Member

Hey @eumel8 ! Sorry for the delay, we did a large refactoring and wanted to get that in before reviewing the new PR's! Your change looks good to me, thanks for contributing!

@FabianKramm FabianKramm merged commit 5429ba7 into loft-sh:main Mar 18, 2024
69 checks passed
@eumel8 eumel8 deleted the fix/connecterror branch March 19, 2024 08:25
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.

Waiting for vcluster to come up
2 participants