canonical / jhack

Chock-full of Juju hackery.
Apache License 2.0
51 stars 24 forks source link

jhack fire is missing `--model` #45

Closed rgildein closed 1 year ago

rgildein commented 1 year ago

I think it would be beneficial to have a "--model" option to target a specific model in fire command.

PietroPasotti commented 1 year ago

The reason why I didn't include that in the first place is that I was assuming that you'd always be firing events in the current model -- so you can see what's happening in the status and log. Or do you often find yourself working on multiple models simultaneously?

rgildein commented 1 year ago

Not really often, but sometimes yes. Good example would be, when I need cross model relations.

PietroPasotti commented 1 year ago

Fair point. added -m to fire and tail in #49