Skip to content

examples: using planetscale's vitess-operator#6154

Merged
morgo merged 5 commits intovitessio:masterfrom
planetscale:ss-operator-example
May 13, 2020
Merged

examples: using planetscale's vitess-operator#6154
morgo merged 5 commits intovitessio:masterfrom
planetscale:ss-operator-example

Conversation

@sougou
Copy link
Copy Markdown
Contributor

@sougou sougou commented May 6, 2020

This is the vitess example deployed against PlanetScale's operator.

The operator is much faster and feels more stable than the helm charts. Until we find a more committed maintainer for the helm charts, I feel like this should be the recommended way to use vitess on Kubernetes.

@sougou sougou requested review from enisoc and morgo May 6, 2020 20:49
Copy link
Copy Markdown
Collaborator

@deepthi deepthi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work!

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any particular reason to pin the version? Local examples run against latest code/build.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here we seem to always use the latest image.

@derekperkins
Copy link
Copy Markdown
Member

The helm charts should be deprecated, they're not a viable way to run Vitess

Copy link
Copy Markdown
Contributor

@morgo morgo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are files being removed from examples/local in this PR? Is it related?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we clean this up into a README.md before merging?

sougou added 3 commits May 12, 2020 14:38
Signed-off-by: Sugu Sougoumarane <ssougou@gmail.com>
Signed-off-by: Sugu Sougoumarane <ssougou@gmail.com>
Signed-off-by: Sugu Sougoumarane <ssougou@gmail.com>
@sougou sougou force-pushed the ss-operator-example branch from 8ba79c3 to 5e3bd8e Compare May 12, 2020 22:46
Signed-off-by: Sugu Sougoumarane <ssougou@gmail.com>
@sougou sougou force-pushed the ss-operator-example branch from 2980487 to 0ee734e Compare May 12, 2020 23:37
@enisoc
Copy link
Copy Markdown
Member

enisoc commented May 13, 2020

@sougou I've created a tag so we can pin the operator image too now:

planetscale/vitess-operator:v2.0.0

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

@enisoc enisoc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@morgo morgo merged commit 3f68101 into vitessio:master May 13, 2020
@sougou sougou deleted the ss-operator-example branch May 18, 2020 02:48
@deepthi deepthi added this to the v7.0 milestone Jul 17, 2020
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.

5 participants