Skip to content

Update Helm Chart#3582

Merged
enisoc merged 5 commits intomasterfrom
helm
Jan 23, 2018
Merged

Update Helm Chart#3582
enisoc merged 5 commits intomasterfrom
helm

Conversation

@enisoc
Copy link
Copy Markdown
Member

@enisoc enisoc commented Jan 23, 2018

This proposes to merge #3487 into master, which was previously merged into a temporary branch to let me add tweaks.

@derekperkins

derekperkins and others added 5 commits January 22, 2018 16:12
This needs to be added to the Docker Hub auto build process
- Allows for custom database images
- Supports MySQL protocol
- Upgrade to etcd2 topology server
- Uses etcd-operator
- Default pod affinity/anti-affinity
- HPA for vtgates
- Pod security context doesn’t allow root privileges
- Supports default credential chains for GCS/S3
- Moves backup and other config to ConfigMap
- Temporarily removes Orchestrator support
- Vitess components log to stdout/stderr only
- MySQL slow/error logs redirected to stdout
- Removed syslog and hostPath mounts
- Uses new vitess:k8s Docker image
- Support regional failure domain affinity
@googlebot
Copy link
Copy Markdown

So there's good news and bad news.

👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there.

😕 The bad news is that it appears that one or more commits were authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that here in the pull request.

Note to project maintainer: This is a terminal state, meaning the cla/google commit status will not change from this State. It's up to you to confirm consent of the commit author(s) and merge this pull request when appropriate.

@derekperkins
Copy link
Copy Markdown
Member

LGTM

@enisoc enisoc merged commit 5a7f4df into master Jan 23, 2018
@enisoc enisoc deleted the helm branch January 23, 2018 21:33
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.

3 participants