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

kctrl : Progress tailing for repo add and delete. #671

Closed
5 tasks
100mik opened this issue May 10, 2022 · 2 comments · Fixed by #764
Closed
5 tasks

kctrl : Progress tailing for repo add and delete. #671

100mik opened this issue May 10, 2022 · 2 comments · Fixed by #764
Assignees
@100mik
Labels
carvel-accepted This issue should be considered for future work and that the triage process has been completed cli Issue for kapp-controller cli enhancement This issue is a feature request priority/unprioritized-backlog Higher priority than priority/awaiting-more-evidence but not planned. Contributions are welcome.

Comments

@100mik
Copy link
Contributor

100mik commented May 10, 2022 •
edited
Loading

Describe the problem/challenge you have
kctrl should tail the status of a PackageRepo similar to App and PackageInstall status tailing

Describe the solution you'd like
A tail similar to App tailing which illustrates reconciliation progress

Acceptance criteria

  • Repository statuses are tailed while reconciling/deleting
  • Errors that might happen are surfaced
  • Can we indicate packages that are shared between repositories better? (ongoing work by @joe-kimmel-vmw )
  • kick, status and pause commands for PackageRepository
  • Tests

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 working on this issue.
@100mik 100mik added enhancement This issue is a feature request carvel-triage This issue has not yet been reviewed for validity labels May 10, 2022
@100mik
Copy link
Contributor Author

100mik commented May 10, 2022

@joe-kimmel-vmw I believe we were iterating over a lot of PackageRepo stuff.
Especially, regarding shared pakages and how they are created on the cluster.
This can go ahead once we have sort of decided on that and maybe when the information surfaced by the status field is "stable"
Feel free to add anything relevant that will help here!

@100mik 100mik added carvel-accepted This issue should be considered for future work and that the triage process has been completed cli Issue for kapp-controller cli and removed carvel-triage This issue has not yet been reviewed for validity labels May 10, 2022
@100mik 100mik mentioned this issue May 10, 2022
Closed
62 tasks
@100mik 100mik added the priority/unprioritized-backlog Higher priority than priority/awaiting-more-evidence but not planned. Contributions are welcome. label May 10, 2022
@100mik 100mik self-assigned this May 10, 2022
@cppforlife
Copy link
Contributor

we are not changing status API for PackageRepository CR as part of above mentioned work.

@100mik 100mik mentioned this issue Jun 26, 2022
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@100mik 100mik

Labels
carvel-accepted This issue should be considered for future work and that the triage process has been completed cli Issue for kapp-controller cli enhancement This issue is a feature request priority/unprioritized-backlog Higher priority than priority/awaiting-more-evidence but not planned. Contributions are welcome.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add tailing behaviour to package repo and add a package repo kick command carvel-dev/kapp-controller
2 participants
@cppforlife @100mik