Sandro Bonazzola has posted comments on this change. Change subject: Add a controlling TTY for ausearch ......................................................................
Patch Set 1: (1 comment) Thanks for the patch! Just a minor comment about the commit message. Have you seen any side effect introduced by the -t flag? .................................................... Commit Message Line 9: The ausearch command required a controlling TTY in order to finish Line 10: execution propely. This fix adds the '-t' flag to the ssh command Line 11: used in the log-collector Line 12: Line 13: Signed-off-by: James W. Mills <jami...@redhat.com> Bug-Url: https://bugzilla.redhat.com/1010472 -- To view, visit http://gerrit.ovirt.org/19471 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5b7b04c037d44c6d699db670d1d98b0db678fccf Gerrit-PatchSet: 1 Gerrit-Project: ovirt-log-collector Gerrit-Branch: master Gerrit-Owner: James W. Mills <jami...@redhat.com> Gerrit-Reviewer: James W. Mills <jami...@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com> 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