Skip to content

helm: minor fixes#3875

Merged
sougou merged 1 commit intovitessio:masterfrom
sougou:helm
May 2, 2018
Merged

helm: minor fixes#3875
sougou merged 1 commit intovitessio:masterfrom
sougou:helm

Conversation

@sougou
Copy link
Copy Markdown
Contributor

@sougou sougou commented Apr 29, 2018

Couple of invocations were using keyspaceClean instead of
keyspace.name, which is "a-b" vs "a_b".

Signed-off-by: Sugu Sougoumarane ssougou@gmail.com

@sougou sougou requested a review from enisoc April 29, 2018 13:13
@sougou
Copy link
Copy Markdown
Contributor Author

sougou commented Apr 29, 2018

@derekperkins

@derekperkins
Copy link
Copy Markdown
Member

And the cell local etcd will pick up the changes from registering it with the global server?

@sougou
Copy link
Copy Markdown
Contributor Author

sougou commented Apr 29, 2018

The local cell still uses the global zookeeper. I didn't make the fix because I haven't tested it yet.

Couple of invocations were using keyspaceClean instead of
keyspace.name, which is "a-b" vs "a_b".

Signed-off-by: Sugu Sougoumarane <ssougou@gmail.com>
@derekperkins
Copy link
Copy Markdown
Member

LGTM

@sougou sougou merged commit a755692 into vitessio:master May 2, 2018
@sougou sougou deleted the helm branch May 18, 2018 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants