-
Notifications
You must be signed in to change notification settings - Fork 771
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
sidecarset support patch pod metadata proposal #993
sidecarset support patch pod metadata proposal #993
Conversation
70593ff
to
b0f022e
Compare
|
||
// IgnorePatchPolicy indicates if PatchPodFields conflicts with Pod, | ||
// will ignore PatchPodFields, and keep the corresponding fields of pods. | ||
IgnorePatchPolicy PatchPolicyType = "Ignore" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ignore -> Retain
b0f022e
to
10c6d1b
Compare
Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com>
10c6d1b
to
d7ee280
Compare
Codecov Report
@@ Coverage Diff @@
## master #993 +/- ##
=======================================
Coverage 49.34% 49.34%
=======================================
Files 125 125
Lines 12355 12355
=======================================
Hits 6096 6096
Misses 5320 5320
Partials 939 939
Flags with carried forward coverage won't be shown. Click here to find out more. 📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: FillZpp 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 |
Signed-off-by: peng.xin <peng.xin@ly.com> optimize workloadspread when suitable subset maxReplicas is niil (openkruise#1066) Signed-off-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com> Signed-off-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com> Co-authored-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com> Optimize performance of LabelSelector conversion (openkruise#1068) Signed-off-by: FillZpp <FillZpp.pub@gmail.com> Signed-off-by: FillZpp <FillZpp.pub@gmail.com> Support timezone for AdvancedCronJob (openkruise#1070) Signed-off-by: FillZpp <FillZpp.pub@gmail.com> Signed-off-by: FillZpp <FillZpp.pub@gmail.com> ignore if pod condition has been updated (openkruise#1074) Signed-off-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com> Signed-off-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com> Co-authored-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com> pod probe marker apis (openkruise#1073) * pod probe marker apis Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> * fix statefulset truncateHistory panic Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> pod probe marker controller (openkruise#1075) Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> consider whether patch field is matched when assign existing pods to subset (openkruise#1083) Signed-off-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com> Signed-off-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com> Co-authored-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com> pod probe marker webhook (openkruise#1078) Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> sidecarset support pods ns(kube-system, kube-public) (openkruise#1084) Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> pod probe marker proposal (openkruise#1053) Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> node pod probe daemon (openkruise#1077) Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> sidecarset support patch pod metadata proposal (openkruise#993) Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> Support predownload image in ads (openkruise#1057) Signed-off-by: Abner <abner199709@gmail.com> * support predownload image for ads Add PreDownloadImageForDaemonSetUpdate featureGate (openkruise#1093) Signed-off-by: FillZpp <FillZpp.pub@gmail.com> Signed-off-by: FillZpp <FillZpp.pub@gmail.com> Add changelog for v1.3.0 (openkruise#1092) Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> fix sidecarset inject annotations abnormal (openkruise#1101) Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> fix InPlaceUpdateEnvFromMetadata bug (openkruise#1108) Co-authored-by: 郭已钦 <xiangyang2@xiaohongshu.com> support uniteddeployment persistentVolumeClaimRetentionPolicy inherit from advancestatefulset template (openkruise#1110) Signed-off-by: cheyuexian <cheyuexian@bilibili.com> Signed-off-by: cheyuexian <cheyuexian@bilibili.com> Co-authored-by: cheyuexian <cheyuexian@bilibili.com> Add PreNormal Lifecycle Hook for CloneSet (openkruise#1071) * add pre-normal(pre-available) hook Signed-off-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com> * add pre-normal(pre-available) hook Signed-off-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com> Signed-off-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com> Co-authored-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com> partition support float percent (openkruise#1124) Signed-off-by: shiyan2016 <shiyan20160606@gmail.com> Signed-off-by: shiyan2016 <shiyan20160606@gmail.com> improve error hanlding of inplace update env from metadata (openkruise#1125) Signed-off-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com> Signed-off-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com> Co-authored-by: mingzhou.swx <mingzhou.swx@alibaba-inc.com> pub support to configure Evict,Delete,Update Operation (openkruise#1126) Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> Signed-off-by: liheng.zms <liheng.zms@alibaba-inc.com> feat: add watch whitelist feat: add watch whitelist add watch whitelist check add dynamic watch option feat: add persistent annotations format feat: add pps support custom workload Signed-off-by: peng.xin <peng.xin@ly.com> fix: nodeSelector and nodeAffinity have not been modified Signed-off-by: peng.xin <peng.xin@ly.com> fix: error string Signed-off-by: peng.xin <peng.xin@ly.com> fix: error string Signed-off-by: peng.xin <peng.xin@ly.com> fix: goimports Signed-off-by: peng.xin <peng.xin@ly.com> refactor: refactoring ersistentPodAnnotations type Signed-off-by: peng.xin <peng.xin@ly.com> - add statefulet group check - modify the pod create processing logic Signed-off-by: peng.xin <peng.xin@ly.com> refactor: pps watch of whitelist workload Signed-off-by: peng.xin <peng.xin@ly.com> fix: add statefulsetlike's pod controller Signed-off-by: peng.xin <peng.xin@ly.com>
Signed-off-by: liheng.zms liheng.zms@alibaba-inc.com
Ⅰ. Describe what this PR does
Some sidecar containers may require special configurations that take effect via annotations/labels. Thus, sidecarSet should support inject or in-place update these configurations.
User Story: