Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add CeilometerProxy image #758

Conversation

vyzigold
Copy link
Contributor

The ceilometer proxy image (which is used for TLS termination) is missing.

rh-jira: https://issues.redhat.com/browse/OSPRH-6400

@openshift-ci openshift-ci bot requested review from dprince and viroel April 17, 2024 07:34
@vyzigold
Copy link
Contributor Author

/hold

@vyzigold
Copy link
Contributor Author

I want to check the CI and confirm, that the image is being set correctly before "unholding".

Copy link

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://review.rdoproject.org/zuul/buildset/a82b7c4ddf8d49e5b99bd751d0a21572

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 47m 01s
podified-multinode-edpm-deployment-crc FAILURE in 1h 01m 51s
cifmw-crc-podified-edpm-baremetal FAILURE in 1h 12m 44s
openstack-operator-tempest-multinode FAILURE in 1h 07m 13s

@vyzigold
Copy link
Contributor Author

After the following PR is merged, it'll be worth to give the CI another try. The image should appear: #771

@vyzigold
Copy link
Contributor Author

recheck
Let's see if it appears

@vyzigold
Copy link
Contributor Author

/retest

@vyzigold
Copy link
Contributor Author

/unhold

@vyzigold
Copy link
Contributor Author

The new image is there and it's being used in the telemetry CR created by the openstack-operator. CI is passing, this is ready for reviews.

Copy link
Contributor

@stuggi stuggi left a comment

Choose a reason for hiding this comment

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

/lgtm

Copy link
Contributor

openshift-ci bot commented Apr 26, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: stuggi, vyzigold

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:

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

@vyzigold
Copy link
Contributor Author

/retest
Swift isn't ready. Swift-proxy pod events:

  Normal   Started         20m (x2 over 20m)    kubelet            Started container proxy-httpd
  Normal   Pulled          20m (x2 over 20m)    kubelet            Container image "quay.io/podified-antelope-centos9/openstack-swift-proxy-server@sha256:d28bdd9e1119fd420576c07f00ab44618f16b68117d51c3c9c3a42472e007a00" already present on machine
  Warning  BackOff         20m (x5 over 20m)    kubelet            Back-off restarting failed container proxy-server in pod swift-proxy-5f666c98d7-2sxz6_openstack(4ad23809-7cdb-4a74-a394-822415dc2175)
  Warning  Unhealthy       20m (x3 over 20m)    kubelet            Liveness probe failed: HTTP probe failed with statuscode: 503
  Warning  Unhealthy       20m (x4 over 20m)    kubelet            Readiness probe failed: HTTP probe failed with statuscode: 503
  Normal   Killing         20m                  kubelet            Container proxy-httpd failed liveness probe, will be restarted
  Warning  BackOff         46s (x102 over 20m)  kubelet            Back-off restarting failed container proxy-httpd in pod swift-proxy-5f666c98d7-2sxz6_openstack(4ad23809-7cdb-4a74-a394-822415dc2175)

@openshift-merge-bot openshift-merge-bot bot merged commit ce2e369 into openstack-k8s-operators:main Apr 26, 2024
7 checks passed
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