Juju run-action should not accept parameters not defined in actions.yaml

Bug #1989984 reported by Erik Lönroth
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Triaged
Wishlist
Unassigned

Bug Description

I have a charm that defines an action (add-peer) with parameters, defined as expected in actions.yaml

When I accidentally called my action with an undefined parameter, it was still passed to the charm code.

This caused issues, since I assumed juju validated that up before passing it to my action.

So, when I ran my code - I ended up with parameters which was undefined but found their way into my code here:

https://github.com/erik78se/charm-wireguard-sub/blob/86d4405872b516e70d71181ae11939989ed5d906/wireguard-sub/src/charm.py#L247

Should juju to accept un-named parameters when calling actions, or should I make this enforced somewhere in actions.yaml ?

Changed in juju:
status: New → Triaged
importance: Undecided → Wishlist
milestone: none → 3.1.0
milestone: 3.1.0 → none
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.