karmada icon indicating copy to clipboard operation
karmada copied to clipboard

Generate work permanent id in webhook

Open whitewindmills opened this issue 1 year ago • 7 comments

What type of PR is this? /kind cleanup

What this PR does / why we need it: Since the PR(https://github.com/karmada-io/karmada/pull/4474) has been merged, we're going to remove those codes which generate work permanent ID outside of webhook.

Which issue(s) this PR fixes: Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

NONE

whitewindmills avatar May 10 '24 11:05 whitewindmills

The test is failing.

RainbowMango avatar May 11 '24 01:05 RainbowMango

Codecov Report

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

Project coverage is 53.17%. Comparing base (4ccffcc) to head (93a04cb).

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

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4928      +/-   ##
==========================================
- Coverage   53.18%   53.17%   -0.01%     
==========================================
  Files         252      252              
  Lines       20510    20508       -2     
==========================================
- Hits        10909    10906       -3     
  Misses       8882     8882              
- Partials      719      720       +1     
Flag Coverage Δ
unittests 53.17% <ø> (-0.01%) :arrow_down:

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

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

codecov-commenter avatar May 11 '24 01:05 codecov-commenter

please take a look @RainbowMango @XiShanYongYe-Chang

whitewindmills avatar May 11 '24 02:05 whitewindmills

This breaks version compatibility, it is recommended to add releast-note for this.

liangyuanpeng avatar May 11 '24 06:05 liangyuanpeng

@liangyuanpeng The feature version that enables work permanent ID has not been released yet.

whitewindmills avatar May 11 '24 07:05 whitewindmills

thanks for the clarification,make sense!

liangyuanpeng avatar May 11 '24 07:05 liangyuanpeng

/retitle cleanup dead code of generating Work permanent id

RainbowMango avatar May 13 '24 07:05 RainbowMango

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: RainbowMango

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

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

karmada-bot avatar May 13 '24 07:05 karmada-bot