-
Notifications
You must be signed in to change notification settings - Fork 78
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
feat: add watch step to monitor resource changes for kusion apply #125
Conversation
312a3a4
to
844667b
Compare
Pull Request Test Coverage Report for Build 3127971705
💛 - Coveralls |
844667b
to
94fc840
Compare
8d83787
to
1a14945
Compare
1a14945
to
73f4ec3
Compare
73f4ec3
to
a13d961
Compare
a13d961
to
d2ba302
Compare
updated: refactor k8s printer structure |
6de3756
to
b155a5a
Compare
- '-w' flag can control it on or off - support k8s namespace/service/deployment temporarily
b155a5a
to
a2ec8da
Compare
Please add more UTs, since the coverage decreased 0.6% |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
- add Convertors to restore resource type of runtime.Object - add TableGenerator to register new k8s resource - add Printers to get detail and status of k8s resources
a2ec8da
to
10abee3
Compare
lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
related: #23
preview:
![image](https://user-images.githubusercontent.com/23924124/189325221-f3c00139-9b8c-4486-990a-161c220dcdfc.png)