Neels Hofmeyr has submitted this change and it was merged. Change subject: resource: Allow passing keyword args to NoResourceExn ......................................................................
resource: Allow passing keyword args to NoResourceExn Same code is already being used by log.Error exception. This way we can use: raise resource.NoResourceExn("No free arfcns in any of bands", bands=supported_bands) And get: osmo_gsm_tester.resource.NoResourceExn: No free arfcns in any of bands {bands=['GSM-850', 'GSM-900', 'GSM-1800', 'GSM-1900']} Change-Id: Ic960ae8db5372b0a4640356f0b4820ed42477b46 --- M src/osmo_gsm_tester/resource.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Neels Hofmeyr: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/osmo_gsm_tester/resource.py b/src/osmo_gsm_tester/resource.py index f1697ea..7e55129 100644 --- a/src/osmo_gsm_tester/resource.py +++ b/src/osmo_gsm_tester/resource.py @@ -208,7 +208,7 @@ return next_msisdn -class NoResourceExn(Exception): +class NoResourceExn(log.Error): pass class Resources(dict): -- To view, visit https://gerrit.osmocom.org/3954 To unsubscribe, visit https://gerrit.osmocom.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic960ae8db5372b0a4640356f0b4820ed42477b46 Gerrit-PatchSet: 4 Gerrit-Project: osmo-gsm-tester Gerrit-Branch: master Gerrit-Owner: Pau Espin Pedrol <pes...@sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Neels Hofmeyr <nhofm...@sysmocom.de>