Ravi Nori has uploaded a new change for review.

Change subject: core: LocalAdminPassword exception after engine restart
......................................................................

core: LocalAdminPassword exception after engine restart

After oVirt host comes up from reboot, we see
javax.crypto.BadPaddingException: Data must start with zero
exception in engine.log:

Change-Id: I000bbb3cf5a6d1d636c49247e85858b02b2f358a
Bug-Url: https://bugzilla.redhat.com/994454
Signed-off-by: Ravi Nori <rn...@redhat.com>
---
M 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/generic/DBConfigUtils.java
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/65/18065/1

diff --git 
a/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/generic/DBConfigUtils.java
 
b/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/generic/DBConfigUtils.java
index 75d7497..e30ed48 100644
--- 
a/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/generic/DBConfigUtils.java
+++ 
b/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/generic/DBConfigUtils.java
@@ -221,7 +221,7 @@
     public <T> T GetValue(ConfigValues name, String version) {
         T returnValue;
         Map<String, Object> values = null;
-        if ((values = _vdcOptionCache.get(name.toString())) != null && 
values.containsKey(version)) {
+        if ((values = _vdcOptionCache.get(name.toString())) != null && 
(values.containsKey(version) || values.containsKey("general"))) {
             returnValue = (T) values.get(version);
         } else {
             VdcOption option = new VdcOption();
@@ -274,6 +274,7 @@
             values = new HashMap<String, Object>();
             _vdcOptionCache.put(option.getoption_name(), values);
         }
+        log.info("In updateOption decrypting and saving option 
"+option.getoption_name()+" version = "+option.getversion());
         values.put(option.getversion(), GetValue(option));
     }
 


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I000bbb3cf5a6d1d636c49247e85858b02b2f358a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rn...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to