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=1020344
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/11/22311/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 b94ad40..7fccf7d 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/22311
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I03e4a98919105bbb0d7ef382234b8810db044215
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
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