-
Notifications
You must be signed in to change notification settings - Fork 919
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 cluster registration step in pull mode to installation-cli.yaml #6006
base: master
Are you sure you want to change the base?
feat : add cluster registration step in pull mode to installation-cli.yaml #6006
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #6006 +/- ##
==========================================
- Coverage 48.33% 48.12% -0.22%
==========================================
Files 666 668 +2
Lines 54858 55163 +305
==========================================
+ Hits 26516 26545 +29
- Misses 26617 26893 +276
Partials 1725 1725
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
…kflow Signed-off-by: Akash <[email protected]>
e94498a
to
5d8cefa
Compare
/assign |
@SkySingh04 the ci failed due to not finding the command karmadactl register cluster \
--cluster-name ${{ matrix.k8s }} \
--mode Pull |
… generation Signed-off-by: Akash Singh <[email protected]>
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…command generation Signed-off-by: Akash Singh <[email protected]>
…and maintainability Signed-off-by: Sky Singh <[email protected]>
…ency Signed-off-by: Sky Singh <[email protected]>
…C configuration steps Signed-off-by: Sky Singh <[email protected]>
…egistration steps and improving log handling Signed-off-by: Sky Singh <[email protected]>
…d RBAC configuration steps Signed-off-by: Sky Singh <[email protected]>
…eval and cluster registration steps Signed-off-by: Sky Singh <[email protected]>
…ents and organizing test steps Signed-off-by: Sky Singh <[email protected]>
…blank lines Signed-off-by: Sky Singh <[email protected]>
@zhzhuang-zju This PR is also related to the issue we are facing in #6007 as well. Reference : |
@SkySingh04 The CI has two jobs, For example, the job karmada/hack/cli-testing-environment.sh Lines 115 to 118 in 3d25407
|
Signed-off-by: Akash [email protected]
What type of PR is this?
What this PR does / why we need it:
Which issue(s) this PR fixes:
Parts of #5916
Special notes for your reviewer:
Does this PR introduce a user-facing change?: