Skip to content

[release-4.22] OCPBUGS-89322: cluster-restore.sh: move extra /var/lib/etcd files to backup#1632

Open
openshift-cherrypick-robot wants to merge 4 commits into
openshift:release-4.22from
openshift-cherrypick-robot:cherry-pick-1628-to-release-4.22
Open

[release-4.22] OCPBUGS-89322: cluster-restore.sh: move extra /var/lib/etcd files to backup#1632
openshift-cherrypick-robot wants to merge 4 commits into
openshift:release-4.22from
openshift-cherrypick-robot:cherry-pick-1628-to-release-4.22

Conversation

@openshift-cherrypick-robot

Copy link
Copy Markdown

This is an automated cherry-pick of #1628

/assign apurvanisal5

Apurva Nisal and others added 4 commits June 17, 2026 10:04
When cluster-restore.sh runs the restore-pod path, it moves member/ and
revision.json to /var/lib/etcd-backup, deletes etcd_perf*, then exits if
anything remains in /var/lib/etcd. Extra files (perf artifacts, stray
snapshots, etc.) cause DR restore to fail before the restore pod starts.

Add backup_remaining_etcd_data_dir_contents() to move all remaining
top-level entries to /var/lib/etcd-backup instead of failing.

Fixes: ETCD-704
Related: https://access.redhat.com/solutions/6958920
Move leftover /var/lib/etcd entries to extra-data-dir-contents/
so staged snapshot.db, revision.json, and member/ are not overwritten.

Co-authored-by: Cursor <cursoragent@cursor.com>
…ing_etcd_data_dir_contents via etcd-common-tools per review feedback. Behavior unchanged.
…ll backup_remaining_etcd_data_dir_contents before rm -rf so leftover files are moved to extra-data-dir-contents/ instead of deleted.
@openshift-ci-robot

openshift-ci-robot commented Jun 17, 2026

Copy link
Copy Markdown

@openshift-cherrypick-robot: Ignoring requests to cherry-pick non-bug issues: ETCD-704

Details

In response to this:

This is an automated cherry-pick of #1628

/assign apurvanisal5

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci openshift-ci Bot requested review from ironcladlou and tjungblu June 17, 2026 10:05
@apurvanisal5

Copy link
Copy Markdown

/cherry-pick release-4.21

@openshift-cherrypick-robot

Copy link
Copy Markdown
Author

@apurvanisal5: once the present PR merges, I will cherry-pick it on top of release-4.21 in a new PR and assign it to you.

Details

In response to this:

/cherry-pick release-4.21

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@apurvanisal5

Copy link
Copy Markdown

/jira refresh

@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Jun 17, 2026
@openshift-ci-robot

openshift-ci-robot commented Jun 17, 2026

Copy link
Copy Markdown

@apurvanisal5: This pull request references ETCD-704 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the task to target either version "4.22." or "openshift-4.22.", but it targets "openshift-5.0" instead.

Details

In response to this:

/jira refresh

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@apurvanisal5

apurvanisal5 commented Jun 17, 2026

Copy link
Copy Markdown

/verified by @apurvanisal5

Backport of #1628. Manual restore verified (ETCD-704).

@openshift-ci-robot openshift-ci-robot added the verified Signifies that the PR passed pre-merge verification criteria label Jun 17, 2026
@openshift-ci-robot

openshift-ci-robot commented Jun 17, 2026

Copy link
Copy Markdown

@apurvanisal5: This PR has been marked as verified by @apurvanisal5.

Details

In response to this:

/verified by @apurvanisal5

Backport of #1628. Manual restore verified (ETCD-704).

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@tjungblu

Copy link
Copy Markdown
Contributor

🤖 Automated labeling by agent on behalf of @tjungblu

This is an automated bot PR. Adding required Tide labels:

/approve
/label backport-risk-assessed


This action was performed by an automated agent. If this is incorrect, please review and adjust manually.

@openshift-ci openshift-ci Bot added the backport-risk-assessed Indicates a PR to a release branch has been evaluated and considered safe to accept. label Jun 17, 2026
@openshift-ci

openshift-ci Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: tjungblu

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 17, 2026
@tjungblu

Copy link
Copy Markdown
Contributor

/lgtm

@openshift-ci openshift-ci Bot added the lgtm Indicates that a PR is ready to be merged. label Jun 17, 2026
@openshift-ci

openshift-ci Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

@openshift-cherrypick-robot: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-metal-ipi-ovn-ipv6 04434a0 link true /test e2e-metal-ipi-ovn-ipv6
ci/prow/e2e-gcp-operator-disruptive 04434a0 link true /test e2e-gcp-operator-disruptive
ci/prow/e2e-aws-ovn-single-node 04434a0 link true /test e2e-aws-ovn-single-node

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@apurvanisal5

Copy link
Copy Markdown

/retitle [release-4.22] OCPBUGS-89322: cluster-restore.sh: move extra /var/lib/etcd files to backup
/jira refresh

@openshift-ci openshift-ci Bot changed the title [release-4.22] ETCD-704: cluster-restore.sh: move extra /var/lib/etcd files to backup [release-4.22] OCPBUGS-89322: cluster-restore.sh: move extra /var/lib/etcd files to backup Jun 18, 2026
@openshift-ci-robot

openshift-ci-robot commented Jun 18, 2026

Copy link
Copy Markdown

@apurvanisal5: This pull request references ETCD-704 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the task to target either version "4.22." or "openshift-4.22.", but it targets "openshift-5.0" instead.

Details

In response to this:

/retitle [release-4.22] OCPBUGS-89322: cluster-restore.sh: move extra /var/lib/etcd files to backup
/jira refresh

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@openshift-ci-robot openshift-ci-robot added the jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. label Jun 18, 2026
@openshift-ci-robot

Copy link
Copy Markdown

@openshift-cherrypick-robot: This pull request references Jira Issue OCPBUGS-89322, which is invalid:

  • expected the bug to target the "4.22.0" version, but no target version was set
  • release note text must be set and not match the template OR release note type must be set to "Release Note Not Required". For more information you can reference the OpenShift Bug Process.
  • expected Jira Issue OCPBUGS-89322 to depend on a bug targeting a version in 5.0.0 and in one of the following states: MODIFIED, ON_QA, VERIFIED, but no dependents were found

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

The bug has been updated to refer to the pull request using the external bug tracker.

Details

In response to this:

This is an automated cherry-pick of #1628

/assign apurvanisal5

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

@apurvanisal5

Copy link
Copy Markdown

/jira refresh

@openshift-ci-robot

Copy link
Copy Markdown

@apurvanisal5: This pull request references Jira Issue OCPBUGS-89322, which is invalid:

  • expected the bug to target the "4.22.0" version, but no target version was set
  • release note text must be set and not match the template OR release note type must be set to "Release Note Not Required". For more information you can reference the OpenShift Bug Process.
  • expected Jira Issue OCPBUGS-89322 to depend on a bug targeting a version in 5.0.0 and in one of the following states: MODIFIED, ON_QA, VERIFIED, but no dependents were found

Comment /jira refresh to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.

Details

In response to this:

/jira refresh

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. backport-risk-assessed Indicates a PR to a release branch has been evaluated and considered safe to accept. jira/invalid-bug Indicates that a referenced Jira bug is invalid for the branch this PR is targeting. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged. verified Signifies that the PR passed pre-merge verification criteria

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants