debug-hooks exit 1 , doesn't mark hook as failed

[Jorge Niedbalski]

[Environment]

1.20.14-trusty-amd64

Distributor ID: Ubuntu
Description: Ubuntu 14.04.1 LTS
Release: 14.04
Codename: trusty

[Description]

Set a debug-hooks trace point into any unit

$ juju debug-hooks ceph-radosgw/0 identity-service-relation-changed

 Once jumped into the tmux session, exiting the session with 'exit' or 'exit 1' makes no difference.

According to the current documentation: https://juju.ubuntu.com/docs/authors-hook-debug.html , any exit code != 0
should mark the hook as failed and will require a 'juju resolved' in order to keep the hooks queue execution.

[Curtis Hovey (sinzui)]

I don't think this example is correct for the problem described. Per bug 1221501, juju does not support debug-hooks with the install hook. Is this issue a problem with the other hooks?

[Kapil Thangavelu (hazmat)]

noted a workaround i've been able to use in bug:1415176

On Tue, Jan 27, 2015 at 4:38 PM, Curtis Hovey <email address hidden> wrote:

> ** Changed in: juju-core
> Status: New => Triaged
>
> ** Changed in: juju-core
> Importance: Undecided => Medium
>
> --
> You received this bug notification because you are subscribed to juju-
> core.
> https://bugs.launchpad.net/bugs/1415176
>
> Title:
> debug-hooks exit 1 , doesn't mark hook as failed
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/juju-core/+bug/1415176/+subscriptions
>

[Leonardo Borda (lborda)]

Hi Kapil

Can you describe us the workaround you were able to use it?

Thank Leo

[Kapil Thangavelu (hazmat)]

i found if i closed all the windows in tmux minus the hook window, i could exit 1 from that window and the hook would be marked as failed.

[Kapil Thangavelu (hazmat)]

note this was also added to the original bug report on this issue https://bugs.launchpad.net/juju-core/+bug/1227954 filed in 2013.

[Horacio Durán (hduran-8)]

Just proposed http://reviews.vapour.ws/r/1571/

[Horacio Durán (hduran-8)]

1.24 got merged and http://reviews.vapour.ws/r/1590/ is proposed for master