Alon Bar-Lev has posted comments on this change. Change subject: aaa: Remove dependency at builtin on Common config ......................................................................
Patch Set 11: (2 comments) http://gerrit.ovirt.org/#/c/27607/11/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/Utils.java File backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/Utils.java: Line 16: Line 17: public static void setDefaults(Properties conf) { Line 18: { Line 19: Properties defaults = new Properties(); Line 20: try (Reader reader = new InputStreamReader(Utils.class.getResourceAsStream("defaults.properties"), Charset.forName("UTF-8"))) { you should also close the input stream, no? Line 21: defaults.load(reader); Line 22: for (Map.Entry<Object, Object> entry : defaults.entrySet()) { Line 23: putIfAbsent(conf, (String) entry.getKey(), (String) entry.getValue()); Line 24: } http://gerrit.ovirt.org/#/c/27607/11/packaging/services/ovirt-engine/ovirt-engine.conf.in File packaging/services/ovirt-engine/ovirt-engine.conf.in: Line 100: # Line 101: # -Dmy.param=my.value -Dmy.flag -Dyour.param=your.value Line 102: # Line 103: ENGINE_PROPERTIES="" Line 104: ENGINE_PROPERTIES="${ENGINE_PROPERTIES} jsse.enableSNIExtension=false java.security.krb5.conf=\"${ENGINE_ETC}/krb5.conf\"" please see my previous comment Line 105: Line 106: # Line 107: # Extra Java arguments to be added to command-line. Line 108: # -- To view, visit http://gerrit.ovirt.org/27607 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1384a99f73ab605b61bce8dcdfd63e222b0001fa Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
