Action name doesn't allow numeric characters

Bug #1640223 reported by Adam Israel on 2016-11-08
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
juju
Medium
Adrian Garcia Prado

Bug Description

Version affected: Juju 2.0.1

Juju only allows alpha characters in action names, as seen in cmd/juju/action/run.go:

var actionNameRule = regexp.MustCompile("^[a-z](?:[a-z-]*[a-z])?$")

I feel that this is too restrictive, and that the rule should be updated to allow for numeric characters as well.

For example, the utilities 'iperf' and 'iperf3' are distinct applications that may be integrated as an action, but the current name rule prevents that in the latter case.

Changed in juju:
status: New → Triaged
importance: Undecided → High
tags: added: usability
Changed in juju:
milestone: none → 2.1.0-beta2
Curtis Hovey (sinzui) on 2016-12-01
Changed in juju:
milestone: 2.1-beta2 → none
Curtis Hovey (sinzui) on 2016-12-02
Changed in juju:
milestone: none → 2.1-rc1
Changed in juju:
milestone: 2.1-beta4 → 2.2.0
tags: added: actions
Curtis Hovey (sinzui) on 2017-03-24
Changed in juju:
milestone: 2.2-beta1 → 2.2-beta2
Curtis Hovey (sinzui) on 2017-03-30
Changed in juju:
milestone: 2.2-beta2 → 2.2-beta3
Changed in juju:
milestone: 2.2-beta3 → 2.2-beta4
Changed in juju:
milestone: 2.2-beta4 → 2.2-rc1
Tim Penhey (thumper) wrote :

We have a bunch of actions work being done for the start of 2.3. I expect to get this into that hunk of work, but I'm stopping the punting down the releases as that doesn't help anyone.

tags: added: bitesize
Changed in juju:
milestone: 2.2-rc1 → none
importance: High → Medium
Ian Booth (wallyworld) on 2018-01-09
Changed in juju:
milestone: none → 2.4-beta1
assignee: nobody → Adrian Garcia Prado (agprado)
Changed in juju:
status: Triaged → In Progress
Changed in juju:
status: In Progress → Fix Committed
Changed in juju:
status: Fix Committed → In Progress
Changed in juju:
status: In Progress → Fix Committed
Changed in juju:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers