You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
// I was able to remove the following by removing the namespace checking logic of OTE. Current assumption is origin is making sure all permissions are set before launching OTE.
10
-
//k8s.io/apiserver => github.com/openshift/kubernetes/staging/src/k8s.io/apiserver v0.0.0-20250716113245-b94367cabf3e // openshift kubernetes has very old copy of k8s.io/kubernetes/pkg/kubelet/server/server.go
11
-
//k8s.io/kubelet => github.com/openshift/kubernetes/staging/src/k8s.io/kubelet v0.0.0-20250716113245-b94367cabf3e // openshift kubernetes has very old copy of k8s.io/kubernetes/cmd/kubelet/app/options/options.go:
k8s.io/apiserver => github.com/openshift/kubernetes/staging/src/k8s.io/apiserverv0.0.0-20251015171918-61114aa5a292// openshift kubernetes has very old copy of k8s.io/kubernetes/pkg/kubelet/server/server.go
10
+
k8s.io/kubelet => github.com/openshift/kubernetes/staging/src/k8s.io/kubeletv0.0.0-20251015171918-61114aa5a292// openshift kubernetes has very old copy of k8s.io/kubernetes/cmd/kubelet/app/options/options.go
// Check if host placement is configured. If so, then we need to determine placement affinity and validate configs.
907
-
ifproviderSpec.HostPlacement!=nil {
908
-
klog.V(4).Infof("Validating AWS Host Placement")
909
-
placement:=*providerSpec.HostPlacement
910
-
ifplacement.Affinity==nil {
911
-
errs=append(errs, field.Required(field.NewPath("spec.hostPlacement.affinity"), "affinity is required and must be set to either AnyAvailable or DedicatedHost"))
912
-
} else {
913
-
switch*placement.Affinity {
914
-
casemachinev1beta1.HostAffinityAnyAvailable:
915
-
// Cannot have DedicatedHost set
916
-
ifplacement.DedicatedHost!=nil {
917
-
errs=append(errs, field.Forbidden(field.NewPath("spec.hostPlacement.dedicatedHost"), "dedicatedHost is required when affinity is DedicatedHost, and forbidden otherwise"))
918
-
}
919
-
casemachinev1beta1.HostAffinityDedicatedHost:
920
-
// We need to make sure DedicatedHost is set with a HostID
921
-
ifplacement.DedicatedHost==nil {
922
-
errs=append(errs, field.Required(field.NewPath("spec.hostPlacement.dedicatedHost"), "dedicatedHost is required when affinity is DedicatedHost, and forbidden otherwise"))
923
-
} else {
924
-
// If not set, return required error. If it does not match pattern, return pattern failure message.
925
-
ifplacement.DedicatedHost.ID=="" {
926
-
errs=append(errs, field.Required(field.NewPath("spec.hostPlacement.dedicatedHost.id"), "id is required and must start with 'h-' followed by 17 lowercase hexadecimal characters (0-9 and a-f)"))
errs=append(errs, field.Invalid(field.NewPath("spec.hostPlacement.dedicatedHost.id"), placement.DedicatedHost.ID, "id must start with 'h-' followed by 17 lowercase hexadecimal characters (0-9 and a-f)"))
929
-
}
930
-
}
931
-
default:
932
-
errs=append(errs, field.Invalid(field.NewPath("spec.hostPlacement.affinity"), placement.Affinity, "affinity must be either AnyAvailable or DedicatedHost"))
![openshift-merge-bot[bot]](https://avatars.githubusercontent.com/in/412865?v=4&size=40){kind=avatar}
0 commit comments