Skip to content
This repository has been archived by the owner on Jul 18, 2018. It is now read-only.

name k2 container based on deployment.cluster #10

Merged
merged 1 commit into from
Nov 15, 2016
Merged

name k2 container based on deployment.cluster #10

merged 1 commit into from
Nov 15, 2016

Conversation

spiffxp
Copy link
Contributor

@spiffxp spiffxp commented Nov 14, 2016

at present since container names are randomly generated, and k2cli times out, it's possible for me to have two containers running a cluster up and a cluster down simultaneously; this should make those cases collide

also this makes it easier to script / grep for which container's logs to follow

@maratoid
Copy link
Contributor

lgtm

@maratoid maratoid merged commit 5bb8fbf into samsung-cnct:master Nov 15, 2016
@spiffxp spiffxp deleted the name-k2-container branch November 15, 2016 18:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants