Skip to content

[WIP] Add configurable probes and APITimeout-based placement defaults#1146

Draft
amartyasinha wants to merge 1 commit into
openstack-k8s-operators:mainfrom
amartyasinha:dynamic_probes
Draft

[WIP] Add configurable probes and APITimeout-based placement defaults#1146
amartyasinha wants to merge 1 commit into
openstack-k8s-operators:mainfrom
amartyasinha:dynamic_probes

Conversation

@amartyasinha

@amartyasinha amartyasinha commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Expose spec.override.probes on Nova and PlacementAPI CRDs and build pod
probes via lib-common CreateProbeSet helpers with shared defaults in
internal/common/probes.go. Tune placement-api liveness/readiness from
APITimeout to avoid restarting busy pods.

@openshift-ci

openshift-ci Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

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

openshift-ci Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: amartyasinha

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

Copy link
Copy Markdown

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/openstack-k8s-operators/nova-operator for 1146,6a9081e1696aa70caa80289ab324646f6270fc2b

@centosinfra-prod-github-app

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/349354fc050b489c907ebebf79cdcea3

openstack-meta-content-provider FAILURE in 5m 03s
⚠️ nova-operator-kuttl SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ nova-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-meta-content-provider
⚠️ nova-operator-tempest-multinode-ceph SKIPPED Skipped due to failed job openstack-meta-content-provider

@amartyasinha amartyasinha force-pushed the dynamic_probes branch 2 times, most recently from 55ef06d to 2f6317b Compare July 9, 2026 00:34
Expose spec.override.probes on Nova and PlacementAPI CRDs and build pod
probes via lib-common CreateProbeSet helpers with shared defaults in
internal/common/probes.go. Tune placement-api liveness/readiness from
APITimeout to avoid restarting busy pods.

Signed-off-by: Amartya Sinha <amsinha@redhat.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@centosinfra-prod-github-app

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/e9f177f5e9834494b32714ff5a56c3ff

✔️ openstack-meta-content-provider SUCCESS in 3h 04m 27s
nova-operator-kuttl FAILURE in 1h 03m 13s
✔️ nova-operator-tempest-multinode SUCCESS in 2h 19m 37s
✔️ nova-operator-tempest-multinode-ceph SUCCESS in 2h 47m 03s

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant