Skip to content

Add generic container-image-content-provider job#345

Draft
vyzigold wants to merge 1 commit into
masterfrom
jwysogla-telemetry-forks
Draft

Add generic container-image-content-provider job#345
vyzigold wants to merge 1 commit into
masterfrom
jwysogla-telemetry-forks

Conversation

@vyzigold
Copy link
Copy Markdown
Contributor

@vyzigold vyzigold commented Apr 15, 2026

Add an abstract Zuul job and playbook that builds a container image from the current change and serves it from a local registry. This provides a reusable content-provider pattern for repos like sg-core, prometheus-podman-exporter, and mysqld_exporter that need container image builds as part of the telemetry verification pipeline.

Depends-On: openstack-k8s-operators/ci-framework#3856
Depends-On: openstack-k8s-operators/ci-framework#3938

Depends-On: openstack-k8s-operators/sg-core#40
Depends-On: openstack-k8s-operators/mysqld_exporter#3
Depends-On: openstack-k8s-operators/prometheus-podman-exporter#6
Generated-By: Claude-Code claude-opus-4-6

Comment thread ci/playbooks/container-image-content-provider.yml Outdated

- name: Set full image reference
ansible.builtin.set_fact:
_container_image_url: "{{ cifmw_rp_registry_ip }}:5001/{{ container_build_image_name }}:{{ _container_image_tag }}"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Does this only support single container image?

Is there a usecase where we'd need to build multiple containers? e.g. with multiple depends-on?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, it supported only a single image, which was my original goal. But I think it's a good idea to just build all 3 of the images. Although I don't think we'll need to build more than 1 anytime soon, it'll be a bit more future-proof.

The 3rd commit rewrote the job to build all 3 images.

@elfiesmelfie
Copy link
Copy Markdown
Collaborator

  • I would like to see a concrete child job running before this merges

vyzigold added a commit to vyzigold/sg-core-k8s-operators that referenced this pull request Apr 15, 2026
Copy the abstract job definition from feature-verification-tests into
sg-core so that Zuul can resolve the parent job before the fvt PR
merges. Remove this once infrawatch/feature-verification-tests#345
lands.

Generated-By: Claude-Code claude-opus-4-6
vyzigold added a commit to vyzigold/sg-core-k8s-operators that referenced this pull request Apr 15, 2026
Copy the abstract job definition from feature-verification-tests into
sg-core so that Zuul can resolve the parent job before the fvt PR
merges. Remove this once infrawatch/feature-verification-tests#345
lands.

Generated-By: Claude-Code claude-opus-4-6
vyzigold added a commit to vyzigold/sg-core-k8s-operators that referenced this pull request Apr 15, 2026
Copy the abstract job definition from feature-verification-tests into
sg-core so that Zuul can resolve the parent job before the fvt PR
merges. Remove this once infrawatch/feature-verification-tests#345
lands.

Generated-By: Claude-Code claude-opus-4-6
vyzigold added a commit to vyzigold/sg-core-k8s-operators that referenced this pull request Apr 15, 2026
Copy the abstract job definition from feature-verification-tests into
sg-core so that Zuul can resolve the parent job before the fvt PR
merges. Remove this once infrawatch/feature-verification-tests#345
lands.

Generated-By: Claude-Code claude-opus-4-6
vyzigold added a commit to vyzigold/sg-core-k8s-operators that referenced this pull request Apr 16, 2026
Copy the job definition and playbooks from feature-verification-tests
into sg-core so that Zuul can resolve the job before the fvt PR
merges. Remove this once infrawatch/feature-verification-tests#345
lands.

Generated-By: Claude-Code claude-opus-4-6
@vyzigold
Copy link
Copy Markdown
Contributor Author

  • I would like to see a concrete child job running before this merges

This is a bit of an issue, which is why I wanted to merge this PR and update the job as we start to use it. I'm pretty sure you need the job to exist outside of a PR branch, before you can define child jobs. But for now I solved it with just temporarily duplicating the whole job into my sg-core PR (openstack-k8s-operators/sg-core#33) . I'll try to keep this PR and the sg-core one in sync and once we see the content provider working there, we'll have a pretty good proof that this PR works.

