Alon Bar-Lev has posted comments on this change. Change subject: tools: Adds notifier config validation prior to service startup ......................................................................
Patch Set 6: I think we can do this better and allow print to user the message so we have this both in sysv and systemd. systemd get this as we print it to logger. sysv can have this if we print these validation errors into process stderr, no? So if we print validation errors into stderr we should have this in systemd journal and in sysv console output, no? -- To view, visit http://gerrit.ovirt.org/22629 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I11ad77a33697f254e60e1d0b12e343900e5b0b34 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Martin Peřina <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: Yair Zaslavsky <[email protected]> Gerrit-Reviewer: mooli tayer <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
