6.1.0 does not pass ssl_ca_certs to sentinel

Bug #2063857 reported by Dan Porter
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tooz
Fix Released
Undecided
Unassigned

Bug Description

v6.1.0 introduced a regression during the implementation of optional passing of TLS options to sentinel via the `sentinel_ssl` option.

Commit introducing the issue: 3bce8e1dcaab06175b10fb97d9e681e95c9c6103

A check for "ssl_" prefixed options incorrectly filters for 'ssl_cafile' when the correct option from the client parameters is 'ssl_ca_certs'.

A patch has been attached to rectify this issue.

Revision history for this message
Dan Porter (stealthii) wrote :
description: updated
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tooz (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/tooz/+/918476

Changed in python-tooz:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tooz (master)

Reviewed: https://review.opendev.org/c/openstack/tooz/+/918476
Committed: https://opendev.org/openstack/tooz/commit/d09659ed5b20298234ddc7c1ed75721725dade29
Submitter: "Zuul (22348)"
Branch: master

commit d09659ed5b20298234ddc7c1ed75721725dade29
Author: Daniel Porter <email address hidden>
Date: Fri Apr 26 17:14:52 2024 +0100

    fix: correctly pass ssl_ca_certs to sentinel when enabled

    v6.1.0 introduced a regression when making TLS in sentinel optional, with
    an incorrectly named option being passed (ssl_cafile).

    This change passes the correction option as was previously done in
    v6.0.x.

    Closes-Bug: #2063857
    Change-Id: I784092daea1ab01e4c88157e1c017293a2cb44a8

Changed in python-tooz:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.