Skip to content

Enable deep JSON merging for Kubernetes resources when updating them #1019

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.

Sign up for GitHub

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

Merged
ibm-ci-bot merged 7 commits into from
Mar 27, 2024
Merged

Enable deep JSON merging for Kubernetes resources when updating them #1019

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
f23382c
enable deep json merge for k8s resources
Feb 27, 2024
73450a4
fix deep merge for resource spec
Mar 11, 2024
65e5ee8
merge for all resrources type
Mar 12, 2024
3a4a1a1
each changed object updated once
Mar 14, 2024
9705c95
compare whole resrouces file instead of spec section
Mar 25, 2024
b59fd58
set ownerreferences
Mar 25, 2024
2e86588
ensure ann, label and set owner on updated resrouce
Mar 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
set ownerreferences 
Signed-off-by: YuChen <[email protected]>
  • Loading branch information
YuChen committed Mar 25, 2024
commit b59fd58d5ac5a5cc2611b9bd0adf8201ef86dd0c
3 changes: 3 additions & 0 deletions controllers/operandrequest/reconcile_operand.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1077,6 +1077,9 @@ func (r *Reconciler) updateK8sResource(ctx context.Context, existingK8sRes unstr

r.EnsureAnnotation(existingK8sRes, newAnnotations)
r.EnsureLabel(existingK8sRes, newLabels)
if err := r.setOwnerReferences(ctx, &existingK8sRes, ownerReferences); err != nil {
return false, errors.Wrapf(err, "failed to set ownerReferences for k8s resource -- Kind: %s, NamespacedName: %s/%s", kind, namespace, name)
}

if reflect.DeepEqual(existingK8sRes.Object, updatedExistingK8sRes) {
return true, nil
Expand Down