Eli Mesika has uploaded a new change for review.

Change subject: core: Fixing PM options caching
......................................................................

core: Fixing PM options caching

fencingOptionMapping cache map was declared as static
as a result if it was initialized once with the 'general' version
the values of this version was used for all instances no matter which
version is given to the constructor.
This eliminates new agents added in later versions to function and
display supported port/slot/secure fields correctly

Change-Id: I03e4a98919105bbb0d7ef382234b8810db044215
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1040875
Signed-off-by: Eli Mesika <emes...@redhat.com>
---
M 
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/pm/VdsFenceOptions.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/25/22325/1

diff --git 
a/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/pm/VdsFenceOptions.java
 
b/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/pm/VdsFenceOptions.java
index c9c0a00..8d73b67 100644
--- 
a/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/pm/VdsFenceOptions.java
+++ 
b/backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/pm/VdsFenceOptions.java
@@ -30,7 +30,7 @@
     private static final String MAPPING_FORMAT_ERROR = "Illegal fencing 
mapping format {0}";
 
     private static Log log = LogFactory.getLog(VdsFenceOptions.class);
-    private static HashMap<String, HashMap<String, String>> 
fencingOptionMapping;
+    private HashMap<String, HashMap<String, String>> fencingOptionMapping;
     private static HashMap<String, String> fencingOptionTypes;
 
     private String fenceAgent = "";


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I03e4a98919105bbb0d7ef382234b8810db044215
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Eli Mesika <emes...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to