Message ID | 20191025133646.4635-1-russell@ovn.org |
---|---|
State | Accepted |
Headers | show |
Series | [ovs-dev,ovn] controller: Downgrade a warning log message | expand |
On Fri, Oct 25, 2019 at 7:07 PM Russell Bryant <russell@ovn.org> wrote: > This log message was introduced in commit 5344f24ecb. It gets hit > under normal circumstances, so it would be better as a debug message > instead of a warning. I also expanded it to clarify that the next > step will be to create the chassis record. > > This was found by trying to run the system-ovn.at tests, and they > failed because of these unexpected warning log messages. > > Signed-off-by: Russell Bryant <russell@ovn.org> > Acked-by: Numan Siddique <numans@ovn.org> Thanks Numan --- > controller/chassis.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/controller/chassis.c b/controller/chassis.c > index 699b66281..978273e19 100644 > --- a/controller/chassis.c > +++ b/controller/chassis.c > @@ -484,8 +484,9 @@ chassis_get_record(struct ovsdb_idl_txn *ovnsb_idl_txn, > chassis_rec = chassis_lookup_by_name(sbrec_chassis_by_name, > > chassis_info_id(&chassis_state)); > if (!chassis_rec) { > - VLOG_WARN("Could not find Chassis : stored (%s) ovs (%s)", > - chassis_info_id(&chassis_state), chassis_id); > + VLOG_DBG("Could not find Chassis, will create it" > + ": stored (%s) ovs (%s)", > + chassis_info_id(&chassis_state), chassis_id); > if (ovnsb_idl_txn) { > /* Recreate the chassis record. */ > chassis_rec = sbrec_chassis_insert(ovnsb_idl_txn); > -- > 2.21.0 > > _______________________________________________ > dev mailing list > dev@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev >
On Fri, Oct 25, 2019 at 10:17 AM Numan Siddique <numans@ovn.org> wrote: > > > > On Fri, Oct 25, 2019 at 7:07 PM Russell Bryant <russell@ovn.org> wrote: >> >> This log message was introduced in commit 5344f24ecb. It gets hit >> under normal circumstances, so it would be better as a debug message >> instead of a warning. I also expanded it to clarify that the next >> step will be to create the chassis record. >> >> This was found by trying to run the system-ovn.at tests, and they >> failed because of these unexpected warning log messages. >> >> Signed-off-by: Russell Bryant <russell@ovn.org> > > > Acked-by: Numan Siddique <numans@ovn.org> Thanks! I've applied this to master.
diff --git a/controller/chassis.c b/controller/chassis.c index 699b66281..978273e19 100644 --- a/controller/chassis.c +++ b/controller/chassis.c @@ -484,8 +484,9 @@ chassis_get_record(struct ovsdb_idl_txn *ovnsb_idl_txn, chassis_rec = chassis_lookup_by_name(sbrec_chassis_by_name, chassis_info_id(&chassis_state)); if (!chassis_rec) { - VLOG_WARN("Could not find Chassis : stored (%s) ovs (%s)", - chassis_info_id(&chassis_state), chassis_id); + VLOG_DBG("Could not find Chassis, will create it" + ": stored (%s) ovs (%s)", + chassis_info_id(&chassis_state), chassis_id); if (ovnsb_idl_txn) { /* Recreate the chassis record. */ chassis_rec = sbrec_chassis_insert(ovnsb_idl_txn);
This log message was introduced in commit 5344f24ecb. It gets hit under normal circumstances, so it would be better as a debug message instead of a warning. I also expanded it to clarify that the next step will be to create the chassis record. This was found by trying to run the system-ovn.at tests, and they failed because of these unexpected warning log messages. Signed-off-by: Russell Bryant <russell@ovn.org> --- controller/chassis.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)