Add telemetry containers

[elfiesmelfie]

No description provided.

[amoralej]

Some comments:

• Please run tox -esupdate-sources and include all the automatically generated files in the commit.
• Adapt the containerfiles to the latest content in https://github.com/openstack-k8s-operators/s2i-openstack-containers/blob/main/containers/watcher/watcher-base/Containerfile

Special care:

• https://github.com/openstack-k8s-operators/s2i-openstack-containers/blob/main/containers/watcher/watcher-base/Containerfile#L13-L15
• Differences in the pip commands (we slighly changed the workflow to do two pip3 wheel and change the pip install comands)

[openshift-ci]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign stuggi for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: 17194b97-e129-4f7e-a632-5f14dda16866

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[centosinfra-prod-github-app]

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

❌ s2i-openstack-containers-molecule FAILURE in 1m 47s
❌ s2i-openstack-container-content-provider FAILURE in 2m 51s

[openshift-ci]

PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

[amoralej]

This command affects to all the containers

bindeps always to at least have:

python3
python3-pip

Also, given that cryptography is pulled as dependency of dependencies, this should also have:

python3-cryptography-43.0.0-4.el10

Also, same for builddeps.txt ( I think i missed it in aodh, btw).

Given that you are having multiple images in the service. You may implement base bindeps.txt for the common part + image bindeps.txt for specific ones.

You can check how it was done in cinder i.e.. The script build.sh supports those two levels for bindeps pythondeps, etc.. and it makes easier to support in multi image. In containerfiles you need to copy both files and cat them in the installation commands.

[elfiesmelfie]

How were these building if these were missing? Were they being pulled in as-needed in the regular build.
But this will cause issues in an isolated build

[amoralej]

For confirmation, ceilometer doesn't access database?

[elfiesmelfie]

ceilometer collects metrics and publishes it to external backends.

it doesn't need databases