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

[rbarry at redhat.com]

Ryan Barry 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/40/47140/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/47140
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1c7ae56afb42284784d40e5da0a981ddf34c11a6
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-node
Gerrit-Branch: master
Gerrit-Owner: Ryan Barry <rbarry at redhat.com>