[node-patches] Change in ovirt-node[master]: Stop doing circular imports in process.check_output

fabiand at fedoraproject.org fabiand at fedoraproject.org
Fri Jun 28 12:14:00 UTC 2013


Fabian Deutsch has posted comments on this change.

Change subject: Stop doing circular imports in process.check_output
......................................................................


Patch Set 1: I would prefer that you didn't submit this

(1 inline comment)

I see the problem, but I'd rather like to solve it differently, so without a function level import

....................................................
File src/ovirt-crash.log
Line 1: 7[?47h[?1000h[?1002h)0[?25l oVirt None-None┌───────────────────────────────────────┐                                                                                                                                             │Status                                 │       System Information                                                                                                                    │Network                                │                                                                                                                                             │!
 [0;30;47mSecurity                               │     Status:       No virtualization hardware was detected on this system                                                                    │Keyboard                               │                                                                                                                                             │SNMP                                   │     Networking:   Connected                                           eth0                                                                  │CIM                                    │                                                                       IPv4: 192.168.0.165              !
                                      │Logging                                │                                                                       IPv6: fe80::21a:4aff:fea8:74                                          │Kdump                                  │                                                                                                                                             │Remote Storage                         │     Logs:         Local Only                                                                                                                │Monitoring                             │                                                                                                                 !
                             │Diagnostics                            │     Running VMs:  0                                                                                                                         │Performance                            │                                                                                                                                             │Puppet                                 │     Press F8 for support menu                                                                                                               │Plugins                                │                                                             !
                                                                                 │                                       │     < View Host Key >                                                 < View CPU Details >                                                  │                                       │     < Lock >                         < Log Off >                      < Restart >                      < Power Off >                        │                                       │                                                                                                                                             │                                       │  !
                                                                                                                                            │                                       │                                                                                                                                             │                                       │                                                                                                                                             │                                       │                                                                                                                                             │                        !
                │                                                                                                                                             │                                       │                                                                                                                                             │                                       │                                                                                                                                             │                                       │                                                                                                                                             │                                       │[0!
 ;39;47m                                                                                                                                             │                                       │                                                                                                                                             │                                       │                                                                                                                                             │                                       │                                                                                                                                             │               !
                         │                                                                                                                                             │                                       │                                                                                                                                             │                                       │                                                                                                                                             │                                       │                                                                                                                                             │                                       [0;30;!
 47m│                                                                                                                                             │                                       │                                                                                                                                             │                                       │                                                                                                                                             │                                       │                                                                                                                                             │      !
                                  │                                                                                                                                             │                                       │                                                                                                                                             │                                       │                                                                                                                                             │                                       │                                                                                                                                             │                                     !
   │                                                                                                                                             └───────────────────────────────────────┘                                                                                                                                             Press esc to quit.[?1002l[?1000l[?25h[?47l8
please drop this file
Line 2: [?25hTraceback (most recent call last):
Line 3:   File "/usr/lib64/python2.6/runpy.py", line 122, in _run_module_as_main
Line 4:     "__main__", fname, loader, pkg_name)
Line 5:   File "/usr/lib64/python2.6/runpy.py", line 34, in _run_code


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic779d49b0c1e46f416a5c62fc707a10bb5fb9baf
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Michael Burns <mburns at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the node-patches mailing list