Skip to content

Commit a03a895

Browse files
Merge pull request #428 from vrutkovs/shareProcessNamespace
Bug 1866554: lib/resourcemerge/core: set ShareProcessNamespace, DNSConfig and TerminationGracePeriodSeconds
2 parents 72df098 + 47d87e1 commit a03a895

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

lib/resourcemerge/core.go

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,9 @@ func ensurePodSpec(modified *bool, existing *corev1.PodSpec, required corev1.Pod
5959
ensureTolerations(modified, &existing.Tolerations, required.Tolerations)
6060
setStringIfSet(modified, &existing.PriorityClassName, required.PriorityClassName)
6161
setInt32Ptr(modified, &existing.Priority, required.Priority)
62+
setBoolPtr(modified, &existing.ShareProcessNamespace, required.ShareProcessNamespace)
63+
ensureDNSPolicy(modified, &existing.DNSPolicy, required.DNSPolicy)
64+
setInt64Ptr(modified, &existing.TerminationGracePeriodSeconds, required.TerminationGracePeriodSeconds)
6265
}
6366

6467
func ensureContainers(modified *bool, existing *[]corev1.Container, required []corev1.Container) {
@@ -519,6 +522,13 @@ func ensureResourceList(modified *bool, existing *corev1.ResourceList, required
519522
}
520523
}
521524

525+
func ensureDNSPolicy(modified *bool, existing *corev1.DNSPolicy, required corev1.DNSPolicy) {
526+
if !equality.Semantic.DeepEqual(required, *existing) {
527+
*modified = true
528+
*existing = required
529+
}
530+
}
531+
522532
func setBool(modified *bool, existing *bool, required bool) {
523533
if required != *existing {
524534
*existing = required

0 commit comments

Comments
 (0)