Skip to content

Commit

Permalink
Cherry-pick: Fix the requirement for VAP
Browse files Browse the repository at this point in the history
  • Loading branch information
varshaprasad96 committed Jan 15, 2025
1 parent 3c49a16 commit 68ee180
Show file tree
Hide file tree
Showing 39 changed files with 3,123 additions and 1 deletion.
14 changes: 13 additions & 1 deletion pkg/visibility/server.go
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,10 @@ import (
"os"
"strings"

validatingadmissionpolicy "k8s.io/apiserver/pkg/admission/plugin/policy/validating"
"k8s.io/apiserver/pkg/admission/plugin/resourcequota"
mutatingwebhook "k8s.io/apiserver/pkg/admission/plugin/webhook/mutating"
validatingwebhook "k8s.io/apiserver/pkg/admission/plugin/webhook/validating"
openapinamer "k8s.io/apiserver/pkg/endpoints/openapi"
genericapiserver "k8s.io/apiserver/pkg/server"
genericoptions "k8s.io/apiserver/pkg/server/options"
Expand All @@ -41,6 +45,14 @@ import (

var (
setupLog = ctrl.Log.WithName("visibility-server")
// Admission plugins that are enabled by default in the kubeapi server
// but are not required for the visibility server.
disabledPlugins = []string{
validatingadmissionpolicy.PluginName,
resourcequota.PluginName,
validatingwebhook.PluginName,
mutatingwebhook.PluginName,
}
)

// +kubebuilder:rbac:groups=flowcontrol.apiserver.k8s.io,resources=prioritylevelconfigurations,verbs=list;watch
Expand Down Expand Up @@ -81,7 +93,7 @@ func applyVisibilityServerOptions(config *genericapiserver.RecommendedConfig) er
o.SecureServing.BindPort = 8082
// The directory where TLS certs will be created
o.SecureServing.ServerCert.CertDirectory = "/tmp"

o.Admission.DisablePlugins = disabledPlugins
if err := o.SecureServing.MaybeDefaultWithSelfSignedCerts("localhost", nil, []net.IP{net.ParseIP("127.0.0.1")}); err != nil {
return fmt.Errorf("error creating self-signed certificates: %v", err)
}
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 68ee180

Please sign in to comment.