From 9900da3c93b9f0dfdcd51c4d6cfc77a323c77fef Mon Sep 17 00:00:00 2001 From: =?utf8?q?Daniel=20P=2E=20Berrang=C3=A9?= Date: Tue, 16 Apr 2019 17:39:12 +0100 Subject: [PATCH] network: explain reason for bandwidth floor rejection MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit 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é --- src/network/bridge_driver.c | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) 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; } -- 2.47.2