search

karmada-io / karmada

Open, Multi-Cloud, Multi-Cluster Kubernetes Orchestration
https://karmada.io
Apache License 2.0
4.14k stars 813 forks source link

CI: add ci for operator installation method #4401

Open zhzhuang-zju opened 5 months ago

zhzhuang-zju commented 5 months ago

What type of PR is this? /kind feature

What this PR does / why we need it:

Which issue(s) this PR fixes: Fixes #

Special notes for your reviewer: NONE Does this PR introduce a user-facing change?:

NONE
codecov-commenter commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 53.14%. Comparing base (5bc8c54) to head (934990e). Report is 8 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #4401 +/- ## ========================================== + Coverage 53.12% 53.14% +0.01% ========================================== Files 251 251 Lines 20417 20417 ========================================== + Hits 10847 10851 +4 + Misses 8856 8853 -3 + Partials 714 713 -1 ``` | [Flag](https://app.codecov.io/gh/karmada-io/karmada/pull/4401/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=karmada-io) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/karmada-io/karmada/pull/4401/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=karmada-io) | `53.14% <ø> (+0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=karmada-io#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

XiShanYongYe-Chang commented 3 weeks ago

Ask @calvin0327 to help take a look. /cc @calvin0327

calvin0327 commented 3 weeks ago

/assign

zhzhuang-zju commented 3 weeks ago

@calvin0327 Thank you for the review. Any other review comments?

zhzhuang-zju commented 1 week ago

kindly ping @calvin0327 @XiShanYongYe-Chang

liangyuanpeng commented 1 week ago

/approve for github workflow.

/hold for avoid unexpected merge.

karmada-bot commented 1 week ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: liangyuanpeng To complete the pull request process, please ask for approval from calvin0327 and additionally assign lfbear after the PR has been reviewed. You can assign the PR to them by writing /assign @lfbear in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - ~~[.github/workflows/OWNERS](https://github.com/karmada-io/karmada/blob/master/.github/workflows/OWNERS)~~ [liangyuanpeng] - **[hack/OWNERS](https://github.com/karmada-io/karmada/blob/master/hack/OWNERS)** - **[operator/OWNERS](https://github.com/karmada-io/karmada/blob/master/operator/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment