[node-patches] Change in ovirt-node[master]: ovirt_node: substitute remove_config usage with Config.unper...

[dougsland at redhat.com]

Douglas Schilling Landgraf has posted comments on this change.

Change subject: ovirt_node: substitute remove_config usage with Config.unpersist
......................................................................


Patch Set 3:

humm, looks like jenkins job cannot find the /etc/default/ovirt. Is it a problem in the job?

09:04:58 Traceback (most recent call last):
09:04:58   File "../src/autotest.py", line 55, in <module>
09:04:58     sys.exit(test())
09:04:58   File "../src/autotest.py", line 42, in test
09:04:58     test =  __import__(mod, globals(), locals(), ['object'], -1)
09:04:58   File "/home/jenkins/workspace/ovirt-node_master_check-local_gerrit at 2/src/ovirt/node/setup/rhn/__init__.py", line 22, in <module>
09:04:58     import rhn_page
09:04:58   File "/home/jenkins/workspace/ovirt-node_master_check-local_gerrit at 2/src/ovirt/node/setup/rhn/rhn_page.py", line 22, in <module>
09:04:58     import rhn_model
09:04:58   File "/home/jenkins/workspace/ovirt-node_master_check-local_gerrit at 2/src/ovirt/node/setup/rhn/rhn_model.py", line 25, in <module>
09:04:58     from ovirtnode.ovirtfunctions import unmount_config
09:04:58   File "/home/jenkins/workspace/ovirt-node_master_check-local_gerrit at 2/src/ovirtnode/ovirtfunctions.py", line 1939, in <module>
09:04:58     OVIRT_VARS = parse_defaults()
09:04:58   File "/home/jenkins/workspace/ovirt-node_master_check-local_gerrit at 2/src/ovirtnode/ovirtfunctions.py", line 92, in parse_defaults
09:04:58     f = open(OVIRT_DEFAULTS, 'r')
09:04:58 IOError: [Errno 2] No such file or directory: '/etc/default/ovirt'
09:04:58 make: *** [ovirt/node/setup/rhn/rhn_model.py.doctest] Error 1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I99bd4af6fc21247560f4667e43539d65ebe6a2e4
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at fedoraproject.org>
Gerrit-Reviewer: Ryan Barry <rbarry at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No