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.

Sign up for GitHub

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

Jump to bottom

indicate that portion of the error comes from registry server #395

Open
cppforlife opened this issue Jun 2, 2022 · 2 comments
Open

indicate that portion of the error comes from registry server #395

cppforlife opened this issue Jun 2, 2022 · 2 comments
Labels
carvel accepted This issue should be considered for future work and that the triage process has been completed enhancement This issue is a feature request good first issue An issue that will be a good candidate for a new contributor priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete.

Comments

@cppforlife
Copy link
Contributor

Describe the problem/challenge you have
while reading imgpkg error, it may not be clear to a user that this portion of the error comes from the registry (which would be helpful to understand so that registry docs could be looked at instead of assuming it's an impkg problem).

Describe the solution you'd like
we did something similar in kapp: client-something: API server says: server-something: https://github.com/vmware-tanzu/carvel-kapp/blob/48ff85d5a406257a43e7a8ce1706645ae1f50c85/pkg/kapp/resources/resources.go#L617.

Vote on this request

This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.

👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"

We are also happy to receive and review Pull Requests if you want to help work on this issue.
@cppforlife cppforlife added enhancement This issue is a feature request carvel triage This issue has not yet been reviewed for validity labels Jun 2, 2022
@github-actions
Copy link

This issue is being marked as stale due to a long period of inactivity and will be closed in 5 days if there is no response.

@github-actions github-actions bot added the stale This issue has had no activity for a while and will be closed soon label Jul 13, 2022
@joaopapereira
Copy link
Member

This sounds like an interesting UX optimization. Going to accept the feature

@joaopapereira joaopapereira added good first issue An issue that will be a good candidate for a new contributor carvel accepted This issue should be considered for future work and that the triage process has been completed priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete. and removed carvel triage This issue has not yet been reviewed for validity stale This issue has had no activity for a while and will be closed soon labels Jul 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
carvel accepted This issue should be considered for future work and that the triage process has been completed enhancement This issue is a feature request good first issue An issue that will be a good candidate for a new contributor priority/important-longterm Important over the long term, but may not be staffed and/or may need multiple releases to complete.
Projects
Carvel
Status: Unprioritized
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cppforlife @joaopapereira