[node-patches] Change in ovirt-node[master]: ui: Installer

jboggs at redhat.com jboggs at redhat.com
Fri Feb 1 02:44:19 UTC 2013


Joey Boggs has posted comments on this change.

Change subject: ui: Installer
......................................................................


Patch Set 4:

Am I calling it wrong? I'm able to call the new tui without problems, not sure if you've fixed this in a followup patch

[root at localhost ~]# python -m ovirt.node.installer
Traceback (most recent call last):
  File "/usr/lib64/python2.7/runpy.py", line 162, in _run_module_as_main
  File "/usr/lib64/python2.7/runpy.py", line 72, in _run_code
  File "/usr/lib/python2.7/site-packages/ovirt/node/installer/__main__.py", line 31, in <module>
  File "/usr/lib/python2.7/site-packages/ovirt/node/app.py", line 282, in run
  File "/usr/lib/python2.7/site-packages/ovirt/node/ui/urwid_builder.py", line 400, in run
  File "/usr/lib/python2.7/site-packages/ovirt/node/ui/__init__.py", line 568, in to_first_plugin
  File "/usr/lib/python2.7/site-packages/ovirt/node/ui/__init__.py", line 553, in to_nth
  File "/usr/lib/python2.7/site-packages/ovirt/node/ui/__init__.py", line 538, in to_plugin
  File "/usr/lib/python2.7/site-packages/ovirt/node/app.py", line 244, in switch_to_plugin
  File "/usr/lib/python2.7/site-packages/ovirt/node/installer/welcome_page.py", line 56, in ui_content
  File "/usr/lib/python2.7/site-packages/ovirt/node/installer/welcome_page.py", line 102, in ___installation_option
  File "/usr/lib/python2.7/site-packages/ovirt/node/utils/system.py", line 90, in __str__
TypeError: 'str' object is not callable

--
To view, visit http://gerrit.ovirt.org/11066
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0c81e0e85ff35ee073dd5e47df9f53da0a8f7738
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Joey Boggs <jboggs at redhat.com>



More information about the node-patches mailing list