mirrors/k3d
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Allow the volume argument to be specified multiple times

Browse Source 
Most k3d arguments are using in "stringSlice" style, allowing the
argument to supplied multiple times. Currently "volume" is an exception
to this style, require multiple arguments to be supplied in a single
argument, separated by comma.

This commit improve the k3d usability by improve the consistency of its
argument style.
This commit is contained in:
Andy Zhou 2019-05-14 18:41:27 -07:00
parent 07febe44c0
commit a0eb69b73e
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
cli/commands.go
View File

@ -117,7 +117,7 @@ func CreateCluster(c *cli.Context) error {
k3sServerArgs,
env,
c.String("name"),
strings.Split(c.String("volume"), ","),
c.StringSlice("volume"),
portmap,
)
if err != nil {
@ -184,7 +184,7 @@ func CreateCluster(c *cli.Context) error {
k3sWorkerArgs,
env,
c.String("name"),
strings.Split(c.String("volume"), ","),
c.StringSlice("volume"),
i,
c.String("api-port"),
portmap,

4
main.go
View File

@ -56,9 +56,9 @@ func main() {
Value: defaultK3sClusterName,
Usage: "Set a name for the cluster",
},
cli.StringFlag{
cli.StringSliceFlag{
Name: "volume, v",
Usage: "Mount one or more volumes into every node of the cluster (Docker notation: `source:destination[,source:destination]`)",
Usage: "Mount one or more volumes into every node of the cluster (Docker notation: `source:destination`)",
},
cli.StringSliceFlag{
Name: "publish, add-port",