Skip to content

Conversation

@lahinson
Copy link
Contributor

@lahinson lahinson commented Oct 22, 2025

Version(s): 4.17

Issue: https://issues.redhat.com/browse/OCPBUGS-63431

Link to docs preview:

QE review:

  • QE has approved this change.

Related PRs:

@openshift-ci openshift-ci bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 22, 2025
@lahinson lahinson force-pushed the ocpbugs-63431-revert-etcd-procedure branch from 54b1ec7 to a741979 Compare October 22, 2025 20:20
@openshift-ci openshift-ci bot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Oct 22, 2025
@lahinson
Copy link
Contributor Author

/retest

@lahinson lahinson force-pushed the ocpbugs-63431-revert-etcd-procedure branch from a741979 to 32e0db2 Compare October 22, 2025 20:57
@palonsoro
Copy link
Contributor

When machines are re-created, this step is not needed:

. If the `keepalived` daemon is in use, restore the configuration on the control plane nodes other than the recovery host by running the following command. Otherwise, the network operator will not advance beyond the "Progressing" state.
+
[source,terminal]
----
$ sudo cp -v /home/core/keepalived.yaml /etc/kubernetes/manifests/
----

So it should be removed

@lahinson lahinson force-pushed the ocpbugs-63431-revert-etcd-procedure branch from 32e0db2 to e4eecfb Compare October 23, 2025 13:28
@openshift-ci
Copy link

openshift-ci bot commented Oct 23, 2025

@lahinson: all tests passed!

Full PR test history. Your PR dashboard.

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.

@sandeepknd
Copy link

/lgtm from QE.

@lahinson lahinson added the merge-review-needed Signifies that the merge review team needs to review this PR label Oct 27, 2025
Copy link
Contributor

@ShaunaDiaz ShaunaDiaz left a comment

Choose a reason for hiding this comment

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

/lgtm

@ShaunaDiaz ShaunaDiaz added ok-to-merge and removed merge-review-needed Signifies that the merge review team needs to review this PR labels Oct 27, 2025
@ShaunaDiaz ShaunaDiaz added this to the Continuous Release milestone Oct 27, 2025
@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Oct 27, 2025
@lahinson lahinson merged commit 638ef5b into openshift:enterprise-4.17 Oct 27, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch/enterprise-4.17 lgtm Indicates that a PR is ready to be merged. ok-to-merge size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants