Skip to content

[WIP] Stabilize TNF node replacement test with graceful standby shutdown#30728

Draft
clobrano wants to merge 1 commit intoopenshift:mainfrom
clobrano:fix/stabilize-tnf-suite
Draft

[WIP] Stabilize TNF node replacement test with graceful standby shutdown#30728
clobrano wants to merge 1 commit intoopenshift:mainfrom
clobrano:fix/stabilize-tnf-suite

Conversation

@clobrano
Copy link
Contributor

Use pcs standby mode to gracefully stop resources on the target node
before VM destruction. This prevents Pacemaker from attempting fencing
when the node disappears and provides a cleaner shutdown sequence.

Key changes:

  • Put target node in standby mode before destroying VM
  • Wait for etcd to stop on target node rather than surviving node
  • Add PcsNodeStandby/PcsNodeUnstandby helper functions
  • Increase cluster operators health check timeout to 15 minutes
  • Rename provisioning bridge constant from ostestpr to tnfpr
  • Refactor waitForEtcdResourceToStop to accept explicit node parameters

@openshift-ci-robot
Copy link

Pipeline controller notification
This repo is configured to use the pipeline controller. Second-stage tests will be triggered either automatically or after lgtm label is added, depending on the repository configuration. The pipeline controller will automatically detect which contexts are required and will utilize /test Prow commands to trigger the second stage.

For optional jobs, comment /test ? to see a list of all defined jobs. To trigger manually all jobs from second stage use /pipeline required command.

This repository is configured in: automatic mode

@openshift-ci openshift-ci bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 27, 2026
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 27, 2026

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 27, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: clobrano
Once this PR has been reviewed and has the lgtm label, please assign eggfoobar for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@clobrano
Copy link
Contributor Author

/payload-job periodic-ci-openshift-release-master-nightly-4.22-e2e-metal-ovn-two-node-fencing-recovery-techpreview

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 27, 2026

@clobrano: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-release-master-nightly-4.22-e2e-metal-ovn-two-node-fencing-recovery-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/949da370-fb80-11f0-88d3-7535f9ddec9d-0

…tdown

Use pcs standby mode to gracefully stop resources on the target node
before VM destruction. This prevents Pacemaker from attempting fencing
when the node disappears and provides a cleaner shutdown sequence.

Key changes:
- Put target node in standby mode before destroying VM
- Wait for etcd to stop on target node rather than surviving node
- Add PcsNodeStandby/PcsNodeUnstandby helper functions
- Increase cluster operators health check timeout to 15 minutes
- Rename provisioning bridge constant from ostestpr to tnfpr
- Refactor waitForEtcdResourceToStop to accept explicit node parameters
@clobrano clobrano force-pushed the fix/stabilize-tnf-suite branch from 71b3336 to 8395e2c Compare January 27, 2026 16:11
@clobrano
Copy link
Contributor Author

/payload-job periodic-ci-openshift-release-master-nightly-4.22-e2e-metal-ovn-two-node-fencing-recovery-techpreview

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 27, 2026

@clobrano: trigger 1 job(s) for the /payload-(with-prs|job|aggregate|job-with-prs|aggregate-with-prs) command

  • periodic-ci-openshift-release-master-nightly-4.22-e2e-metal-ovn-two-node-fencing-recovery-techpreview

See details on https://pr-payload-tests.ci.openshift.org/runs/ci/eaca0990-fb9a-11f0-93d8-b2d166ddcfb2-0

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

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants