node remove should require a node name

[seanjensengrey]

riak-mesos node remove

will remove riak-default-1 by default when no name is provided.

On subsequent, rexecution

$ riak-mesos node remove
Resource at http://10.1.12.145:31936/api/v1/clusters/default/nodes/riak-default-1 was not found (Status Code: 404)

The command should require an explicit node name, for example

riak-mesos node remove --node riak-cluster-4

[sanmiguel]

You're right, this should be a required argument. In which case, I think we
should promote it to a positional arg, not a switch: riak-mesos node remove riak-default-1

That's 7 less characters to type!

I'm tempted to suggest that this applies to all node commands (except add).

Thoughts?

On Sat, 6 Aug 2016, 13:41 Sean Jensen-Grey, [email protected]
wrote:

riak-mesos node remove

will remove riak-default-1 by default when no name is provided.

On subsequent, rexecution

$ riak-mesos node remove
Resource at http://10.1.12.145:31936/api/v1/clusters/default/nodes/riak-default-1 was not found (Status Code: 404)

The command should require an explicit node name, for example

riak-mesos node remove --node riak-cluster-4

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#31, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ACBkRmKCz3Ka6SDHYZ4xNeVxflEOFiW0ks5qdHLhgaJpZM4JeQ0n
.

[seanjensengrey]

and if we have a bare

riak-mesos node remove

it should actually call

riak-mesos node list along with help output