Skip to content

Revert identity name discovery#922

Draft
vyzigold wants to merge 2 commits into
openstack-k8s-operators:mainfrom
vyzigold:revert-identity-name-discovery
Draft

Revert identity name discovery#922
vyzigold wants to merge 2 commits into
openstack-k8s-operators:mainfrom
vyzigold:revert-identity-name-discovery

Conversation

@vyzigold

Copy link
Copy Markdown
Contributor

We discovered, that ceilometer version we use has a bug, where it tries to reach Keystone through the public endpoint no matter what's the configuration when performing the identity name discovery. We can't assume the public service endpoints are reachable from the compute nodes. So having identity_name_discovery=True started causing issues in environments without access to public endpoint from the computes. This caused ceilometer compute agent to completely fail and not produce any metrics. We'll need to revert these 2 commits until ceilometer is fixed to adhere to its configuration, so that we can unblock further development of other features.

@openshift-ci openshift-ci Bot requested review from dprince and mgirgisf June 23, 2026 16:56
@openshift-ci

openshift-ci Bot commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: vyzigold

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

@vyzigold vyzigold marked this pull request as draft June 23, 2026 17:17
@vyzigold

Copy link
Copy Markdown
Contributor Author

/retest

@vyzigold vyzigold marked this pull request as ready for review June 23, 2026 17:27
@openshift-ci openshift-ci Bot requested a review from jlarriba June 23, 2026 17:27
@openshift-cherrypick-robot

Copy link
Copy Markdown

@vyzigold: once the present PR merges, I will cherry-pick it on top of 18.0-fr5 in a new PR and assign it to you.

Details

In response to this:

/cherry-pick 18.0-fr5

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.

@vyzigold vyzigold marked this pull request as draft June 23, 2026 17:32
@vyzigold

Copy link
Copy Markdown
Contributor Author

/retest

@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/299cc2e8937d4c9db979636c3dc9c600

✔️ telemetry-openstack-meta-content-provider-master SUCCESS in 2h 35m 29s
✔️ telemetry-operator-multinode-cloudkitty SUCCESS in 1h 50m 59s
✔️ telemetry-openstack-meta-content-provider-all-services-master SUCCESS in 2h 19m 29s
✔️ telemetry-operator-multinode-master SUCCESS in 1h 33m 46s
openstack-k8s-operators-content-provider FAILURE in 8m 28s
⚠️ telemetry-operator-multinode-default-telemetry SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
⚠️ telemetry-operator-multinode-audit-logging SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
✔️ functional-tests-osp18 SUCCESS in 2h 09m 25s

@vyzigold

Copy link
Copy Markdown
Contributor Author

recheck failed image pull

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