-
Notifications
You must be signed in to change notification settings - Fork 776
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
fix info kind error #5701
fix info kind error #5701
Conversation
TBH I prefer gvk/gvr to be objects. |
If so, is it necessary to unify? Not change here, but change other places? Now they are not unified. |
Codecov Report
@@ Coverage Diff @@
## main #5701 +/- ##
==========================================
- Coverage 34.42% 34.42% -0.01%
==========================================
Files 191 191
Lines 21145 21146 +1
==========================================
Hits 7280 7280
- Misses 13064 13065 +1
Partials 801 801
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
For kind I suppose it should be only the kind, for gvr/gvk it makes sense to keep it as an object, it will allow more fine-grained queries. |
Signed-off-by: cleverhu <shouping.hu@daocloud.io>
f8ead64
to
cddb6bb
Compare
I unified it in |
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
Signed-off-by: cleverhu <shouping.hu@daocloud.io> Signed-off-by: cleverhu <shouping.hu@daocloud.io> Signed-off-by: Md Sahil <Mohdssahil1@gmail.com>
Signed-off-by: cleverhu <shouping.hu@daocloud.io> Signed-off-by: cleverhu <shouping.hu@daocloud.io> Signed-off-by: MdSahil-oss <Mohdssahil1@gmail.com>
Signed-off-by: cleverhu <shouping.hu@daocloud.io> Signed-off-by: cleverhu <shouping.hu@daocloud.io> Signed-off-by: MdSahil-oss <Mohdssahil1@gmail.com>
Explanation
Unify the format of gvk, gvr, kind. Fix info kind msg.
Related issue
#5641
Milestone of this PR
What type of PR is this
/kind bug
Proposed Changes
Proof Manifests
Checklist
Further Comments