vyzigold added a commit to vyzigold/sg-core-k8s-operators that referenced this pull request Apr 16, 2026
Copy the job definition and playbooks from feature-verification-tests
into sg-core so that Zuul can resolve the job before the fvt PR
merges. Remove this once infrawatch/feature-verification-tests#345
lands.

Generated-By: Claude-Code claude-opus-4-6
vyzigold added a commit to vyzigold/sg-core-k8s-operators that referenced this pull request Apr 16, 2026
Copy the job definition and playbooks from feature-verification-tests
into sg-core so that Zuul can resolve the job before the fvt PR
merges. Remove this once infrawatch/feature-verification-tests#345
lands.

Generated-By: Claude-Code claude-opus-4-6
vyzigold added a commit to vyzigold/sg-core-k8s-operators that referenced this pull request Apr 16, 2026
Copy the job definition and playbooks from feature-verification-tests
into sg-core so that Zuul can resolve the job before the fvt PR
merges. Remove this once infrawatch/feature-verification-tests#345
lands.

Generated-By: Claude-Code claude-opus-4-6
@vyzigold vyzigold marked this pull request as draft April 16, 2026 20:27
@vyzigold vyzigold force-pushed the jwysogla-telemetry-forks branch from 4acad4e to 47a3500 Compare April 16, 2026 21:37
vyzigold added a commit to vyzigold/sg-core-k8s-operators that referenced this pull request Apr 17, 2026
Copy the job definition and playbooks from feature-verification-tests
into sg-core so that Zuul can resolve the job before the fvt PR
merges. Remove this once infrawatch/feature-verification-tests#345
lands.

Generated-By: Claude-Code claude-opus-4-6
vyzigold added a commit to vyzigold/sg-core-k8s-operators that referenced this pull request Apr 17, 2026
Copy the job definition and playbooks from feature-verification-tests
into sg-core so that Zuul can resolve the job before the fvt PR
merges. Remove this once infrawatch/feature-verification-tests#345
lands.

Generated-By: Claude-Code claude-opus-4-6
vyzigold added a commit to vyzigold/sg-core-k8s-operators that referenced this pull request May 18, 2026
Copy the job definition and playbooks from feature-verification-tests
into sg-core so that Zuul can resolve the job before the fvt PR
merges. Remove this once infrawatch/feature-verification-tests#345
lands.

Generated-By: Claude-Code claude-opus-4-6
@vyzigold
Copy link
Copy Markdown
Contributor Author

recheck

@centosinfra-prod-github-app
Copy link
Copy Markdown

Zuul encountered a syntax error while parsing its
configuration in the repo infrawatch/feature-verification-tests on branch master. The
problem was:

The nodeset "centos-stream-9-vexxhost" was not found.

The problem appears in the "telemetry-container-image-content-provider" job stanza:

job:
name: telemetry-container-image-content-provider
parent: cifmw-base-minimal
nodeset: centos-stream-9-vexxhost
description: |
Build sg-core, prometheus-podman-exporter and mysqld-exporter
container images from the current change and serve them from
a local registry for dependent jobs.
required-projects:
...

in "infrawatch/feature-verification-tests/.zuul.yaml@master", line 24

vyzigold added a commit to vyzigold/sg-core-k8s-operators that referenced this pull request May 18, 2026
Copy the job definition and playbooks from feature-verification-tests
into sg-core so that Zuul can resolve the job before the fvt PR
merges. Remove this once infrawatch/feature-verification-tests#345
lands.

Generated-By: Claude-Code claude-opus-4-6
vyzigold added a commit to vyzigold/sg-core-k8s-operators that referenced this pull request May 18, 2026
Copy the job definition and playbooks from feature-verification-tests
into sg-core so that Zuul can resolve the job before the fvt PR
merges. Remove this once infrawatch/feature-verification-tests#345
lands.

