New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OS-8498 Want 'piadm destroy' as an alias for 'piadm remove' #1080
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm sorry for not catching this 1st time around. I'm also sorry for foisting the re-sorting of things too.
@@ -1544,7 +1545,7 @@ case $cmd in | |||
list "$@" | |||
;; | |||
|
|||
remove ) | |||
destroy | remove ) | |||
privcheck remove | |||
standalone_only remove |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change standalone_only remove
to standalone_only "$cmd"
please.
@@ -12,7 +12,7 @@ piadm(8) -- Manage SmartOS Platform Images | |||
piadm bootable -e [ -i <source> ] <ZFS-pool-name> | |||
piadm install <source> [ZFS-pool-name] | |||
piadm list [ZFS-pool-name] | |||
piadm remove <PI-stamp> [ZFS-pool-name] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because "destroy" realphabetizes things, you'll have to re-sort this list.
@@ -256,12 +256,16 @@ piadm(8) -- Manage SmartOS Platform Images | |||
Lists the available platform images (and boot images) on bootable | |||
pools. | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same with this section needing to move.
@@ -54,7 +55,7 @@ usage() { | |||
eecho " piadm bootable [-d] [-e [-i <source>]] [-r] [ZFS-pool-name]" | |||
eecho " piadm install <source> [ZFS-pool-name]" | |||
eecho " piadm list <-H> [ZFS-pool-name]" | |||
eecho " piadm remove <PI-stamp> [ZFS-pool-name]" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And same with the help message.
We'd filed a bug because of this PR. I'm trying to link them together now. |
For those used to beadm(8), adds an alias
destroy
forremove
.I've tested it by deleting a PI (verified using
piadm list
before and after deletion.