"null" when releasing an IP address is confusing

[Graham Binns]

When you release an IP address you see something like this:

graham@dodger:~/build-area⟫ maas maas ipaddresses release ip=10.0.0.130
null

The least it could do is say "IP address released."

[Gavin Panella (allenap)]

The maas CLI just prints the machine-readable output returned by the API. The CLI is a very thin wrapper around the MAAS API, and I think it's worth making it thicker, by which I mean that the translation from machine-readable to human-readable should be in the CLI. That also makes sense for another reason: the CLI has to be usable for both humans and scripts, so we need both human- and machine-readable available there.

[Graham Binns (gmb)]

On 21 October 2014 12:48, Gavin Panella <email address hidden> wrote:
> The maas CLI just prints the machine-readable output returned by the
> API. The CLI is a very thin wrapper around the MAAS API, and I think
> it's worth making it thicker, by which I mean that the translation from
> machine-readable to human-readable should be in the CLI. That also makes
> sense for another reason: the CLI has to be usable for both humans and
> scripts, so we need both human- and machine-readable available there.

Agreed. In this case, I've just made the offending method return a
meaningful response. The CLI needs a proper bundle of loving,
preferably in the next (i.e. Vivid) cycle.