Skip to content

Conversation

HirazawaUi
Copy link
Contributor

  • One-line PR description: Promote the HostnameOverride feature gate to beta stage
  • Other comments:

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Oct 8, 2025
@k8s-ci-robot k8s-ci-robot added kind/kep Categorizes KEP tracking issues and PRs modifying the KEP directory sig/network Categorizes an issue or PR as relevant to SIG Network. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Oct 8, 2025
Copy link
Contributor

@soltysh soltysh left a comment

Choose a reason for hiding this comment

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

Multiple PRR beta questions are missing answers.

@HirazawaUi HirazawaUi force-pushed the promote-4762-to-beta branch from e8c9f20 to 8ac5de9 Compare October 14, 2025 15:09
@aojea
Copy link
Member

aojea commented Oct 15, 2025

/lgtm
/approve

sig network (modulo the standing comments from PRR and the clarifications)

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 15, 2025
@HirazawaUi HirazawaUi force-pushed the promote-4762-to-beta branch from 8ac5de9 to efe3d81 Compare October 15, 2025 13:52
@k8s-ci-robot k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 15, 2025
Copy link
Member

@SergeyKanzhelev SergeyKanzhelev left a comment

Choose a reason for hiding this comment

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

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 15, 2025
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: aojea, HirazawaUi, SergeyKanzhelev
Once this PR has been reviewed and has the lgtm label, please assign deads2k for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@HirazawaUi
Copy link
Contributor Author

/assign @deads2k @thockin for approval

@aojea
Copy link
Member

aojea commented Oct 16, 2025

Multiple PRR beta questions are missing answers.

@HirazawaUi you need @soltysh PRR approval

@HirazawaUi
Copy link
Contributor Author

@HirazawaUi you need @soltysh PRR approval

Ah, thank you for the reminder. I'm responsible for multiple KEPs in this cycle and got confused. I mistakenly thought this KEP had already received PRR approval.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/kep Categorizes KEP tracking issues and PRs modifying the KEP directory lgtm "Looks good to me", indicates that a PR is ready to be merged. sig/network Categorizes an issue or PR as relevant to SIG Network. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants