Skip to content

kuttl: set empty MetricStorage networkAttachments#359

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
amoralej:fix-kuttl-nettattach
Apr 10, 2026
Merged

kuttl: set empty MetricStorage networkAttachments#359
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
amoralej:fix-kuttl-nettattach

Conversation

@amoralej
Copy link
Copy Markdown
Contributor

@amoralej amoralej commented Apr 7, 2026 •
edited
Loading

telemetry-operator PR #827 (openstack-k8s-operators/telemetry-operator#827) adds a default networkAttachments on MetricStorage, which broke kuttl.

Set networkAttachments: [] under spec.telemetry.template.metricStorage in the shared OpenStackControlPlane deps.

Assisted-By: Cursor (Composer 2)

Depends-On: openstack-k8s-operators/openstack-operator#1878

@amoralej
kuttl: set empty MetricStorage networkAttachments
7beeb15 
telemetry-operator PR #827 (openstack-k8s-operators/telemetry-operator#827)
adds a default networkAttachments on MetricStorage, which broke kuttl.

Set networkAttachments: [] under spec.telemetry.template.metricStorage in the shared OpenStackControlPlane deps.

Assisted-By: Cursor (Composer 2)
Signed-off-by: Alfredo Moralejo <amoralej@redhat.com>
@openshift-ci openshift-ci Bot requested review from abays and raukadah April 7, 2026 09:10
@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented Apr 7, 2026

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/41dfd63f02b645bd956f8ab43ce482b5

✔️ openstack-meta-content-provider-master SUCCESS in 1h 29m 48s
❌ watcher-operator-validation-master RETRY_LIMIT in 3m 29s
✔️ openstack-meta-content-provider-epoxy SUCCESS in 2h 25m 24s
✔️ watcher-operator-validation-epoxy SUCCESS in 1h 57m 49s
✔️ watcher-operator-validation-epoxy-ocp4-16 SUCCESS in 2h 02m 36s
✔️ noop SUCCESS in 0s
✔️ watcher-operator-kuttl SUCCESS in 52m 41s

@amoralej
Copy link
Copy Markdown
Contributor Author

amoralej commented Apr 7, 2026

recheck

@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented Apr 7, 2026

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/4f637bcb26574cfba03d663df092aafc

✔️ openstack-meta-content-provider-master SUCCESS in 3h 00m 00s
✔️ watcher-operator-validation-master SUCCESS in 2h 10m 17s
✔️ openstack-meta-content-provider-epoxy SUCCESS in 1h 42m 37s
❌ watcher-operator-validation-epoxy FAILURE in 22m 27s
❌ watcher-operator-validation-epoxy-ocp4-16 FAILURE in 1h 15m 13s
✔️ noop SUCCESS in 0s
✔️ watcher-operator-kuttl SUCCESS in 52m 27s

@amoralej
Copy link
Copy Markdown
Contributor Author

amoralej commented Apr 7, 2026

check-rdo

@amoralej amoralej mentioned this pull request Apr 7, 2026
Open
@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented Apr 7, 2026

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/03577273b4a54ea581c9aba803b16871

✔️ openstack-meta-content-provider-master SUCCESS in 2h 50m 07s
✔️ watcher-operator-validation-master SUCCESS in 2h 11m 03s
✔️ openstack-meta-content-provider-epoxy SUCCESS in 2h 16m 40s
✔️ watcher-operator-validation-epoxy SUCCESS in 1h 54m 17s
❌ watcher-operator-validation-epoxy-ocp4-16 FAILURE in 33m 12s
✔️ noop SUCCESS in 0s
✔️ watcher-operator-kuttl SUCCESS in 50m 28s

@amoralej
Copy link
Copy Markdown
Contributor Author

amoralej commented Apr 8, 2026

check-rdo

@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented Apr 8, 2026

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/2bd5994ebbf4486bbf6af937fb20caeb

✔️ openstack-meta-content-provider-master SUCCESS in 2h 21m 14s
❌ watcher-operator-validation-master FAILURE in 1h 38m 14s
✔️ openstack-meta-content-provider-epoxy SUCCESS in 2h 10m 40s
❌ watcher-operator-validation-epoxy FAILURE in 1h 44m 21s
❌ watcher-operator-validation-epoxy-ocp4-16 FAILURE in 1h 39m 14s
✔️ noop SUCCESS in 0s
❌ watcher-operator-kuttl RETRY_LIMIT in 58m 18s

@amoralej
Copy link
Copy Markdown
Contributor Author

amoralej commented Apr 8, 2026

check-rdo

@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented Apr 8, 2026

This change depends on a change that failed to merge.

Change openstack-k8s-operators/telemetry-operator#869 is needed.

@amoralej
Copy link
Copy Markdown
Contributor Author

amoralej commented Apr 8, 2026

check-rdo

@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented Apr 8, 2026

This change depends on a change that failed to merge.

Change openstack-k8s-operators/telemetry-operator#869 is needed.

@amoralej
Copy link
Copy Markdown
Contributor Author

amoralej commented Apr 9, 2026

check-rdo

@softwarefactory-project-zuul
Copy link
Copy Markdown

softwarefactory-project-zuul Bot commented Apr 9, 2026

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

✔️ openstack-meta-content-provider-master SUCCESS in 2h 21m 31s
❌ watcher-operator-validation-master FAILURE in 1h 41m 14s
✔️ openstack-meta-content-provider-epoxy SUCCESS in 2h 03m 31s
❌ watcher-operator-validation-epoxy FAILURE in 1h 41m 42s
❌ watcher-operator-validation-epoxy-ocp4-16 FAILURE in 1h 41m 37s
✔️ noop SUCCESS in 0s
❌ watcher-operator-kuttl RETRY_LIMIT in 59m 00s

@amoralej
Copy link
Copy Markdown
Contributor Author

amoralej commented Apr 9, 2026

check-rdo

@amoralej amoralej mentioned this pull request Apr 9, 2026
Merged
@raukadah
Copy link
Copy Markdown
Contributor

raukadah commented Apr 10, 2026

/lgtm

@openshift-ci openshift-ci Bot added the lgtm label Apr 10, 2026
@raukadah
Copy link
Copy Markdown
Contributor

raukadah commented Apr 10, 2026

/approve depends-on is merged

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Apr 10, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: raukadah

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-merge-bot openshift-merge-bot Bot merged commit 454e5e9 into openstack-k8s-operators:main Apr 10, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cescgina cescgina cescgina approved these changes

@abays abays Awaiting requested review from abays

@raukadah raukadah Awaiting requested review from raukadah

Assignees

@cescgina cescgina

@raukadah raukadah

Labels

approved lgtm

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@amoralej @raukadah @cescgina