Skip to content

Commit d4ae12d

Browse files
vakwetuclaude
authored andcommitted
[federation] Refactor CA bundle defaults to chained style in hook_controlplane_config
Address non-blocking review suggestions from evallesp and michburk on ci-framework PR #3766. Refactor _federation_ca_bundle_secret_name to use chained | default() filters instead of nesting them. This removes the redundant trailing | default('custom-ca-certs', true) and flattens the remaining nested default into a more readable two-step chain: | default(cifmw_custom_ca_certs_secret_name, true) | default('custom-ca-certs', true) Signed-off-by: Ade Lee <alee@redhat.com> Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 184a5e1 commit d4ae12d

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

roles/federation/tasks/hook_controlplane_config.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@
4848
_federation_ca_bundle_secret_name: >-
4949
{{
5050
((_federation_oscp_info.resources | first).spec.tls | default({})).caBundleSecretName
51-
| default(cifmw_custom_ca_certs_secret_name | default('custom-ca-certs', true), true)
51+
| default(cifmw_custom_ca_certs_secret_name, true)
5252
| default('custom-ca-certs', true)
5353
}}
5454
_federation_oscp_has_ca_bundle: >-

0 commit comments

Comments
 (0)