Generated-By: Claude-Code claude-opus-4-6
vyzigold added a commit to vyzigold/sg-core-k8s-operators that referenced this pull request May 27, 2026
Copy the job definition and playbooks from feature-verification-tests
into sg-core so that Zuul can resolve the job before the fvt PR
merges. Remove this once infrawatch/feature-verification-tests#345
lands.

Generated-By: Claude-Code claude-opus-4-6
@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/c701d2e968a24864a0324d2f3821e2c7

✔️ telemetry-openstack-meta-content-provider-master SUCCESS in 21m 53s
telemetry-container-image-content-provider ERROR Project github.com/openstack-k8s-operators/prometheus-podman-exporter does not have the default branch master in 15s
functional-chargeback-tests-osp18 ERROR Project github.com/openstack-k8s-operators/mysqld_exporter does not have the default branch master in 21s
⚠️ functional-tests-osp18 SKIPPED Skipped due to failed job telemetry-container-image-content-provider
functional-logging-tests-osp18 ERROR Project github.com/openstack-k8s-operators/mysqld_exporter does not have the default branch master in 22s
functional-periodic-telemetry-with-ceph ERROR Project github.com/openstack-k8s-operators/mysqld_exporter does not have the default branch master in 22s

@vyzigold vyzigold force-pushed the jwysogla-telemetry-forks branch from ad578a7 to be3d5b1 Compare May 28, 2026 06:34
@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/530af2ba1d3a4b79a8cb17ce9c260e89

✔️ telemetry-openstack-meta-content-provider-master SUCCESS in 20m 00s
✔️ telemetry-container-image-content-provider SUCCESS in 19m 38s
functional-chargeback-tests-osp18 ERROR Project github.com/openstack-k8s-operators/mysqld_exporter does not have the default branch master in 21s
functional-tests-osp18 ERROR Project github.com/openstack-k8s-operators/mysqld_exporter does not have the default branch master in 23s
functional-logging-tests-osp18 ERROR Project github.com/openstack-k8s-operators/mysqld_exporter does not have the default branch master in 23s
functional-periodic-telemetry-with-ceph ERROR Project github.com/openstack-k8s-operators/mysqld_exporter does not have the default branch master in 23s

@vyzigold vyzigold force-pushed the jwysogla-telemetry-forks branch from be3d5b1 to b987db8 Compare May 28, 2026 07:20
@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/b2778ae2b2e14a5d838b4e79389f683b

✔️ telemetry-openstack-meta-content-provider-master SUCCESS in 3h 00m 49s
✔️ telemetry-container-image-content-provider SUCCESS in 2h 44m 38s
functional-chargeback-tests-osp18 ERROR Project github.com/openstack-k8s-operators/mysqld_exporter does not have the default branch master in 22s
✔️ functional-tests-osp18 SUCCESS in 2h 28m 29s
✔️ functional-logging-tests-osp18 SUCCESS in 1h 38m 54s
✔️ functional-periodic-telemetry-with-ceph SUCCESS in 2h 44m 29s

@vyzigold vyzigold force-pushed the jwysogla-telemetry-forks branch 2 times, most recently from a6120b9 to 9f8baba Compare June 2, 2026 08:48
@centosinfra-prod-github-app
Copy link
Copy Markdown

This change depends on a change that failed to merge.

Change openstack-k8s-operators/ci-framework#3856 is needed.

@vyzigold
Copy link
Copy Markdown
Contributor Author

vyzigold commented Jun 2, 2026

recheck

@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/e25407f36f8c4720973fa683b009d928

✔️ telemetry-openstack-meta-content-provider-master SUCCESS in 4h 21m 39s
✔️ telemetry-container-image-content-provider SUCCESS in 20m 18s
✔️ functional-chargeback-tests-osp18 SUCCESS in 1h 26m 30s
functional-tests-osp18 RETRY_LIMIT in 29s
✔️ functional-logging-tests-osp18 SUCCESS in 1h 32m 55s
✔️ functional-periodic-telemetry-with-ceph SUCCESS in 2h 30m 55s

@vyzigold
Copy link
Copy Markdown
Contributor Author

vyzigold commented Jun 3, 2026

recheck

@centosinfra-prod-github-app
Copy link
Copy Markdown

This change depends on a change that failed to merge.

Change openstack-k8s-operators/ci-framework#3856 is needed.

@vyzigold vyzigold force-pushed the jwysogla-telemetry-forks branch 2 times, most recently from 075a423 to 40f1de8 Compare June 3, 2026 14:27
@centosinfra-prod-github-app
Copy link
Copy Markdown

Zuul encountered a syntax error while parsing its
configuration in the repo infrawatch/feature-verification-tests on branch master. The
problem was:

The parent of job "feature-verification-tests-noop", "noop" is final and can not act as a parent

The problem appears in the "feature-verification-tests-noop" job stanza:

job:
name: feature-verification-tests-noop
parent: noop
description: |
A job that always passes. Runs when there's a change to jobs that don't
need full zuul to run but still need to report a pass.
run:
- ci/noop.yml

in "infrawatch/feature-verification-tests/.zuul.yaml@master", line 158

@vyzigold vyzigold force-pushed the jwysogla-telemetry-forks branch from 40f1de8 to aec98fa Compare June 3, 2026 16:13
@vyzigold
Copy link
Copy Markdown
Contributor Author

vyzigold commented Jun 3, 2026

recheck nodefail

@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/13b30984974f483d856449ec96700484

✔️ telemetry-openstack-meta-content-provider-master SUCCESS in 3h 20m 03s
✔️ telemetry-container-image-content-provider SUCCESS in 42m 41s
✔️ functional-chargeback-tests-osp18 SUCCESS in 1h 32m 29s
functional-tests-osp18 NODE_FAILURE Node(set) request 099-0000110686 failed in 0s
✔️ functional-logging-tests-osp18 SUCCESS in 1h 28m 41s
✔️ functional-periodic-telemetry-with-ceph SUCCESS in 2h 35m 22s

@vyzigold
Copy link
Copy Markdown
Contributor Author

vyzigold commented Jun 3, 2026

request nodefailure

@vyzigold
Copy link
Copy Markdown
Contributor Author

vyzigold commented Jun 3, 2026

recheck nodefailure

@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/20051090aa834d2eae9c72b7d8a1d071

✔️ telemetry-openstack-meta-content-provider-master SUCCESS in 3h 12m 28s
✔️ telemetry-container-image-content-provider SUCCESS in 2h 38m 19s
✔️ functional-chargeback-tests-osp18 SUCCESS in 1h 27m 33s
functional-tests-osp18 FAILURE in 2h 00m 25s
✔️ functional-logging-tests-osp18 SUCCESS in 1h 28m 00s
✔️ functional-periodic-telemetry-with-ceph SUCCESS in 2h 32m 15s

@vyzigold
Copy link
Copy Markdown
Contributor Author

vyzigold commented Jun 4, 2026

recheck

@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/57cff39402a44c34a47c347a547d9402

✔️ telemetry-openstack-meta-content-provider-master SUCCESS in 3h 33m 44s
✔️ telemetry-container-image-content-provider SUCCESS in 2h 49m 00s
✔️ functional-chargeback-tests-osp18 SUCCESS in 1h 35m 35s
functional-tests-osp18 FAILURE in 1h 59m 35s
✔️ functional-logging-tests-osp18 SUCCESS in 1h 34m 49s
✔️ functional-periodic-telemetry-with-ceph SUCCESS in 2h 36m 56s

Add an abstract Zuul job and playbook that builds a container image
from the current change and serves it from a local registry. This
provides a reusable content-provider pattern for repos like sg-core,
prometheus-podman-exporter, and mysqld_exporter that need container
image builds as part of the telemetry verification pipeline.

Generated-By: Claude-Code claude-opus-4-6
@vyzigold vyzigold force-pushed the jwysogla-telemetry-forks branch from aec98fa to cabcabd Compare June 4, 2026 16:35
@centosinfra-prod-github-app
Copy link
Copy Markdown

This change depends on a change that failed to merge.

Change openstack-k8s-operators/ci-framework#3938 is needed.

@vyzigold
Copy link
Copy Markdown
Contributor Author

vyzigold commented Jun 4, 2026

recheck

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants