Skip to content

[18-stable] Include ssl.conf in httpd configuration - #694

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:18-stablefrom
openshift-cherrypick-robot:cherry-pick-693-to-18-stable
Aug 5, 2026
Merged

[18-stable] Include ssl.conf in httpd configuration#694
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:18-stablefrom
openshift-cherrypick-robot:cherry-pick-693-to-18-stable

Conversation

@openshift-cherrypick-robot

Copy link
Copy Markdown

This is an automated cherry-pick of #693

/assign fmount

The ssl.conf rendered by lib-common was placed by kolla into conf.d/
but never loaded because the Include conf.d/*.conf directive was
commented out. This left global SSL hardening settings (cipher suite,
protocol restrictions, session cache) at mod_ssl defaults instead of
the operator-managed values.

Replace the explicit Include of 10-cinder_wsgi.conf with the conf.d
wildcard so both ssl.conf and the wsgi vhost config are loaded.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Francesco Pantano <fpantano@redhat.com>
@fmount

fmount commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

/approve
/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Aug 5, 2026
@openshift-ci

openshift-ci Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: fmount, openshift-cherrypick-robot

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot added the approved label Aug 5, 2026
@openshift-merge-bot
openshift-merge-bot Bot merged commit d767fb3 into openstack-k8s-operators:18-stable Aug 5, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants