Cole Robinson <crobinso@xxxxxxxxxx> writes: > On 03/26/2014 08:51 AM, Giuseppe Scrivano wrote: >> Signed-off-by: Giuseppe Scrivano <gscrivan@xxxxxxxxxx> >> --- >> OK to include this patch into the series? >> >> virtinst/osdict.py | 7 +++++-- >> 1 file changed, 5 insertions(+), 2 deletions(-) >> >> diff --git a/virtinst/osdict.py b/virtinst/osdict.py >> index 13f6670..f37ccab 100644 >> --- a/virtinst/osdict.py >> +++ b/virtinst/osdict.py >> @@ -165,9 +165,12 @@ def get_recommended_resources(variant, arch): >> >> >> def lookup_os_by_media(location): >> - media = libosinfo.Media.create_from_location(location, None) >> + try: >> + media = libosinfo.Media.create_from_location(location, None) >> + except: >> + return None > > I'd like to see the error here logged, it either shouldn't happen so it will > rarely trigger, or its potentially informative. ACK otherwise I've preferred to just mute it as the error was that the location could not be opened, and that makes sense when we pass a URL. Should this be changed? Thanks, Giuseppe _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
