[node-patches] Change in ovirt-node[ovirt-3.6]: RHN Page should persist conf, not the entire config

fabiand at redhat.com fabiand at redhat.com
Tue Oct 20 15:59:55 UTC 2015


Fabian Deutsch has uploaded a new change for review.

Change subject: RHN Page should persist conf, not the entire config
......................................................................

RHN Page should persist conf, not the entire config

persist() can't persist dicts. Typo in the code. Not sure how it
didn't get caught during testing, since all flows were tested.

Change-Id: I1c7ae56afb42284784d40e5da0a981ddf34c11a6
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1259499
Signed-off-by: Ryan Barry <rbarry at redhat.com>
---
M src/ovirt/node/setup/rhn/rhn_model.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-node refs/changes/33/47533/1

diff --git a/src/ovirt/node/setup/rhn/rhn_model.py b/src/ovirt/node/setup/rhn/rhn_model.py
index 5dc852e..048fe3e 100755
--- a/src/ovirt/node/setup/rhn/rhn_model.py
+++ b/src/ovirt/node/setup/rhn/rhn_model.py
@@ -464,7 +464,7 @@
                                "/var/lib/rhsm/facts/facts.json"]
 
                     for conf in configs:
-                        Config().persist(cfg)
+                        Config().persist(conf)
                         Config().persist("/etc/pki/consumer/key.pem")
                         Config().persist("/etc/pki/consumer/cert.pem")
                         if cfg["url"]:


-- 
To view, visit https://gerrit.ovirt.org/47533
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1c7ae56afb42284784d40e5da0a981ddf34c11a6
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand at redhat.com>



More information about the node-patches mailing list