Skip to content

Commit 8529b7d

Browse files
Merge pull request #1257 from tchap/hostUsers
OCPBUGS-64732: lib/resourcemerge: Add support for hostUsers flag
2 parents a26c85e + ce5cac7 commit 8529b7d

File tree

2 files changed

+12
-0
lines changed

2 files changed

+12
-0
lines changed

lib/resourcemerge/core.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ func ensurePodSpec(modified *bool, existing *corev1.PodSpec, required corev1.Pod
4747

4848
setStringIfSet(modified, &existing.ServiceAccountName, required.ServiceAccountName)
4949
setBool(modified, &existing.HostNetwork, required.HostNetwork)
50+
setBoolPtr(modified, &existing.HostUsers, required.HostUsers)
5051
mergeMap(modified, &existing.NodeSelector, required.NodeSelector)
5152
ensurePodSecurityContextPtr(modified, &existing.SecurityContext, required.SecurityContext)
5253
ensureAffinityPtr(modified, &existing.Affinity, required.Affinity)

lib/resourcemerge/core_test.go

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,17 @@ func TestEnsurePodSpec(t *testing.T) {
4545
Containers: []corev1.Container{
4646
{Name: "test"}}},
4747
},
48+
{
49+
name: "hostUsers flag is set",
50+
existing: corev1.PodSpec{},
51+
input: corev1.PodSpec{
52+
HostUsers: boolPtr(false),
53+
},
54+
expectedModified: true,
55+
expected: corev1.PodSpec{
56+
HostUsers: boolPtr(false),
57+
},
58+
},
4859
{
4960
name: "PodSecurityContext empty",
5061
existing: corev1.PodSpec{

0 commit comments

Comments
 (0)