On Tue, Aug 30, 2022 at 6:05 PM Mike Pattrick <[email protected]> wrote:
>
> Add additional logging for debug and info level with a focus on code
> related to bond members coming up, go down, and changing.
>
> Several existing log messages were modified to handle sub 1kB log
> messages with more grace. Instead of reporting 0kB of traffic load
> shifting from one member to another, we now suppress these messages at
> the INFO level and display exact byte count at the debug level.
>
> Signed-off-by: Mike Pattrick <[email protected]>

[snip]

> @@ -1131,9 +1135,15 @@ log_bals(struct bond *bond, const struct ovs_list 
> *bals)
>          LIST_FOR_EACH (member, bal_node, bals) {
>              if (ds.length) {
>                  ds_put_char(&ds, ',');
> +                ds_put_cstr(&ds, ", ");

Hum, double ',' here.
This can probably be fixed while applying.


With this fixed, you can add:
Reviewed-by: David Marchand <[email protected]>


-- 
David Marchand

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to