Skip to content

NE-2743: Support centralized TLS security profiles for CDO metrics #482

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
openshift-merge-bot merged 6 commits into from
Jul 21, 2026
Merged

NE-2743: Support centralized TLS security profiles for CDO metrics #482

Show file tree
Hide file tree
Changes from 4 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
a91d2ca
NE-2743: Vendor openshift/api and bump dependencies
Thealisyed Jul 10, 2026
4c34182
NE-2743: Align static operator deployment ciphers with Intermediate T…
Thealisyed Jul 10, 2026
669cf57
NE-2743: Sync CRD manifests with vendored openshift/api
Thealisyed Jul 10, 2026
d060e74
NE-2743: Replace kube-rbac-proxy with direct TLS serving for operator…
Thealisyed Jul 14, 2026
98f0b49
NE-2743: Log warning for unsupported TLS groups instead of silently s…
Thealisyed Jul 16, 2026
d4368ad
Address Davide's review feedback
Thealisyed Jul 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
52 changes: 46 additions & 6 deletions cmd/dns-operator/main.go
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,26 +1,33 @@
package main

import (
"context"
"crypto/tls"
"flag"
"os"

configv1 "github.com/openshift/api/config/v1"
configclient "github.com/openshift/client-go/config/clientset/versioned"
operatorcontroller "github.com/openshift/cluster-dns-operator/pkg/operator/controller"

"github.com/openshift/cluster-dns-operator/pkg/operator"
operatorconfig "github.com/openshift/cluster-dns-operator/pkg/operator/config"
statuscontroller "github.com/openshift/cluster-dns-operator/pkg/operator/controller/status"

"github.com/sirupsen/logrus"

metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
ctrl "sigs.k8s.io/controller-runtime"
"sigs.k8s.io/controller-runtime/pkg/client/config"
"sigs.k8s.io/controller-runtime/pkg/log/zap"
"sigs.k8s.io/controller-runtime/pkg/manager/signals"
metricsserver "sigs.k8s.io/controller-runtime/pkg/metrics/server"
)

const operatorNamespace = "openshift-dns-operator"

