-
Notifications
You must be signed in to change notification settings - Fork 16.8k
Conversation
Helps the user backup mysql databases to a persistent volume
Please use the |
/assign |
thanks @unguiculus I actually skipped setting an app label as I wasn't sure if it was appropriate due to this being purely a job/cronjob. I'll go back and add it in. |
labels: | ||
chart: "{{ .Chart.Name }}-{{ .Chart.Version }}" | ||
release: "{{ .Release.Name }}" | ||
heritage: "{{ .Release.Service }}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You missed the app label here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤦♂️ fixing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you think a lot of people need this job? If we get rid of the job, we can simplify the chart.
What do you think of adding this cronjob to mysql chart instead? For me it would make sense.
For a future iteration, we could think of sending the backup to s3 too.
@@ -0,0 +1,33 @@ | |||
spec: | |||
containers: | |||
- name: xtrabackup |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you call it xtrabackup? It doesn't look like related to percona?
Any news on this? |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Activing will cause the issue to no longer be considered stale. Thank you for your contributions. |
/retest |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: paulczar, unguiculus The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
new chart stable/mysqldump (helm#4069) * new chart stable/mysqldump Helps the user backup mysql databases to a persistent volume * use chart.name from tpl, add app label * add app/chart fixes to mysqldump-cron.yaml * fix backoffLimit in wrong place. * exit gracefully if db host not specified Merge with charts/stable
* [stable/cockroachdb] Added support for node selector * Merge with charts/stable * [stable/cockroachdb] Added support for node selector * [stable/cockroachdb] Added support for node selector new chart stable/mysqldump (#4069) * new chart stable/mysqldump Helps the user backup mysql databases to a persistent volume * use chart.name from tpl, add app label * add app/chart fixes to mysqldump-cron.yaml * fix backoffLimit in wrong place. * exit gracefully if db host not specified Merge with charts/stable * Chart version bump
* new chart stable/mysqldump Helps the user backup mysql databases to a persistent volume * use chart.name from tpl, add app label * add app/chart fixes to mysqldump-cron.yaml * fix backoffLimit in wrong place. * exit gracefully if db host not specified Signed-off-by: Marek Bartik <mab@revolgy.com> Signed-off-by: Marek Bartik <bartimar6@gmail.com>
* [stable/cockroachdb] Added support for node selector * Merge with charts/stable * [stable/cockroachdb] Added support for node selector * [stable/cockroachdb] Added support for node selector new chart stable/mysqldump (helm#4069) * new chart stable/mysqldump Helps the user backup mysql databases to a persistent volume * use chart.name from tpl, add app label * add app/chart fixes to mysqldump-cron.yaml * fix backoffLimit in wrong place. * exit gracefully if db host not specified Merge with charts/stable * Chart version bump Signed-off-by: Marek Bartik <mab@revolgy.com> Signed-off-by: Marek Bartik <bartimar6@gmail.com>
* [stable/cockroachdb] Added support for node selector * Merge with charts/stable * [stable/cockroachdb] Added support for node selector * [stable/cockroachdb] Added support for node selector new chart stable/mysqldump (helm#4069) * new chart stable/mysqldump Helps the user backup mysql databases to a persistent volume * use chart.name from tpl, add app label * add app/chart fixes to mysqldump-cron.yaml * fix backoffLimit in wrong place. * exit gracefully if db host not specified Merge with charts/stable * Chart version bump * Added external and internal GRPC port as options. Also added options to customize service annotations. Need these for istio config. * Updated README to include values I added. Reformatted the table so it reads pretty in plaintext * cockroach chart version bump for istio work Signed-off-by: Marek Bartik <mab@revolgy.com> Signed-off-by: Marek Bartik <bartimar6@gmail.com>
* new chart stable/mysqldump Helps the user backup mysql databases to a persistent volume * use chart.name from tpl, add app label * add app/chart fixes to mysqldump-cron.yaml * fix backoffLimit in wrong place. * exit gracefully if db host not specified Signed-off-by: aba182 <ajwilhel@gmail.com>
* [stable/cockroachdb] Added support for node selector * Merge with charts/stable * [stable/cockroachdb] Added support for node selector * [stable/cockroachdb] Added support for node selector new chart stable/mysqldump (helm#4069) * new chart stable/mysqldump Helps the user backup mysql databases to a persistent volume * use chart.name from tpl, add app label * add app/chart fixes to mysqldump-cron.yaml * fix backoffLimit in wrong place. * exit gracefully if db host not specified Merge with charts/stable * Chart version bump Signed-off-by: aba182 <ajwilhel@gmail.com>
* [stable/cockroachdb] Added support for node selector * Merge with charts/stable * [stable/cockroachdb] Added support for node selector * [stable/cockroachdb] Added support for node selector new chart stable/mysqldump (helm#4069) * new chart stable/mysqldump Helps the user backup mysql databases to a persistent volume * use chart.name from tpl, add app label * add app/chart fixes to mysqldump-cron.yaml * fix backoffLimit in wrong place. * exit gracefully if db host not specified Merge with charts/stable * Chart version bump * Added external and internal GRPC port as options. Also added options to customize service annotations. Need these for istio config. * Updated README to include values I added. Reformatted the table so it reads pretty in plaintext * cockroach chart version bump for istio work Signed-off-by: aba182 <ajwilhel@gmail.com>
* new chart stable/mysqldump Helps the user backup mysql databases to a persistent volume * use chart.name from tpl, add app label * add app/chart fixes to mysqldump-cron.yaml * fix backoffLimit in wrong place. * exit gracefully if db host not specified Signed-off-by: aba182 <ajwilhel@gmail.com>
* [stable/cockroachdb] Added support for node selector * Merge with charts/stable * [stable/cockroachdb] Added support for node selector * [stable/cockroachdb] Added support for node selector new chart stable/mysqldump (helm#4069) * new chart stable/mysqldump Helps the user backup mysql databases to a persistent volume * use chart.name from tpl, add app label * add app/chart fixes to mysqldump-cron.yaml * fix backoffLimit in wrong place. * exit gracefully if db host not specified Merge with charts/stable * Chart version bump Signed-off-by: aba182 <ajwilhel@gmail.com>
* [stable/cockroachdb] Added support for node selector * Merge with charts/stable * [stable/cockroachdb] Added support for node selector * [stable/cockroachdb] Added support for node selector new chart stable/mysqldump (helm#4069) * new chart stable/mysqldump Helps the user backup mysql databases to a persistent volume * use chart.name from tpl, add app label * add app/chart fixes to mysqldump-cron.yaml * fix backoffLimit in wrong place. * exit gracefully if db host not specified Merge with charts/stable * Chart version bump * Added external and internal GRPC port as options. Also added options to customize service annotations. Need these for istio config. * Updated README to include values I added. Reformatted the table so it reads pretty in plaintext * cockroach chart version bump for istio work Signed-off-by: aba182 <ajwilhel@gmail.com>
* new chart stable/mysqldump Helps the user backup mysql databases to a persistent volume * use chart.name from tpl, add app label * add app/chart fixes to mysqldump-cron.yaml * fix backoffLimit in wrong place. * exit gracefully if db host not specified Signed-off-by: Jakob Niggel <info@jakobniggel.de>
* [stable/cockroachdb] Added support for node selector * Merge with charts/stable * [stable/cockroachdb] Added support for node selector * [stable/cockroachdb] Added support for node selector new chart stable/mysqldump (helm#4069) * new chart stable/mysqldump Helps the user backup mysql databases to a persistent volume * use chart.name from tpl, add app label * add app/chart fixes to mysqldump-cron.yaml * fix backoffLimit in wrong place. * exit gracefully if db host not specified Merge with charts/stable * Chart version bump Signed-off-by: Jakob Niggel <info@jakobniggel.de>
* [stable/cockroachdb] Added support for node selector * Merge with charts/stable * [stable/cockroachdb] Added support for node selector * [stable/cockroachdb] Added support for node selector new chart stable/mysqldump (helm#4069) * new chart stable/mysqldump Helps the user backup mysql databases to a persistent volume * use chart.name from tpl, add app label * add app/chart fixes to mysqldump-cron.yaml * fix backoffLimit in wrong place. * exit gracefully if db host not specified Merge with charts/stable * Chart version bump * Added external and internal GRPC port as options. Also added options to customize service annotations. Need these for istio config. * Updated README to include values I added. Reformatted the table so it reads pretty in plaintext * cockroach chart version bump for istio work Signed-off-by: Jakob Niggel <info@jakobniggel.de>
Helps the user backup mysql databases to a persistent volume