KNOX-2940 - knoxcli create-alias/create-aliases command doesn't support values starting with dash #778

zeroflag · 2023-07-21T12:46:58Z

What changes were proposed in this pull request?

Modified a check for the create-alias/create-aliases commands to support values with leading dash.

missing value:

 bin/knoxcli.sh  create-alias a1 --value

value with dashes

bin/knoxcli.sh  create-alias a1 --value ---content---

Incorrect usage

bin/knoxcli.sh  create-alias a1 --value --generate

bin/knoxcli.sh  create-alias a1 --value --cluster

Batch alias with dashes

bin/knoxcli.sh  create-aliases --alias --a1-- --value v1 --alias a2 --value --v2--

Batch alias missing value

bin/knoxcli.sh  create-aliases --alias --a1-- --value

Batch alias incorrect usage

bin/knoxcli.sh  create-aliases --alias --a1-- --value --generate
bin/knoxcli.sh  create-aliases --alias --a1-- --value --cluster

…rt values starting with dash

smolnar82

LGTM

KNOX-2940 - knoxcli create-alias/create-aliases command doesn't suppo…

90ee250

…rt values starting with dash

zeroflag self-assigned this Jul 21, 2023

zeroflag requested review from smolnar82, pzampino and moresandeep July 21, 2023 13:07

KNOX-2940 - knoxcli create-alias/create-aliases command doesn't suppo…

955069d

…rt values starting with dash

smolnar82 approved these changes Jul 23, 2023

View reviewed changes

zeroflag merged commit 6eac567 into apache:master Jul 31, 2023
2 checks passed