Skip to content

fix: revert etcd-backup-restore to 0.25.1#1082

Closed
s10 wants to merge 2 commits into
masterfrom
revert/etcd-kubernikus-backup
Closed

fix: revert etcd-backup-restore to 0.25.1#1082
s10 wants to merge 2 commits into
masterfrom
revert/etcd-kubernikus-backup

Conversation

@s10

@s10 s10 commented Jun 22, 2026

Copy link
Copy Markdown
Contributor
  • guard check currently contains release name as a namespace variabe
  • pod namespace is used in newer etcd-backup-restore versions

Use cluster name for POD_NAME variable instead of generic "kubernikus" in >=1.34 clusters

POD_NAME is used by etcd-backup-restore to get `initial-advertise-peer-urls` and `advertise-client-urls`.

In gardener, etcd is running as sts, so static name could be used there, but in kubernikus we run etcd as a deployment
@s10 s10 requested review from a team, databus23 and jknipper as code owners June 22, 2026 15:31
* guard check currently contains release name as a namespace variabe
* pod namespace is used in newer etcd-backup-restore versions
@s10 s10 force-pushed the revert/etcd-kubernikus-backup branch from da94325 to 3d4b53a Compare June 22, 2026 15:32
- name: POD_NAME
{{- if (semverCompare ">= 1.34-0" .Values.version.kubernetes) }}
value: "kubernikus"
value: {{ include "fullname" . }}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Used in etcd.conf.yaml

@s10 s10 closed this Jun 25, 2026
@s10 s10 deleted the revert/etcd-kubernikus-backup branch June 25, 2026 09:00
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.

1 participant