-
Notifications
You must be signed in to change notification settings - Fork 859
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
Introduce a new annotation propagationpolicy.karmada.io/generation #5156
base: master
Are you sure you want to change the base?
Conversation
[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 |
/cc @chaosi-zju PTAL. |
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #5156 +/- ##
==========================================
+ Coverage 28.21% 28.22% +0.01%
==========================================
Files 632 632
Lines 43553 43590 +37
==========================================
+ Hits 12289 12305 +16
- Misses 30367 30388 +21
Partials 897 897
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
/assign Thanks, this is a nice improvement! I'll take a closer look tomorrow. |
Do you think it is necessary to add generation annotations to ResourceBinding to improve observability? |
I'm not sure, we don't have that requirement at the moment. |
6e0476c
to
4a1c91f
Compare
It seems fine, but I suggest bringing this |
Signed-off-by: chaunceyjiang <chaunceyjiang@gmail.com>
4a1c91f
to
3e53285
Compare
A question is whether your product uses the It seems a general requirement that users want to know whether their workload is correctly handled by specific PropagationPolicy, now we have the permanent ID on the resource template, but we still lack more accurate info to figure out if subsequent changes are applied or not. |
Yes, our product has not yet adopted it. The plan is to introduce some features of Karmada into our product this month. We have conducted research on this special effect and after internal discussions, we found the problem I described in the issue. Before introducing this special effect, we always assumed that PP takes effect immediately, so our product can find the effective PP through the PP name and PP namespace on the resource template. |
If you do mean the |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Previously, a new feature was introduced for PP via #4602. Simply put, when the PP is changed, it will not take effect immediately. It only takes effect when the resource template selected by the PP changes.
However, this has raised a new issue in our product. Users cannot quickly determine whether the current resource template has applied the latest PP.
Therefore, I am introducing a new annotation here that stores the
generation
of an already effective PP. This way users can quickly confirm whether they have applied the latest PP by comparing if theirgeneration
is equal.Which issue(s) this PR fixes:
When PropagationPolicy is in
Lazy
mode, introduce a new annotationpropagationpolicy.karmada.io/generation
andclusterpropagationpolicy.karmada.io/generation
.Special notes for your reviewer:
Does this PR introduce a user-facing change?: