Replace Ceilometer's ProxyImage with a generic one to avoid duplicati…

[jlarriba]

…on of related images

[jlarriba]

/hold

As discussed with Dan, I am not really comfortable of having Octavia and Telemetry sharing the same image, as that could mean that the image gets updated by one of the teams unkowingly to the other, and that can cause issues.

Sharing images is this going to be a pattern for all services?

[dprince]

/hold

As discussed with Dan, I am not really comfortable of having Octavia and Telemetry sharing the same image, as that could mean that the image gets updated by one of the teams unkowingly to the other, and that can cause issues.

Sharing images is this going to be a pattern for all services?

My viewpoint on this is it is an Apache image (pretty basic service) so probably okay to share it right? And if down the road we need to isolate and have multiple settings referring to different container images so be it... we can.

One more detail, the HTTP image would actually get used in 3 places: baremetal-operator(the original reason we have it), telemetry-operator, and then octavia as well.

[dprince]

@jlarriba code here looks good. Aside from your question I am LGTM to merge this.

[openshift-ci[bot]]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dprince, jlarriba

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

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/openstack-k8s-operators/openstack-operator/blob/main/OWNERS)~~ [dprince] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment

[jlarriba]

/unhold