Skip to content

Commit

Permalink
ovn-northd: Make it harder to specify a bad database remote.
Browse files Browse the repository at this point in the history
Without this change, --ovnnb-db='' produces bad results, such as an
assertion failure.  With it, ovn-northd uses the default database.  The
latter seems preferable.  Similarly for --ovnsb-db=''.

Acked-by: Numan Siddique <numans@ovn.org>
Signed-off-by: Ben Pfaff <blp@ovn.org>
  • Loading branch information
blp committed Jun 29, 2020
1 parent b160e68 commit e9d3cc1
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions northd/ovn-northd.c
Original file line number Diff line number Diff line change
Expand Up @@ -11758,11 +11758,11 @@ parse_options(int argc OVS_UNUSED, char *argv[] OVS_UNUSED)
}
}

if (!ovnsb_db) {
if (!ovnsb_db || !ovnsb_db[0]) {
ovnsb_db = default_sb_db();
}

if (!ovnnb_db) {
if (!ovnnb_db || !ovnnb_db[0]) {
ovnnb_db = default_nb_db();
}

Expand Down

0 comments on commit e9d3cc1

Please sign in to comment.