Skip to content

Enable northd metrics#1587

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
karelyatin:OSPRH-12568
Sep 4, 2025
Merged

Enable northd metrics#1587
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
karelyatin:OSPRH-12568

Conversation

@karelyatin

Copy link
Copy Markdown
Contributor

@openshift-ci openshift-ci Bot requested review from dprince and slagle September 2, 2025 07:17
@softwarefactory-project-zuul

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.

@karelyatin

Copy link
Copy Markdown
Contributor Author

recheck rdo zuul fixed

@softwarefactory-project-zuul

Copy link
Copy Markdown

This change depends on a change that failed to merge.

Change openstack-k8s-operators/ovn-operator#475 is needed.

@softwarefactory-project-zuul

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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/8ebe334efd3b4698aa81c337d2a3903d

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 16m 41s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 17m 01s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 29m 48s
✔️ adoption-standalone-to-crc-ceph-provider SUCCESS in 3h 02m 26s
openstack-operator-tempest-multinode RETRY_LIMIT in 9m 26s

@karelyatin

Copy link
Copy Markdown
Contributor Author

recheck RETRY_LIMIT CRC

@softwarefactory-project-zuul

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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/191a28df68784490ae7b201fc4e92b9d

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 24m 04s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 16m 22s
✔️ cifmw-crc-podified-edpm-baremetal SUCCESS in 1h 34m 08s
✔️ adoption-standalone-to-crc-ceph-provider SUCCESS in 3h 05m 34s
openstack-operator-tempest-multinode FAILURE in 1h 40m 24s

@karelyatin

Copy link
Copy Markdown
Contributor Author

recheck slow libvirt dps

@softwarefactory-project-zuul

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://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/fd47fafc058743a1b379a72067ced12a

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 24m 06s
✔️ podified-multinode-edpm-deployment-crc SUCCESS in 1h 19m 53s
cifmw-crc-podified-edpm-baremetal FAILURE in 1h 37m 32s
✔️ adoption-standalone-to-crc-ceph-provider SUCCESS in 3h 09m 31s
✔️ openstack-operator-tempest-multinode SUCCESS in 1h 30m 12s

@karelyatin

Copy link
Copy Markdown
Contributor Author

recheck mirror issues

@abays abays 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.

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Sep 4, 2025
@openshift-ci

openshift-ci Bot commented Sep 4, 2025

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: abays, karelyatin

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 label Sep 4, 2025
@openshift-merge-bot openshift-merge-bot Bot merged commit 673a45e into openstack-k8s-operators:main Sep 4, 2025
8 checks passed
karelyatin added a commit to karelyatin/ovn-operator that referenced this pull request Sep 5, 2025
For minor update scenarios we need to honor exporterImage as
in older versions will not have OpenstackNetworkExporterImage
set so exporterImage="".
So with this patch metrics will be enabled only when
metricsEnabled=true and exporterImage is set.

Also removed the default image as now included in
openstack-operator like other images, this was temporary
change.

Also dropped unnecessary log message when deployment do not exist.

Related-Issue: https://issues.redhat.com/browse/OSPRH-12568
Depends-On: openstack-k8s-operators/openstack-operator#1587
@karelyatin karelyatin deleted the OSPRH-12568 branch September 25, 2025 14:04
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.

2 participants