func main() {
metricsserver.DefaultBindAddress = "127.0.0.1:60000"
metricsBindAddr := flag.String("metrics-bind-addr", "127.0.0.1:60000", "The TCP address for the metrics endpoint.")
metricsCertDir := flag.String("metrics-cert-dir", "", "The directory containing tls.crt and tls.key for the metrics endpoint.")

opts := zap.Options{
Development: true,
Expand Down Expand Up @@ -49,18 +56,51 @@ func main() {
logrus.Fatalf("KUBE_RBAC_PROXY_IMAGE environment variable is required")
}

kubeConfig, err := config.GetConfig()
if err != nil {
logrus.Fatalf("failed to get kube config: %v", err)
}

// Build TLS options for the operator metrics server from the
// cluster-wide TLS security profile when secure serving is enabled.
var metricsTLSOpts []func(*tls.Config)
if *metricsCertDir != "" {
cfgClient, err := configclient.NewForConfig(kubeConfig)
if err != nil {
logrus.Fatalf("failed to create config client: %v", err)
}
apiServer, err := cfgClient.ConfigV1().APIServers().Get(context.TODO(), "cluster", metav1.GetOptions{})
var tlsSecurityProfile *configv1.TLSSecurityProfile
Comment thread
Thealisyed marked this conversation as resolved.
if err != nil {
logrus.Warningf("failed to get apiserver config, using Intermediate TLS profile: %v", err)
} else {
tlsSecurityProfile = apiServer.Spec.TLSSecurityProfile
}
profileSpec := operatorcontroller.TLSProfileSpecForSecurityProfile(tlsSecurityProfile)
tlsCfg, err := operatorcontroller.TLSConfigFromProfile(profileSpec)
if err != nil {
logrus.Fatalf("failed to build TLS config from profile: %v", err)
}
metricsTLSOpts = append(metricsTLSOpts, func(cfg *tls.Config) {
cfg.CipherSuites = tlsCfg.CipherSuites
cfg.MinVersion = tlsCfg.MinVersion
if len(tlsCfg.CurvePreferences) > 0 {
cfg.CurvePreferences = tlsCfg.CurvePreferences
}
})
}

operatorConfig := operatorconfig.Config{
OperatorNamespace: operatorNamespace,
OperatorReleaseVersion: releaseVersion,
CoreDNSImage: coreDNSImage,
OpenshiftCLIImage: cliImage,
KubeRBACProxyImage: kubeRBACProxyImage,
MetricsBindAddress: *metricsBindAddr,
MetricsCertDir: *metricsCertDir,
MetricsTLSOpts: metricsTLSOpts,
}

kubeConfig, err := config.GetConfig()
if err != nil {
logrus.Fatalf("failed to get kube config %v", err)
}
ctx := signals.SetupSignalHandler()
// Set up and start the operator.
op, err := operator.New(ctx, operatorConfig, kubeConfig)
Expand Down
43 changes: 32 additions & 11 deletions go.mod
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,40 +6,47 @@ require (
github.com/apparentlymart/go-cidr v1.1.0
github.com/go-logr/logr v1.4.3
github.com/google/go-cmp v0.7.0
github.com/openshift/api v0.0.0-20251111193948-50e2ece149d7
github.com/openshift/api v0.0.0-20260615110019-261e3a0546f3
github.com/openshift/build-machinery-go v0.0.0-20250530140348-dc5b2804eeee
github.com/openshift/client-go v0.0.0-20251015124057-db0dee36e235
github.com/openshift/client-go v0.0.0-20260317180604-743f664b82d1
github.com/openshift/coredns-ocp-dnsnameresolver/operator v0.0.0-20260211095308-1a5277b4db1c
github.com/openshift/library-go v0.0.0-20251119174848-88c26bf0df68
github.com/sirupsen/logrus v1.9.3
github.com/stretchr/testify v1.11.1
k8s.io/api v0.35.0
k8s.io/apimachinery v0.35.0
k8s.io/client-go v0.35.0
k8s.io/kubectl v0.35.0
k8s.io/api v0.35.1
k8s.io/apimachinery v0.35.1
k8s.io/client-go v0.35.1
k8s.io/kubectl v0.35.1
k8s.io/utils v0.0.0-20251002143259-bc988d571ff4
sigs.k8s.io/controller-runtime v0.23.1
)

require (
cel.dev/expr v0.24.0 // indirect
github.com/antlr4-go/antlr/v4 v4.13.0 // indirect
github.com/beorn7/perks v1.0.1 // indirect
github.com/blang/semver/v4 v4.0.0 // indirect
github.com/cenkalti/backoff/v4 v4.3.0 // indirect
github.com/cespare/xxhash/v2 v2.3.0 // indirect
github.com/davecgh/go-spew v1.1.2-0.20180830191138-d8f796af33cc // indirect
github.com/emicklei/go-restful/v3 v3.12.2 // indirect
github.com/evanphx/json-patch v5.6.0+incompatible // indirect
github.com/evanphx/json-patch/v5 v5.9.11 // indirect
github.com/felixge/httpsnoop v1.0.4 // indirect
github.com/fsnotify/fsnotify v1.9.0 // indirect
github.com/fxamacker/cbor/v2 v2.9.0 // indirect
github.com/go-logr/stdr v1.2.2 // indirect
github.com/go-logr/zapr v1.3.0 // indirect
github.com/go-openapi/jsonpointer v0.21.0 // indirect
github.com/go-openapi/jsonreference v0.20.2 // indirect
github.com/go-openapi/swag v0.23.0 // indirect
github.com/gogo/protobuf v1.3.2 // indirect
github.com/google/btree v1.1.3 // indirect
github.com/google/cel-go v0.26.0 // indirect
github.com/google/gnostic-models v0.7.0 // indirect
github.com/google/uuid v1.6.0 // indirect
github.com/grpc-ecosystem/grpc-gateway/v2 v2.26.3 // indirect
github.com/imdario/mergo v0.3.12 // indirect
github.com/inconshreveable/mousetrap v1.1.0 // indirect
github.com/josharian/intern v1.0.0 // indirect
github.com/json-iterator/go v1.1.12 // indirect
github.com/mailru/easyjson v0.7.7 // indirect
Expand All @@ -53,14 +60,24 @@ require (
github.com/prometheus/common v0.66.1 // indirect
github.com/prometheus/procfs v0.16.1 // indirect
github.com/robfig/cron v1.2.0 // indirect
github.com/spf13/cobra v1.10.0 // indirect
github.com/spf13/pflag v1.0.9 // indirect
github.com/stoewer/go-strcase v1.3.0 // indirect
github.com/x448/float16 v0.8.4 // indirect
go.opentelemetry.io/auto/sdk v1.1.0 // indirect
go.opentelemetry.io/contrib/instrumentation/net/http/otelhttp v0.61.0 // indirect
go.opentelemetry.io/otel v1.36.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace v1.34.0 // indirect
go.opentelemetry.io/otel/exporters/otlp/otlptrace/otlptracegrpc v1.34.0 // indirect
go.opentelemetry.io/otel/metric v1.36.0 // indirect
go.opentelemetry.io/otel/sdk v1.36.0 // indirect
go.opentelemetry.io/otel/trace v1.36.0 // indirect
Comment on lines +69 to +74

@coderabbitai coderabbitai Bot Jul 10, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔒 Security & Privacy | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '\n## .agents/context.md\n'
cat -n .agents/context.md

printf '\n## go.mod (relevant lines)\n'
sed -n '45,70p' go.mod | cat -n

printf '\n## otel usage in the repo\n'
rg -n --hidden --glob '!**/vendor/**' 'go\.opentelemetry\.io/otel|go\.opentelemetry\.io/otel/trace' .

printf '\n## go.sum entries for otel\n'
rg -n '^go\.opentelemetry\.io/otel|^go\.opentelemetry\.io/otel/trace' go.sum

Repository: openshift/cluster-dns-operator

Length of output: 242

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '## repo root\n'
pwd
printf '\n## top-level files\n'
git ls-files | sed -n '1,200p'

printf '\n## locate .agents and go.mod\n'
fd -a '^\.agents$|^go\.mod$|^go\.sum$' .

printf '\n## go.mod relevant lines\n'
sed -n '1,140p' go.mod | cat -n

printf '\n## go.sum otel entries\n'
rg -n '^go\.opentelemetry\.io/otel|^go\.opentelemetry\.io/otel/trace' go.sum || true

printf '\n## repo references to otel\n'
rg -n --hidden --glob '!**/vendor/**' 'go\.opentelemetry\.io/otel|go\.opentelemetry\.io/otel/trace' . || true

Repository: openshift/cluster-dns-operator

Length of output: 15447

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '## .agents/context.md\n'
sed -n '1,220p' .agents/context.md | cat -n

printf '\n## any otel/opentelemetry references outside go.mod/go.sum\n'
rg -n --hidden --glob '!**/vendor/**' 'opentelemetry|go\.opentelemetry\.io/otel' . || true

printf '\n## any otel/opentelemetry references in vendor\n'
rg -n --hidden 'opentelemetry|go\.opentelemetry\.io/otel' vendor || true

Repository: openshift/cluster-dns-operator

Length of output: 252

🌐 Web query:

OSV CVE-2026-29181 go.opentelemetry.io/otel v1.36.0 fixed version 1.41.0

💡 Result:

CVE-2026-29181 is a security vulnerability in the OpenTelemetry-Go library (go.opentelemetry.io/otel) related to improper control of resource allocation [1][2][3]. Specifically, the vulnerability exists in the handling of multi-value 'baggage' headers [2][3][4]. Versions 1.36.0 through 1.40.0 are affected because the library parses each header field-value independently and aggregates members across these values, allowing an attacker to trigger excessive CPU and memory allocations even when individual values remain within the 8192-byte parsing limit [3][4][5]. This issue was addressed and is fixed in version 1.41.0 of the OpenTelemetry-Go library [3][4][6]. Users are advised to upgrade to version 1.41.0 or higher to mitigate this risk [5][6].

Citations:

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '## hidden files at repo root\n'
ls -la

printf '\n## .agents files if present\n'
fd -a 'context.md|architecture.md|controllers.md|rules.md|testing.md|operand.md' .agents || true

printf '\n## otel references outside go.mod/go.sum\n'
rg -n --hidden --glob '!go.mod' --glob '!go.sum' --glob '!**/vendor/**' 'go\.opentelemetry\.io/otel|go\.opentelemetry\.io/otel/trace|opentelemetry' . || true

printf '\n## otel files in vendor\n'
fd -a 'otel|opentelemetry' vendor || true

Repository: openshift/cluster-dns-operator

Length of output: 1861

🏁 Script executed:

#!/bin/bash
set -euo pipefail

printf '## repo source imports\n'
rg -n --glob '!vendor/**' '^import \(|"sigs\.k8s\.io/controller-runtime|"go\.opentelemetry\.io/otel' cmd pkg test tools | sed -n '1,220p'

printf '\n## vendor packages that import otel\n'
rg -n 'go\.opentelemetry\.io/otel' vendor | sed -n '1,220p'

printf '\n## controller-runtime dependency chain hints\n'
rg -n 'controller-runtime' go.mod go.sum vendor | sed -n '1,220p'

Repository: openshift/cluster-dns-operator

Length of output: 40847

Upgrade the OpenTelemetry pins
go.mod:57-58 still pin go.opentelemetry.io/otel and go.opentelemetry.io/otel/trace to v1.36.0, which is affected by GHSA-mh2q-q3fh-2475 / CVE-2026-29181. Bump both to v1.41.0 or newer.

Suggested diff 
-	go.opentelemetry.io/otel v1.36.0 // indirect
-	go.opentelemetry.io/otel/trace v1.36.0 // indirect
+	go.opentelemetry.io/otel v1.41.0 // indirect
+	go.opentelemetry.io/otel/trace v1.41.0 // indirect
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
go.opentelemetry.io/otel v1.36.0 // indirect
go.opentelemetry.io/otel/trace v1.36.0 // indirect
go.opentelemetry.io/otel v1.41.0 // indirect
go.opentelemetry.io/otel/trace v1.41.0 // indirect
🧰 Tools
🪛 OSV Scanner (2.4.0)

[HIGH] 57-57: go.opentelemetry.io/otel 1.36.0: OpenTelemetry-Go: multi-value baggage header extraction causes excessive allocations (remote dos amplification)

(GHSA-mh2q-q3fh-2475)

🤖 Prompt for AI Agents 
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@go.mod` around lines 57 - 58, Update the go.opentelemetry.io/otel and
go.opentelemetry.io/otel/trace dependencies in go.mod from v1.36.0 to v1.41.0 or
newer, then refresh go.sum as needed and verify the module dependency graph
remains consistent.

Sources: Path instructions, Linters/SAST tools

go.opentelemetry.io/proto/otlp v1.5.0 // indirect
go.uber.org/multierr v1.11.0 // indirect
go.uber.org/zap v1.27.0 // indirect
go.yaml.in/yaml/v2 v2.4.3 // indirect
go.yaml.in/yaml/v3 v3.0.4 // indirect
golang.org/x/exp v0.0.0-20240719175910-8a7402abbf56 // indirect
golang.org/x/mod v0.29.0 // indirect
golang.org/x/net v0.47.0 // indirect
golang.org/x/oauth2 v0.30.0 // indirect
Expand All @@ -71,16 +88,20 @@ require (
golang.org/x/time v0.9.0 // indirect
golang.org/x/tools v0.38.0 // indirect
gomodules.xyz/jsonpatch/v2 v2.4.0 // indirect
google.golang.org/genproto/googleapis/api v0.0.0-20250303144028-a0af3efb3deb // indirect
google.golang.org/genproto/googleapis/rpc v0.0.0-20250528174236-200df99c418a // indirect
google.golang.org/grpc v1.72.2 // indirect
google.golang.org/protobuf v1.36.8 // indirect
gopkg.in/evanphx/json-patch.v4 v4.13.0 // indirect
gopkg.in/inf.v0 v0.9.1 // indirect
gopkg.in/yaml.v3 v3.0.1 // indirect
k8s.io/apiextensions-apiserver v0.35.0 // indirect
k8s.io/apiserver v0.35.0 // indirect
k8s.io/component-base v0.35.0 // indirect
k8s.io/apiextensions-apiserver v0.35.1 // indirect
k8s.io/apiserver v0.35.1 // indirect
k8s.io/component-base v0.35.1 // indirect
k8s.io/klog/v2 v2.130.1 // indirect
k8s.io/kube-aggregator v0.35.0 // indirect
k8s.io/kube-aggregator v0.35.1 // indirect
k8s.io/kube-openapi v0.0.0-20250910181357-589584f1c912 // indirect
sigs.k8s.io/apiserver-network-proxy/konnectivity-client v0.31.2 // indirect
sigs.k8s.io/json v0.0.0-20250730193827-2d320260d730 // indirect
sigs.k8s.io/kube-storage-version-migrator v0.0.6-0.20230721195810-5c8923c5ff96 // indirect
sigs.k8s.io/randfill v1.0.0 // indirect
Expand Down
Loading