Updating etcd chart with added backup validation function empty implementation(subject for future realization). This has to be done because helm-toolkit chart in openstack-helm-infra is now calling that function verify_databases_backup_archives() as part of backup_databases() function implementation:
https://review.opendev.org/c/openstack/openstack-helm-infra/+/853027
Changed apiVersion of etcd cronjob from batch/v1beta to batch/v1 and fixed securityContext for etcd_backup.
Also bumping up HTK version to 0.2.48 from a commit id obtained from merge of https://review.opendev.org/c/openstack/openstack-helm-infra/+/853027 and set proper commit id in this file: tools/helm_tk.sh
Change-Id: Ie047dd0e6a2aae6483ace89cad22d6720890cdfc
Removing set -x from within the dump_databases_to_directory function.
The set -x from within the function is causing all the code that
follows the function call to have debug tracing on. This in turns
causing multiple identical logs for the same event. Looking at this
function, there should be enough logging to aid debugging.
Reference ps: https://review.opendev.org/c/openstack/openstack-helm-infra/+/830533
(commit 2fc1ce4a142e605a9fc6c90dceabbf7c4bfb81e3)
Change-Id: Id442972bbcca983afab7c4f3c29f3686e9e0b481
Pick up the helm-toolkit DB backup enhancement in etcd
to add capability to retry uploading backup to remote server.
Change-Id: If6ea347a4c2c55f14f35d95681aaf482d0a6103c
1) Uplift helm-toolkit to include db-backup-restore error log string
prefixes for the generation of alert
https://review.opendev.org/c/openstack/openstack-helm-infra/+/823867
2) Error log string prefixes are added to etcd backup-restore as well
Change-Id: Iad51a3e55567d0861140a97c17a1b7d859e13938
Include fix [0] return code when remote rgw fails.
Moving set -x in backup/restore.tpl below the source
of the framework code to reduce debug output.
[0] https://review.opendev.org/#/c/738665/
Change-Id: If9b7b317dff439ecb293d9837cac256884c53c6a
1) Include framework for remote etcd backups.
2) Use porthole etcdctl utility image for backups.
3) Move helm-toolkit pin to latest commit.
4) Add a keystone user for RGW.
5) Add a secret for Swift API access.
6) Add a secret for backup/restore configuration.
Change-Id: Ica549c3b6bc00ca55540b8ffedd4c46af0d8d25e
1. Fix directory listing used to identify newest backup file to be
archived (was sometimes archiving files twice; e.g., a.tar.gz.tar.gz)
2. Fix directory listings used to identify and clean up old backups
Change-Id: Icb1ddd96613f4ab6a28c4f617001c336951568bc
This PS will fix the etcd backup pod crashlooping because of the
backup file name not getting passed correctly to tar
Change-Id: I782055f8c97160cb3697f2ccb7ef340091ad7491
Signed-off-by: Sreejith Punnapuzha <Sreejith.Punnapuzha@outlook.com>
- Changed backup path to /var/backups/etcd
- Chanded backup filename to service name to support multiple releases
- Removed additional etcd from cronjob name
Change-Id: I1fabdfe1dccd8e170090eec0a69b2598e1e3e422
Signed-off-by: Sreejith Punnapuzha <Sreejith.Punnapuzha@outlook.com>
This is an effort to impletment etcd backup.
This will create a k8s cron job to take a regular backup.
Change-Id: If2c89ac01540c0f13f9b57a6833a8ea770379717
Signed-off-by: Sreejith Punnapuzha <Sreejith.Punnapuzha@outlook.com>
Markin, Sergiy (sm515x)