Skip to content

Fix readiness probe command argument handling - #519

Merged
morningman merged 1 commit into
apache:masterfrom
Al-assad:fix-readiness-probe-args
Sep 18, 2026
Merged

morningman merged 1 commit into
apache:masterfrom
Al-assad:fix-readiness-probe-args

Conversation

@Al-assad

Copy link
Copy Markdown
Member

Summary

  • pass TLS certificate paths to readiness probes as command arguments
  • execute non-FQDN TLS probes directly with curl
  • preserve the FQDN DNS check while parameterizing curl values
  • add coverage for TLS and FQDN command construction

Testing

  • go test ./pkg/common/utils/resource
  • go vet ./...
  • go build -o /tmp/dorisoperator-pr-check ./cmd/operator
  • make manifests generate
  • validated FQDN and non-FQDN TLS readiness probes on Kubernetes with yulinying/xxx@sha256:f461e8383c1e757ed1a3ebae1130ceed2bfd43c34910cbb29b35ba7f73980a2c

@morningman
morningman merged commit 50ab162 into apache:master Sep 18, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants