← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 2072978] Re: Show some error in logs when failing to load nb connection certificate

 

Reviewed:  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/924059
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/efd63d1721742400e7ba2c0bfc55249ef15fc549
Submitter: "Zuul (22348)"
Branch:    master

commit efd63d1721742400e7ba2c0bfc55249ef15fc549
Author: Chris Buggy <cbuggy@xxxxxxxxxx>
Date:   Mon Jul 29 15:16:30 2024 +0200

    Error log for missing certs with NB and SB DBs
    
    When the ovn-provider starts up,
    it attempts to connect to the NB and SB databases
    by retrieving SSL and cert files.
    To avoid errors, the code will now check if these
    files exist before using them.
    If the files are missing,
    connections will be skipped and an error message
    will be displayed in the logs.
    
    Refactoring _check_and_set_ssl_files method to be
    public and reusable. it will now check to see if a
    string value is set and will now check path and
    LOG an error message if not found.
    
    Adding unit tests for ovsdb_monitor to bring up test coverage.
    Updated ovsdb_tests to improve code.
    
    Closes-Bug: #2072978
    Change-Id: I2a21b94fee03767a5f703486bdab2908cda18746


** Changed in: neutron
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/2072978

Title:
  Show some error in logs when failing to load nb connection certificate

Status in neutron:
  Fix Released

Bug description:
  When ovn-provider (api or driver-agent) start up they should connect
  to OVN NB/SB db using certificates in case they are configured in
  config file. Currently in case any of those files are not found they
  avoid the connection and no msg on the logs are shown.

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/2072978/+subscriptions



References