Content parameter for commissioning-scripts and node-scripts is not helpful

Bug #1932554 reported by Sandor Zeestraten
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
MAAS
Triaged
Low
Bill Wear
3.3
Triaged
Low
Bill Wear

Bug Description

The docs on how to add commissioning scripts or node scripts via the CLI are wrong.

`maas <profile> commissioning-scripts create --help` and `maas <profile> node-scripts create --help` say to use the `content` and `script` parameter but not how. It is not clear that this should be the path to script.

The help text also does not say that the parameter requires an @ sign (e.g `content@=<path>`) so it does not work as it is documented today.

I recommend changing the parameter name to path and documenting the @ sign.

Source code:
* https://github.com/maas/maas/blob/31a95206afe563012c48a20bcd8492fa32c7f405/src/maasserver/api/commissioning_scripts.py#L84-L86
* https://github.com/maas/maas/blob/31a95206afe563012c48a20bcd8492fa32c7f405/src/maasserver/api/scripts.py#L80-L84
* https://github.com/maas/maas/blob/31a95206afe563012c48a20bcd8492fa32c7f405/src/maasserver/api/scripts.py#L80-L84

Please also update the docs: https://maas.io/docs/cli-testing-scripts

Tags: doc
Alberto Donato (ack)
Changed in maas:
status: New → Triaged
importance: Undecided → Medium
milestone: none → next
tags: added: doc
summary: - [doc] [cli] content parameter for commissioning-scripts and node-scripts
- is not helpful
+ Content parameter for commissioning-scripts and node-scripts is not
+ helpful
Revision history for this message
Jerzy Husakowski (jhusakowski) wrote :

We will improve the https://maas.io/docs/cli-testing-scripts contents.

The way that MAAS API and CLI documentation is created is contributing to this confusion. We are planning to revise the doc generation mechanism and the way that API is exposed, which should make it easier to improve the documentation as well. Until these improvements are in place, we are not able to meaningfully remove this type of confusion.

Changed in maas:
assignee: nobody → Bill Wear (billwear)
importance: Medium → Low
milestone: next → 3.3.0
Changed in maas:
milestone: 3.3.0 → 3.4.0
Alberto Donato (ack)
Changed in maas:
milestone: 3.4.0 → 3.4.x
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.