Skip to content

Exclude Google Trust Services ClusterIssuers when reencrypt route is enabled#143

Merged
treddy08 merged 1 commit into
mainfrom
rhacs-exclude-google-trust-reencrypt
May 2, 2026
Merged

Exclude Google Trust Services ClusterIssuers when reencrypt route is enabled#143
treddy08 merged 1 commit into
mainfrom
rhacs-exclude-google-trust-reencrypt

Conversation

@treddy08
Copy link
Copy Markdown
Contributor

@treddy08 treddy08 commented May 2, 2026

Summary

  • Filter out Google Trust Services ClusterIssuers when ocp4_workload_rhacs_enable_reencrypt_route is enabled
  • Fixes incompatibility between Google Trust Services certificates and OpenShift reencrypt routes

Changes

Modified roles/ocp4_workload_rhacs/tasks/certificate.yml:

  • Updated ClusterIssuer filter on line 40 to exclude issuers with /acme/google/ in their ACME server URL when reencrypt route is enabled
  • Maintains existing behavior when reencrypt route is disabled

Testing

Tested on clusters with both Google Trust Services and ZeroSSL certificates:

  • ✅ Reencrypt route works with ZeroSSL certificates
  • ❌ Reencrypt route fails (503) with Google Trust Services certificates
  • ✅ Passthrough route works with both certificate providers

Impact

When ocp4_workload_rhacs_enable_reencrypt_route: true, the role will automatically use ZeroSSL, Let's Encrypt, or other non-Google ACME providers for certificate issuance.

…enabled

When ocp4_workload_rhacs_enable_reencrypt_route is set to true, filter out
ClusterIssuers that use Google Trust Services (/acme/google/) as they are
incompatible with OpenShift reencrypt routes.

The role will now prefer ZeroSSL, Let's Encrypt, or other ACME providers
for certificate issuance when the reencrypt route feature is enabled, while
maintaining existing behavior when reencrypt is disabled.
@treddy08 treddy08 merged commit dc8d2a6 into main May 2, 2026
1 check passed
@treddy08 treddy08 deleted the rhacs-exclude-google-trust-reencrypt branch May 2, 2026 13:11
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.

1 participant