[github-actions]

Automated changes by create-pull-request GitHub action

[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.

[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: 98990f8d-f1e0-4519-974d-a28ec1ae1de7

📥 Commits

Reviewing files that changed from the base of the PR and between 293675e and 5d3f55a.

⛔ Files ignored due to path filters: 2

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

📒 Files selected for processing: 12

• api/bases/designate.openstack.org_designateapis.yaml
• api/bases/designate.openstack.org_designatebackendbind9s.yaml
• api/bases/designate.openstack.org_designatemdnses.yaml
• api/bases/designate.openstack.org_designates.yaml
• api/bases/designate.openstack.org_designateunbounds.yaml
• api/go.mod
• config/crd/bases/designate.openstack.org_designateapis.yaml
• config/crd/bases/designate.openstack.org_designatebackendbind9s.yaml
• config/crd/bases/designate.openstack.org_designatemdnses.yaml
• config/crd/bases/designate.openstack.org_designates.yaml
• config/crd/bases/designate.openstack.org_designateunbounds.yaml
• go.mod

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

📝 Summary

Summary by CodeRabbit

• New Features
  • Service overrides for Designate components can now change exposed ports by matching a port name and providing a replacement value from 1–65535. Other port settings remain unchanged; unmatched names are rejected.

Service override schemas add named port overrides for Designate services. The schemas constrain replacement ports to 1–65535 and describe matching and preservation rules. The root and API Go modules also update dependency revisions.

Changes: Service port overrides

Layer / File(s)	Summary
Add ports to service override schemas api/bases/designate.openstack.org_designate*.yaml, config/crd/bases/designate.openstack.org_designate*.yaml	Standalone and aggregate Designate schemas add atomic ports arrays. Each entry requires a port name and a replacement port from 1 through 65535. The schema descriptions specify name matching, rejection of unmatched names, and preservation of TargetPort and other fields.

Dependency revision updates

Layer / File(s)	Summary
Update Go module revisions api/go.mod, go.mod	The module files pin newer revisions of infra-operator APIs, lib-common modules, and other dependencies.

Priority: ➖ Normal

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

Change: Feature

Suggested reviewers: stuggi

Merge Risk: ⚪ Minimal · up to 5d3f5

The port overrides appear consistent with the Service and endpoint behavior. No issue identified here needs to block merging after normal checks.

Pre-merge checks: 5

Check name	Status	Explanation
Title check		The title clearly identifies an automated dependency bump targeting the main branch, which matches the stated objective and dependency changes.
Description check		The description identifies the changes as automated pull request updates, which is related to the dependency bump changeset.
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. (12 skipped: 12 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.}

[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/3241f5ac07c5437c841e150923b948d9

❌ openstack-k8s-operators-content-provider FAILURE in 24m 25s
⚠️ designate-operator-tempest-multinode SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider

[stuggi]

recheck

[centosinfra-prod-github-app]

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

✔️ openstack-k8s-operators-content-provider SUCCESS in 3h 34m 21s
✔️ designate-operator-tempest-multinode SUCCESS in 1h 29m 12s

Warning:
File ansible/tmp/673ba683d15a449bbddacdad93aa5a05 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job designate-operator-tempest-multinode. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/e45cbdfe12174a949c53fc4b86ad7016 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/collect-logs.yml for job designate-operator-tempest-multinode. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/673ba683d15a449bbddacdad93aa5a05 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/multinode-autohold.yml for job designate-operator-tempest-multinode. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/e45cbdfe12174a949c53fc4b86ad7016 found in build home directory during playbook github.com/openstack-k8s-operators/ci-framework/ci/playbooks/multinode-autohold.yml for job designate-operator-tempest-multinode. Add to preserve-home-paths if safe, or otherwise remove.
File ansible/tmp/673ba683d15a449bbddacdad93aa5a05 found in build home directory during playbook review.rdoproject.org/config/playbooks/crc/print-machineconfigpool.yaml for job designate-operator-tempest-multinode. Add to preserve-home-paths if safe, or otherwise remove.

[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