alces-software / adminware

A sandbox CLI for running commands remotely across nodes
1 stars 0 forks source link

Feat/list tools #99

Closed DavidMarchant closed 6 years ago

DavidMarchant commented 6 years ago

Fixes #79 when merged ~Based off #98~ Replaces #98 Implements batch avail and batch avail-families commands. The former takes a namespace (offset from batch) and displays all commands in & subdirectories of that namespace. If the argument is blank it treats the namespace as batch.

adminware> batch avail

test_cmd3 -- testing help 123

pgrep -- show program status

test_cmd5 -- testing help 456

test_cmd6 -- another test command's help

test_cmd10 -- test_cmd10's help string 

depth_test -- see 'batch avail depth_test'

deeper_test -- see 'batch avail deeper_test'

adminware> batch avail deeper_test

test_cmd9 -- test_cmd9's help string

deeper_test2 -- see 'batch avail deeper_test/deeper_test2'

deeper_test3 -- see 'batch avail deeper_test/deeper_test3'

The latter shows all families.

adminware> batch avail-families

family1
test_cmd3 --> test_cmd8 --> test_cmd11

family2
test_cmd3 --> test_cmd5 --> test_cmd6 --> test_cmd9 --> test_cmd10

Currently there is no click autocomplete for the arguments to batch avail, how essential is this?

EDIT: Additionally with the closure of #98 this PR also contains all refactoring of the now outdated action.py.

DavidMarchant commented 6 years ago

With the previous 4 commits this PR makes redundant #98, which shall be closed shortly. This PR now additionally fixes #93

DavidMarchant commented 6 years ago

Those changes have been made - apologies again for shoving so much stuff into one PR, it was symptomatic of having to make so many changes based on yet un-merged code

WilliamMcCumstie commented 6 years ago

Responded to your comments, once they have been addressed this is good to merge