Hooks Logging

Steffen Gebert

unread,

Dec 10, 2017, 10:34:51 AM12/10/17

to Repo and Gerrit Discussion

Hi,

we've updated our Gerrit installation to 2.14.

The hooks plugin is installed, used with all default values and hooks _are_ triggered.

But I miss the log output created by the hooks. Any pointer, where this should be visible?

Thanks
Steffen

David Pursehouse

unread,

Dec 10, 2017, 6:43:09 PM12/10/17

to Steffen Gebert, Repo and Gerrit Discussion

Since [1] the output is logged at debug level. The documentation added in [2] explains how to enable it.

[1] https://gerrit-review.googlesource.com/c/plugins/hooks/+/98270

[2] https://gerrit-review.googlesource.com/c/plugins/hooks/+/128770

Thanks
Steffen

--
--
To unsubscribe, email repo-discuss...@googlegroups.com
More info at http://groups.google.com/group/repo-discuss?hl=en

---
You received this message because you are subscribed to the Google Groups "Repo and Gerrit Discussion" group.
To unsubscribe from this group and stop receiving emails from it, send an email to repo-discuss...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Steffen Gebert

unread,

Dec 11, 2017, 1:05:42 AM12/11/17

to David Pursehouse, Repo and Gerrit Discussion

Thanks David!

I should have looked at the documentation for master and not only for stable-2.14 as it seems :-)

Best
Steffen

Matthias Sohn

unread,

Dec 11, 2017, 11:15:56 AM12/11/17

to Steffen Gebert, David Pursehouse, Repo and Gerrit Discussion

> On 11. Dec 2017, at 00:42, David Pursehouse <david.pu...@gmail.com> wrote:
>
> On Mon, Dec 11, 2017 at 12:34 AM Steffen Gebert <ste...@steffen-gebert.de> wrote:
> Hi,
>
> we've updated our Gerrit installation to 2.14.
>
> The hooks plugin is installed, used with all default values and hooks _are_ triggered.
>
> But I miss the log output created by the hooks. Any pointer, where this should be visible?
>
>
> Since [1] the output is logged at debug level. The documentation added in [2] explains how to enable it.
>
> [1] https://gerrit-review.googlesource.com/c/plugins/hooks/+/98270
> [2] https://gerrit-review.googlesource.com/c/plugins/hooks/+/128770
>

On Mon, Dec 11, 2017 at 7:05 AM, Steffen Gebert <steffen@steffen-gebert.de> wrote:
Thanks David!

I should have looked at the documentation for master and not only for stable-2.14 as it seems :-)

Best
Steffen

looks like [2] should be backported to stable-2.14 to ensure the documentation matches the behavior.

-Matthias

David Pursehouse

unread,

Dec 11, 2017, 8:50:04 PM12/11/17

to Matthias Sohn, Steffen Gebert, Repo and Gerrit Discussion

Agreed, and done in [3] along with another documentation cherry-pick.

[3] https://gerrit-review.googlesource.com/c/plugins/hooks/+/147531

-Matthias

Matthias Sohn

unread,

Dec 12, 2017, 4:56:51 AM12/12/17

to David Pursehouse, Steffen Gebert, Repo and Gerrit Discussion

thanks

Reply all

Reply to author

Forward