[github-actions]

Automated changes by create-pull-request GitHub action

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Central YAML (base), Organization UI (inherited)
• Review profile: CHILL
• Plan: Enterprise
• Run ID: 2f783900-2cb2-4cf5-b2df-b829ca65602d

📥 Commits

Reviewing files that changed from the base of the PR and between e3be6e9 and dfa131d.

⛔ Files ignored due to path filters: 2

• api/go.sum is excluded by !**/*.sum
• go.sum is excluded by !**/*.sum

📒 Files selected for processing: 4

• api/bases/horizon.openstack.org_horizons.yaml
• api/go.mod
• config/crd/bases/horizon.openstack.org_horizons.yaml
• go.mod

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 8 remain after this review.

📝 Summary

Summary by CodeRabbit

• New Features
  • Service overrides can now specify exposed ports by matching each entry to a generated Service port by name. Port values must be between 1 and 65535; other port settings, including the backend target port, remain unchanged. Entries with names that do not match a generated Service port are rejected.

The Horizon service override schemas add a named port list with values limited to 1–65535. The Go module files also update dependency versions.

Changes: Service port override schema

Layer / File(s)	Summary
Define service port overrides api/bases/horizon.openstack.org_horizons.yaml, config/crd/bases/horizon.openstack.org_horizons.yaml	Both schemas document Ports as a supported service override field and add an atomic list. Each entry requires a name and a port value from 1 to 65535. The CRD description specifies that matching is by name, only the exposed port changes, and unmatched names are rejected.

Dependency version updates

Layer / File(s)	Summary
Update dependency versions api/go.mod, go.mod	The module files update versions for infra-operator and lib-common modules. The root module also updates the keystone-operator API and lib-common openstack dependency.

Priority: ➖ Normal

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Suggested reviewers: stuggi

Merge Risk: ⚪ Minimal · up to dfa13

The service port override appears consistent with its documented behavior. No actionable merge risk is established beyond normal checks.

Pre-merge checks: 5

Check name	Status	Explanation
Title check		The title clearly identifies an automated dependency bump on the main branch, which matches the primary changes.
Description check		The description identifies the changes as automated pull request updates, which is related to the dependency and generated schema changes.
Docstring Coverage		No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (4 skipped: 4 unsupported.)

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

[openshift-ci]

Hi @github-actions[bot]. Thanks for your PR.

I'm waiting for a openstack-k8s-operators member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work.

Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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.

[centosinfra-prod-github-app]

Build succeeded (check pipeline).
https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/0304d548404a4d43bb44212906899b49

✔️ horizon-s2i-content-provider SUCCESS in 4h 42m 58s (non-voting)
❌ horizon-s2i-test FAILURE in 1h 46m 14s (non-voting)
✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 13m 01s
✔️ horizon-operator-kuttl SUCCESS in 38m 02s (non-voting)

Warning:
File ansible/tmp/dfa2011089c845f3bbeea56cdb6521e8 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job horizon-operator-kuttl. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/ede1c36bdebe4c35825eb99d044aa9bc found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job horizon-operator-kuttl. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/b26d1e48e6f4452099cf26b7f3b5dc21 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job horizon-operator-kuttl. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/37a05bdc55324b978c5a9c4c76fb357b found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job horizon-operator-kuttl. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/dfa2011089c845f3bbeea56cdb6521e8 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/multinode-autohold.yml for job horizon-operator-kuttl. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/c86451931d97447e9cda6ceb8b1186e6 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job horizon-s2i-test. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/77b4525105684d10aca0062a2c4a23d5 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job horizon-s2i-test. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/896065978a3e4852b38af89175fa0824 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job horizon-s2i-test. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/97af242f2f5a4151b1f29ff5a8422392 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job horizon-s2i-test. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/c86451931d97447e9cda6ceb8b1186e6 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/multinode-autohold.yml for job horizon-s2i-test. Add to preserve-home-paths if safe, or otherwise remove.
File .ssh/known_hosts.old found in build home directory during playbook review.rdoproject.org/config/playbooks/base-minimal/post-logs.yaml for job horizon-s2i-test. Add to preserve-home-paths if safe, or otherwise remove.
File .ssh/config found in build home directory during playbook github.com/openstack-k8s-operators/s2i-openstack-containers/playbooks/container-ci/zuul/pre.yaml for job horizon-s2i-content-provider. Add to preserve-home-paths if safe, or otherwise remove.

[stuggi]

/retest

[stuggi]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: github-actions[bot], stuggi

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:

• OWNERS [stuggi]

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