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

Feat: support remote cluster client to allow user use CRD resource in managed cluster without installing CRD to hub cluster #6008

Merged

Conversation

Somefive
Copy link
Collaborator

Description of your changes

Fixes #5973

Depends on kubevela/pkg#83

I have:

  • Read and followed KubeVela's contribution process.
  • Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
  • Run make reviewable to ensure this PR is ready for review.
  • Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Special notes for your reviewer

@codecov
Copy link

codecov bot commented May 18, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.07 ⚠️

Comparison is base (933d85c) 60.98% compared to head (b80b375) 60.91%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6008      +/-   ##
==========================================
- Coverage   60.98%   60.91%   -0.07%     
==========================================
  Files         225      225              
  Lines       31345    31345              
==========================================
- Hits        19116    19095      -21     
- Misses      10458    10473      +15     
- Partials     1771     1777       +6     
Flag Coverage Δ
core-unittests 56.01% <ø> (-0.03%) ⬇️
e2e-multicluster-test 24.99% <ø> (-0.02%) ⬇️
e2etests 25.28% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 8 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@Somefive Somefive force-pushed the feat/support-remote-cluster-client branch 6 times, most recently from 9792fc9 to 1464ff8 Compare May 19, 2023 06:50
Copy link
Collaborator

@StevenLeiZhang StevenLeiZhang left a 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: Somefive <yd219913@alibaba-inc.com>
@Somefive Somefive force-pushed the feat/support-remote-cluster-client branch from 1464ff8 to b80b375 Compare May 22, 2023 08:56
@Somefive Somefive marked this pull request as ready for review May 22, 2023 11:04
@Somefive Somefive changed the title Feat: support remote cluster client Feat: support remote cluster client to allow user use CRD resource in managed cluster without installing CRD to hub cluster May 22, 2023
@chivalryq chivalryq merged commit c66a639 into kubevela:master May 23, 2023
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Don't need install CRD on HubCluster for customized Component/Trait
3 participants