Skip to content

Commit 5fad4eb

Browse files
authored
Merge pull request #445 from rohan47/dns_policy_ctrlplugin
driver: set correct DNSPolicy based on HostNetwork for ctrl plugins
2 parents d8aea4e + 26a2600 commit 5fad4eb

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

internal/controller/driver_controller.go

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -597,8 +597,14 @@ func (r *driverReconcile) reconcileControllerPluginDeployment() error {
597597
ServiceAccountName: serviceAccountName,
598598
PriorityClassName: ptr.Deref(pluginSpec.PrioritylClassName, ""),
599599
HostNetwork: ptr.Deref(pluginSpec.HostNetwork, false),
600-
Affinity: getControllerPluginPodAffinity(pluginSpec, &appSelector),
601-
Tolerations: pluginSpec.Tolerations,
600+
DNSPolicy: utils.Call(func() corev1.DNSPolicy {
601+
if ptr.Deref(pluginSpec.HostNetwork, false) {
602+
return corev1.DNSClusterFirstWithHostNet
603+
}
604+
return corev1.DNSClusterFirst
605+
}),
606+
Affinity: getControllerPluginPodAffinity(pluginSpec, &appSelector),
607+
Tolerations: pluginSpec.Tolerations,
602608
Containers: utils.Call(func() []corev1.Container {
603609
containers := []corev1.Container{
604610
// Plugin Container

0 commit comments

Comments
 (0)