Bug 1410480
Summary: | "NameError: global name '_' is not defined" appears in rhui.log if there's an issue with Docker | ||
---|---|---|---|
Product: | Red Hat Update Infrastructure for Cloud Providers | Reporter: | Radek Bíba <rbiba> |
Component: | Tools | Assignee: | RHUI Bug List <rhui-bugs> |
Status: | CLOSED ERRATA | QA Contact: | Radek Bíba <rbiba> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | 3.0.0 | CC: | bizhang, pcreech |
Target Milestone: | --- | ||
Target Release: | 3.0.0 | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | If docs needed, set a value | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2017-03-01 22:15:14 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Radek Bíba
2017-01-05 15:31:09 UTC
This is only partially fixed. I see that "from gettext import gettext as _" has been added, which resolves the traceback, but I can still add a nonexisting repo: ------------------------------------------------------------------------------ rhui (repo) => ad Name of the container in the registry: qwe Unique ID for the container (alphanumerics, _, and - only) [qwe]: Display name for the container [qwe]: The following container will be added: Container Id: qwe Display Name: qwe Upstream Container Name: qwe Proceed? (y/n) y Successfully added container qwe ------------------------------------------------------------------------------ Meanwhile, in rhui.log, a correct message appears reflecting the fact that the registry isn't available at 127.0.0.1: 2017-01-18 05:36:46,111 - Search for upstream container failed with message [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:579) Please see the note about "return True" in comment 0. I see a change in the way invalid (non-existing) containers in the real and accessible (and working) registry are handled. Previously, rhui-manager would print: ------------------------------------------------------------------------------ rhui (repo) => ad Name of the container in the registry: qwe Could not find qwe on registry https://registry.access.redhat.com, please make sure that the name is valid Name of the container in the registry: ------------------------------------------------------------------------------ As the repeated prompt suggests, this would allow one to try again with a different (correctly spelled?) name. With ISO 20170118, however, rhui-manager prints: ------------------------------------------------------------------------------ rhui (repo) => ad Name of the container in the registry: qwe Could not validate qwe on registry https://registry.access.redhat.com, this may cause problems during repo creation. Please check ~/.rhui/rhui.log for more details. Unique ID for the container (alphanumerics, _, and - only) [qwe]: ------------------------------------------------------------------------------ Meaning, it's assumed that the entered name is OK but only can't be found/validated at the moment. There are no details in the rhui log, though. Is that the intended fix? OTOH, details are in the rhui.log if there's a network problem, which is good. Radek, yes. This reason for this change is the registry search api issue that happened a couple of weeks ago. After discussion with pcreech we decided that not being able to validate a container name should not block the attempted creation of the container. This ensures that if the search api ever malfunctions again, it does not affect the rhui container creation use case. I see. Thanks for the explanation, Bihan. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHBA-2017:0367 |