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