Allow `RunnerDeployment`s to configure `dnsPolicy` for runners (#1892)
* Add DnsPolicy field to RunnerPodSpec struct * Ensure the runnerSpec's DNSPolicy is mirrored to the pod.Spec * Run `make manifests`
This commit is contained in:
		
							parent
							
								
									6eadb03669
								
							
						
					
					
						commit
						a389292478
					
				| 
						 | 
					@ -170,6 +170,9 @@ type RunnerPodSpec struct {
 | 
				
			||||||
	// +optional
 | 
						// +optional
 | 
				
			||||||
	RuntimeClassName *string `json:"runtimeClassName,omitempty"`
 | 
						RuntimeClassName *string `json:"runtimeClassName,omitempty"`
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						// +optional
 | 
				
			||||||
 | 
						DnsPolicy corev1.DNSPolicy `json:"dnsPolicy,omitempty"`
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	// +optional
 | 
						// +optional
 | 
				
			||||||
	DnsConfig *corev1.PodDNSConfig `json:"dnsConfig,omitempty"`
 | 
						DnsConfig *corev1.PodDNSConfig `json:"dnsConfig,omitempty"`
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -1381,6 +1381,9 @@ spec:
 | 
				
			||||||
                                type: string
 | 
					                                type: string
 | 
				
			||||||
                              type: array
 | 
					                              type: array
 | 
				
			||||||
                          type: object
 | 
					                          type: object
 | 
				
			||||||
 | 
					                        dnsPolicy:
 | 
				
			||||||
 | 
					                          description: DNSPolicy defines how a pod's DNS will be configured.
 | 
				
			||||||
 | 
					                          type: string
 | 
				
			||||||
                        dockerEnabled:
 | 
					                        dockerEnabled:
 | 
				
			||||||
                          type: boolean
 | 
					                          type: boolean
 | 
				
			||||||
                        dockerEnv:
 | 
					                        dockerEnv:
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -1378,6 +1378,9 @@ spec:
 | 
				
			||||||
                                type: string
 | 
					                                type: string
 | 
				
			||||||
                              type: array
 | 
					                              type: array
 | 
				
			||||||
                          type: object
 | 
					                          type: object
 | 
				
			||||||
 | 
					                        dnsPolicy:
 | 
				
			||||||
 | 
					                          description: DNSPolicy defines how a pod's DNS will be configured.
 | 
				
			||||||
 | 
					                          type: string
 | 
				
			||||||
                        dockerEnabled:
 | 
					                        dockerEnabled:
 | 
				
			||||||
                          type: boolean
 | 
					                          type: boolean
 | 
				
			||||||
                        dockerEnv:
 | 
					                        dockerEnv:
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -1325,6 +1325,9 @@ spec:
 | 
				
			||||||
                        type: string
 | 
					                        type: string
 | 
				
			||||||
                      type: array
 | 
					                      type: array
 | 
				
			||||||
                  type: object
 | 
					                  type: object
 | 
				
			||||||
 | 
					                dnsPolicy:
 | 
				
			||||||
 | 
					                  description: DNSPolicy defines how a pod's DNS will be configured.
 | 
				
			||||||
 | 
					                  type: string
 | 
				
			||||||
                dockerEnabled:
 | 
					                dockerEnabled:
 | 
				
			||||||
                  type: boolean
 | 
					                  type: boolean
 | 
				
			||||||
                dockerEnv:
 | 
					                dockerEnv:
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -1381,6 +1381,9 @@ spec:
 | 
				
			||||||
                                type: string
 | 
					                                type: string
 | 
				
			||||||
                              type: array
 | 
					                              type: array
 | 
				
			||||||
                          type: object
 | 
					                          type: object
 | 
				
			||||||
 | 
					                        dnsPolicy:
 | 
				
			||||||
 | 
					                          description: DNSPolicy defines how a pod's DNS will be configured.
 | 
				
			||||||
 | 
					                          type: string
 | 
				
			||||||
                        dockerEnabled:
 | 
					                        dockerEnabled:
 | 
				
			||||||
                          type: boolean
 | 
					                          type: boolean
 | 
				
			||||||
                        dockerEnv:
 | 
					                        dockerEnv:
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -1378,6 +1378,9 @@ spec:
 | 
				
			||||||
                                type: string
 | 
					                                type: string
 | 
				
			||||||
                              type: array
 | 
					                              type: array
 | 
				
			||||||
                          type: object
 | 
					                          type: object
 | 
				
			||||||
 | 
					                        dnsPolicy:
 | 
				
			||||||
 | 
					                          description: DNSPolicy defines how a pod's DNS will be configured.
 | 
				
			||||||
 | 
					                          type: string
 | 
				
			||||||
                        dockerEnabled:
 | 
					                        dockerEnabled:
 | 
				
			||||||
                          type: boolean
 | 
					                          type: boolean
 | 
				
			||||||
                        dockerEnv:
 | 
					                        dockerEnv:
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -1325,6 +1325,9 @@ spec:
 | 
				
			||||||
                        type: string
 | 
					                        type: string
 | 
				
			||||||
                      type: array
 | 
					                      type: array
 | 
				
			||||||
                  type: object
 | 
					                  type: object
 | 
				
			||||||
 | 
					                dnsPolicy:
 | 
				
			||||||
 | 
					                  description: DNSPolicy defines how a pod's DNS will be configured.
 | 
				
			||||||
 | 
					                  type: string
 | 
				
			||||||
                dockerEnabled:
 | 
					                dockerEnabled:
 | 
				
			||||||
                  type: boolean
 | 
					                  type: boolean
 | 
				
			||||||
                dockerEnv:
 | 
					                dockerEnv:
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -650,6 +650,10 @@ func (r *RunnerReconciler) newPod(runner v1alpha1.Runner) (corev1.Pod, error) {
 | 
				
			||||||
		pod.Spec.HostAliases = runnerSpec.HostAliases
 | 
							pod.Spec.HostAliases = runnerSpec.HostAliases
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
						if runnerSpec.DnsPolicy != "" {
 | 
				
			||||||
 | 
							pod.Spec.DNSPolicy = runnerSpec.DnsPolicy
 | 
				
			||||||
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	if runnerSpec.DnsConfig != nil {
 | 
						if runnerSpec.DnsConfig != nil {
 | 
				
			||||||
		pod.Spec.DNSConfig = runnerSpec.DnsConfig
 | 
							pod.Spec.DNSConfig = runnerSpec.DnsConfig
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in New Issue