From: Daniel P. Berrangé Date: Tue, 16 Apr 2019 16:39:12 +0000 (+0100) Subject: network: explain reason for bandwidth floor rejection X-Git-Tag: v5.3.0-rc1~27 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=9900da3c93b9f0dfdcd51c4d6cfc77a323c77fef;p=thirdparty%2Flibvirt.git network: explain reason for bandwidth floor rejection Reword error messages to make it clear that the combined floor settings of all NICs are exceeding the network inbound peak/average settings. Including the actual values being checked helps to diagnose what is actually wrong. Reviewed-by: Laine Stump Signed-off-by: Daniel P. Berrangé --- diff --git a/src/network/bridge_driver.c b/src/network/bridge_driver.c index 3008846074..6ed0bf1e8e 100644 --- a/src/network/bridge_driver.c +++ b/src/network/bridge_driver.c @@ -5225,10 +5225,13 @@ networkCheckBandwidth(virNetworkObjPtr obj, tmp_new_rate = netBand->in->peak; if (tmp_floor_sum > netBand->in->peak) { virReportError(VIR_ERR_OPERATION_INVALID, - _("Cannot plug '%s' interface into '%s' because it " - "would overcommit 'peak' on network '%s'"), + _("Cannot plug '%s' interface into '%s' because " + "new combined inbound floor=%llu would overcommit " + "peak=%llu on network '%s'"), ifmac, def->bridge, + tmp_floor_sum, + netBand->in->peak, def->name); goto cleanup; } @@ -5236,10 +5239,13 @@ networkCheckBandwidth(virNetworkObjPtr obj, /* tmp_floor_sum can be between 'average' and 'peak' iff 'peak' is set. * Otherwise, tmp_floor_sum must be below 'average'. */ virReportError(VIR_ERR_OPERATION_INVALID, - _("Cannot plug '%s' interface into '%s' because it " - "would overcommit 'average' on network '%s'"), + _("Cannot plug '%s' interface into '%s' because " + "new combined inbound floor=%llu would overcommit " + "average=%llu on network '%s'"), ifmac, def->bridge, + tmp_floor_sum, + netBand->in->average, def->name); goto cleanup; }