Skip to content

[update] Make post-reboot sanity check timeout configurable - #4214

Open
srac0 wants to merge 1 commit into
openstack-k8s-operators:mainfrom
srac0:update/compute-recheck-var
Open

[update] Make post-reboot sanity check timeout configurable#4214
srac0 wants to merge 1 commit into
openstack-k8s-operators:mainfrom
srac0:update/compute-recheck-var

Conversation

@srac0

@srac0 srac0 commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

Make the retry count for post-reboot service sanity checks configurable via cifmw_update_wait_retries_sanity. This allows operators to adjust wait times for nova-compute, ovn-controller, and neutron-ovn-metadata-agent services based on their environment's performance characteristics.

@qodo-code-review

qodo-code-review Bot commented Sep 29, 2026

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@srac0
[update] Make post-reboot sanity check timeout configurable
b033655 
Make the retry count for post-reboot service sanity checks configurable
via `cifmw_update_wait_retries_sanity`. This allows operators to adjust
wait times for nova-compute, ovn-controller, and neutron-ovn-metadata-agent
services based on their environment's performance characteristics.

Signed-off-by: Samuel Rac <srac@redhat.com>
@srac0
srac0 force-pushed the update/compute-recheck-var branch from 75806e5 to b033655 Compare September 29, 2026 08:06
michburk
michburk approved these changes Sep 29, 2026
View reviewed changes

@michburk michburk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

/approve

@openshift-ci

openshift-ci Bot commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: michburk

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

@centosinfra-prod-github-app

centosinfra-prod-github-app Bot commented Sep 29, 2026

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/cd350979da64430ba4fde302fe81cba1

✔️ openstack-k8s-operators-content-provider SUCCESS in 4h 37m 01s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 25m 41s
✔️ podified-multinode-edpm-deployment-crc-centos-10 SUCCESS in 1h 26m 02s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 37m 04s
✔️ cifmw-crc-podified-edpm-baremetal-minor-update SUCCESS in 1h 54m 58s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 50s
✔️ openstack-k8s-operators-content-provider-bootc SUCCESS in 8h 09m 48s
✔️ cifmw-crc-podified-edpm-baremetal-bootc SUCCESS in 1h 26m 53s
✔️ noop SUCCESS in 0s
✔️ cifmw-pod-ansible-test SUCCESS in 10m 02s
✔️ cifmw-pod-pre-commit SUCCESS in 9m 54s
❌ cifmw-molecule-update FAILURE in 6m 16s

This branch has not been deployed

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

Reviewers

@michburk michburk michburk approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

approved

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@srac0 @michburk