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

Show more user feedback when Zarf cannot connect to a cluster #1541

Closed
Racer159 opened this issue Apr 2, 2023 · 1 comment · Fixed by #1590
Closed

Show more user feedback when Zarf cannot connect to a cluster #1541

Racer159 opened this issue Apr 2, 2023 · 1 comment · Fixed by #1590
Assignees
Labels
bug 🐞 Something isn't working
Milestone

Comments

@Racer159
Copy link
Contributor

Racer159 commented Apr 2, 2023

Is your feature request related to a problem? Please describe.

When Zarf cannot connect to a cluster it will stall after "Copying 1 files" for quite some time.
image

Describe the solution you'd like

We should show more clearly that we are trying to connect to a cluster but are unable to do so. We should also likely lower the timeout for receiving a response.

Describe alternatives you've considered

None

Additional context

This is something that has impacted users with misconfigured ~/./kube/config files in the real world.

@github-project-automation github-project-automation bot moved this to New Requests in Zarf Project Board Apr 2, 2023
@Racer159 Racer159 added the bug 🐞 Something isn't working label Apr 2, 2023
@Racer159
Copy link
Contributor Author

Racer159 commented Apr 2, 2023

Labeling this as a bug because the 30 second timeout we are setting is getting ignored.

@Racer159 Racer159 self-assigned this Apr 18, 2023
@Racer159 Racer159 added this to the v0.26.0 milestone Apr 18, 2023
Racer159 added a commit that referenced this issue Apr 18, 2023
…ushes (#1590)

## Description

This PR creates a tunnel per image push (making it easier to implement
concurrency - may do that in this PR if we can confirm that issues are
mitigated) moves the CRC from the image name to the tag and changes the
UI to use a progressbar instead of a spinner for better user feedback.

## Related Issue

Relates to #1568 , #1433, #1218, #1364

This also will make #1594 slightly easier.

(See aws/containers-roadmap#853)

Fixes: #1541

## Type of change

- [X] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [ ] Other (security config, docs update, etc)

## Checklist before merging

- [X] Test, docs, adr added or updated as needed
- [X] [Contributor Guide
Steps](https://github.com/defenseunicorns/zarf/blob/main/CONTRIBUTING.md#developer-workflow)
followed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant