55 Commits (ae6c88636442980b2e26b999c865abe172674fb6)

Author	SHA1	Message	Date
Andy Zhou	a0eb69b73e	Allow the volume argument to be specified multiple times ... 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.	5 years ago
iwilltry42	6823470ef5	don't allow empty node-specifiers	5 years ago
iwilltry42	8512ac0c8a	make server the default node for portmaps	5 years ago
iwilltry42	2640fc13ce	make server the default node for portmaps	5 years ago
iwilltry42	6d0fc76b22	improve portspec and ndoespec validation	5 years ago
iwilltry42	464d2e23c5	Offset multiplication	5 years ago
iwilltry42	107700dc98	update log messages	5 years ago
iwilltry42	a68d9e7042	generate container names in a func and use it to determine if node-specifier is valid	5 years ago
Andy Zhou	4674a5d6c8	Improve cluster status display ... Before this change, k3d list stopped cluster as "exited", This patch change it to "stopped". $ k3d create -n test $ k3d stop -n test $ k3d list --all +------+------------------------------+--------+---------+ | NAME | IMAGE | STATUS | WORKERS | +------+------------------------------+--------+---------+ | test | docker.io/rancher/k3s:v0.5.0 | exited | 0/0 | +------+------------------------------+--------+---------+ Before this change, k3d report the server status as the cluster status. This commit output the server status if the server and all worker containers status agree. Otherwise, the cluster status is reported as "unhealthy". $ k3d create --workers 2 $ docker ps -a d14135f5929c rancher/k3s:v0.5.0 "/bin/k3s agent" 6 seconds ago Up 5 seconds k3d-k3s-default-worker-1 612d71f3ec23 rancher/k3s:v0.5.0 "/bin/k3s agent" 6 seconds ago Up 5 seconds k3d-k3s-default-worker-0 7f201731bf45 rancher/k3s:v0.5.0 "/bin/k3s server --h…" 7 seconds ago Up 6 seconds 0.0.0.0:6443->6443/tcp k3d-k3s-default-server $ docker stop d14135f5929c $ bin/k3d list --all +-------------+------------------------------+-----------+---------+ | NAME | IMAGE | STATUS | WORKERS | +-------------+------------------------------+-----------+---------+ | k3s-default | docker.io/rancher/k3s:v0.5.0 | unhealthy | 1/2 | +-------------+------------------------------+-----------+---------+	5 years ago
Andy Zhou	6292c89ec4	Minor simplification of getCluster() ... This make the code easier to read.	5 years ago
Andy Zhou	953578ed11	Avoid output an empty table when no cluster to list ... Before this patch: $ k3d create $ k3d stop $ k3d list +------+-------+--------+---------+ | NAME | IMAGE | STATUS | WORKERS | +------+-------+--------+---------+ +------+-------+--------+---------+ This commit omits such empty table.	5 years ago
iwilltry42	a0c0c3ff9d	add --api-port and --port-auto-offset	5 years ago
iwilltry42	7d6ae201e9	make node-specifier mandatory	5 years ago
iwilltry42	0670a0e11b	resovle conflicts	5 years ago
iwilltry42	a881a87ef6	go fmt	5 years ago
Andy Zhou	2f07bdb78a	Center align list output ... Make list output center aligne. It is slightly eaier to read. Before the commit: $ bin/k3d list --all +-------------+------------------------------+---------+---------+ | NAME | IMAGE | STATUS | WORKERS | +-------------+------------------------------+---------+---------+ | k3s-default | docker.io/rancher/k3s:v0.5.0 | running | 2/2 | +-------------+------------------------------+---------+---------+ After this commit: $ bin/k3d list --all +-------------+------------------------------+---------+---------+ | NAME | IMAGE | STATUS | WORKERS | +-------------+------------------------------+---------+---------+ | k3s-default | docker.io/rancher/k3s:v0.5.0 | running | 2/2 | +-------------+------------------------------+---------+---------+ In this output, only the last cell shows up differently.	5 years ago
Rishabh Gupta	787d6608ba	Delete a cluster if it is not started due to port conflicts or any other unforseen error ... Signed-off-by: Rishabh Gupta <r.g.gupta@outlook.com>	5 years ago
iwilltry42	fd720b0887	move hostname check to util	5 years ago
Andy Zhou	93f72e672a	Make sure the culster name is valid before creating it. ... MAke sure the cluster name is a RFC 1123 compliant host name, since the cluster name is used as base for expanding into a auto generated host names for k3s nodes.	5 years ago
iwilltry42	124a45688f	working version	5 years ago
iwilltry42	d35f12a77e	init	5 years ago
Andy Zhou	65f2820b3f	Support publishing ports for the worker nodes ... All ports exposed by --publish will also be exported for all worker nodes. The host port will be auto indexed based worker id. For example: with the following command option: k3d create --publish 80:80 --publish 90:90/udp --workers 1 The exposed ports will be: host TCP port 80 -> k3s server TCP 80 host TCP port 90 -> k3s server TCP 90 host UDP port 81 -> k3s worker 0 UDP 80 host UDP port 91 -> k3s worker 0 UDP 90	5 years ago
Andy Zhou	988fbdbdc5	Refactor createWorker() API ... Use postfix as int instead string. This make the following patch easier to read.	5 years ago
Andy Zhou	f70a8b42f7	Add support for the --publish option for create subcommand ... Inspired by the docker CLI, --publish take same input as docker CLI and provides similar functions. For the k3s cluster server node, it behaves the same as docker cli; it exports the k3d server ports to the host ports. Handling for worker nodes will be added in the subsequent patches. This option can be used mutiple times for exposing more ports. --add-port is an alias to this option.	5 years ago
Andy Zhou	6fe75640bb	Add the PublishedPorts helping class ... This class holds the parsed results of the --publish options. Its methods helps the create clones of class, with mutations applied. Currently, there are two methods: Offset() change the host ports by a fixed amount. Addport() adds one additional port to the class.	5 years ago
iwilltry42	40fb583160	better image and version flags	5 years ago
iwilltry42	4fbda454f7	remove todo	5 years ago
iwilltry42	9ac81982e2	[Enhancement] set hostnames for nodes	5 years ago
iwilltry42	e78a548da2	[Feature] add --image/-i flag to specify custom k3s image	5 years ago
Darren Shepherd	13aaec64ed	Only create network if not found	5 years ago
Darren Shepherd	7c76f4621a	Always to force delete	5 years ago
Darren Shepherd	c86518e3d4	Only pull image if the image is not found	5 years ago
Andy Zhou	2267924578	cli: Improve error message for 'k3d get-kubeconfig' ... Before this change, command: $ k3d create -name test $ k3d get-kubeconfig Produces the following error message: panic: runtime error: index out of range goroutine 1 [running]: github.com/rancher/k3d/cli.GetKubeConfig(0xc00031a160, 0x0, 0x0) /Users/azhou/projs/k3d/cli/commands.go:335 +0x105d github.com/urfave/cli.HandleAction(0x13e7ca0, 0x148f0b0, 0xc00031a160, 0xc000304000, 0x0) /Users/azhou/projs/rcloud/pkg/mod/github.com/urfave/cli@v1.20.0/app.go:490 +0xc8 github.com/urfave/cli.Command.Run(0x1477cb5, 0xe, 0x0, 0x0, 0x0, 0x0, 0x0, 0x1480cd8, 0x23, 0x0, ...) /Users/azhou/projs/rcloud/pkg/mod/github.com/urfave/cli@v1.20.0/command.go:210 +0x996 github.com/urfave/cli.(*App).Run(0xc0000bc1a0, 0xc00000e060, 0x2, 0x2, 0x0, 0x0) /Users/azhou/projs/rcloud/pkg/mod/github.com/urfave/cli@v1.20.0/app.go:255 +0x6af main.main() /Users/azhou/projs/k3d/main.go:185 +0x11e0 This patch improve the error message by avoid the panic. Now the output becomes: 019/05/02 12:05:30 No server container for cluster k3s_default	5 years ago
iwilltry42	942e667b24	add some comments	5 years ago
iwilltry42	068172aa1a	fix output order for logs	5 years ago
iwilltry42	e9206a74b2	make start/stop functions start/stop workers as well	5 years ago
iwilltry42	9142529c62	get-kubeconfig using docker sdk	5 years ago
iwilltry42	6c072fb6d9	connect workers and server via own docker network	6 years ago
iwilltry42	92039edbb1	fix clusters not being filtered correctly	6 years ago
iwilltry42	7d5964c3b9	fix image not being pulled if reader is not being used	6 years ago
iwilltry42	b0b08d2119	bring back env vars, enable booting workers and base list output on docker data instead of dirs	6 years ago
iwilltry42	7eec5b061b	use docker sdk wherever possible	6 years ago
iwilltry42	65b5f06bfc	add server-arg and env flags	6 years ago
iwilltry42	65c0bf9e5c	better errors	6 years ago
iwilltry42	0b35d7c138	get back --all function for all commands	6 years ago
Rishabh Gupta	29fced4ef7	Added version tags, separated command implementaitons, added wait and timeout ... Signed-off-by: Rishabh Gupta <r.g.gupta@outlook.com>	6 years ago