8000 DO NOT MERGE: initial v1.4.0 feedback by shaneutt · Pull Request #4042 · kubernetes-sigs/gateway-api · GitHub
[go: up one dir, main page]

Skip to content

Conversation

shaneutt
Copy link
Member

This is a PR to gather some early initial feedback on the upcoming changes for #3756. It should not be merged.

LiorLieberman and others added 30 commits April 24, 2025 14:42
…s#3729)

* add mesh confomance tests structure and first test

* move mesh tests to mesh folder
kubernetes-sigs#3777)

* add redirect-port and redirect scheme mesh tests

* mesh tests for path host and status redirect
Signed-off-by: Norwin Schnyder <norwin.schnyder+github@gmail.com>
* Add conformance report for Contour 1.31.0

Supports gateway API 1.2.1

Signed-off-by: Sunjay Bhatia <sunjay.bhatia@broadcom.com>

* update implementations.md

Signed-off-by: Sunjay Bhatia <sunjay.bhatia@broadcom.com>

---------

Signed-off-by: Sunjay Bhatia <sunjay.bhatia@broadcom.com>
…3781)

* add csm mesh to implementations.md

* fix

* fix1

* Update implementations.md

Co-authored-by: Rob Scott <rob.scott87@gmail.com>

---------

Co-authored-by: Rob Scott <rob.scott87@gmail.com>
…igs#3564)

Add feature name processing to remove redundant prefixes and improve
readability in conformance comparison tables. Changes include:

- Remove "HTTPRoute" and "Gateway" prefixes from feature names
- Split camelCase words into space-separated words
- Add process_feature_name() function for consistent text processing
- Update generate_profiles_report() to use processed feature names

This makes the conformance reports easier to read
Signed-off-by: bitliu <bitliu@tencent.com>
The report is generated by Cilium GHA CI

https://github.com/cilium/cilium/actions/runs/15111523465

Signed-off-by: Tam Mach <sayboras@yahoo.com>
…igs#3809)

* conformance: Add Airlock Microgateway report for v1.3.0

Signed-off-by: Norwin Schnyder <norwin.schnyder+github@gmail.com>

* add trailing new lines

Signed-off-by: Norwin Schnyder <norwin.schnyder+github@gmail.com>

---------

Signed-off-by: Norwin Schnyder <norwin.schnyder+github@gmail.com>
…#3811)

Signed-off-by: Norwin Schnyder <norwin.schnyder+github@gmail.com>
* conformance: add Hook in ConformanceTestSuite

* update comment

Signed-off-by: zirain <zirain2009@gmail.com>

* add Hook in ConformanceOptions

Signed-off-by: zirain <zirain2009@gmail.com>

---------

Signed-off-by: zirain <zirain2009@gmail.com>
)

This PR removes the Dependabot configuration for the `github-actions` package ecosystem since Github Actions are not used in this repository. There is no PR ever created with a `dependencies` tag which is not for `go`, `docker`, or `python`.
* add mesh conformance for request header modifier

* address feedback
* docs: Add v1.3 conformance report table

Signed-off-by: Norwin Schnyder <norwin.schnyder+github@gmail.com>

* Automatically update conformance table files

Signed-off-by: Norwin Schnyder <norwin.schnyder+github@gmail.com>

---------

Signed-off-by: Norwin Schnyder <norwin.schnyder+github@gmail.com>
Signed-off-by: Shane Utt <shaneutt@linux.com>
…-sigs#3827)

* add httproute weights mesh conformance tests

* gofmt
…#2712)

* draft dns configuration for gateway API GEP-2627

minor tweaks

Update geps/gep-2627/index.md

Co-authored-by: Candace Holman <candita@users.noreply.github.com>

Update geps/gep-2627/index.md

Co-authored-by: Candace Holman <candita@users.noreply.github.com>

Update geps/gep-2627/index.md

Co-authored-by: Candace Holman <candita@users.noreply.github.com>

* changes post review

* rewording

* Update geps/gep-2627/index.md

Co-authored-by: Nick Young <inocuo@gmail.com>

* Update geps/gep-2627/index.md

Co-authored-by: Nick Young <inocuo@gmail.com>

* Update geps/gep-2627/index.md

Co-authored-by: Nick Young <inocuo@gmail.com>

* Update geps/gep-2627/index.md

Co-authored-by: Nick Young <inocuo@gmail.com>

* Update geps/gep-2627/index.md

Co-authored-by: Shane Utt <shane@shaneutt.com>

* Update geps/gep-2627/index.md

Co-authored-by: Shane Utt <shane@shaneutt.com>

* Update geps/gep-2627/index.md

Co-authored-by: Shane Utt <shane@shaneutt.com>

* Update geps/gep-2627/index.md

Co-authored-by: Shane Utt <shane@shaneutt.com>

* Update geps/gep-2627/index.md

Co-authored-by: Shane Utt <shane@shaneutt.com>

* Update geps/gep-2627/index.md

Co-authored-by: Shane Utt <shane@shaneutt.com>

* Update geps/gep-2627/index.md

Co-authored-by: Shane Utt <shane@shaneutt.com>

* Update geps/gep-2627/index.md

Co-authored-by: Shane Utt <shane@shaneutt.com>

* minor tweaks to the text and link to kuadrant as an example of a DNSPolicy type API

* fix new line

* move kuadrant to a reference

---------

Co-authored-by: Candace Holman <candita@users.noreply.github.com>
Co-authored-by: Nick Young <inocuo@gmail.com>
Co-authored-by: Shane Utt <shane@shaneutt.com>
Signed-off-by: Keith Mattix II <keithmattix@microsoft.com>
* Kubevernor conformance report

* Adding Kubvernor section to implementations section
Adding conformance report for the v2.0 NGINX Gateway Fabric release. Now supporting Gateway API v1.3.
@k8s-ci-robot
Copy link
Contributor

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

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.

@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. do-not-merge/invalid-commit-message Indicates that a PR should not merge because it has an invalid commit message. labels Aug 28, 2025
@k8s-ci-robot
Copy link
Contributor

Keywords which can automatically close issues and at(@) or hashtag(#) mentions are not allowed in commit messages.

The list of commits with invalid commit messages:

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. I understand the commands that are listed here.

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: shaneutt

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

@k8s-ci-robot k8s-ci-robot added the kind/gep PRs related to Gateway Enhancement Proposal(GEP) label Aug 28, 2025
@k8s-ci-robot k8s-ci-robot added approved Indicates a PR has been approved by an approver from all required OWNERS files. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Aug 28, 2025
@shaneutt shaneutt requested review from aojea, bowei, danwinship, mlavacca, robscott and youngnick and removed request for LiorLieberman and howardjohn August 28, 2025 12:28
@shaneutt shaneutt moved this to Review in Release v1.4.0 Aug 28, 2025
@shaneutt
Copy link
Member Author
shaneutt commented Oct 3, 2025

After feedback and testing with an RC1 and RC2, just a note that we're intending to release the final release Monday, October 6th.

@shaneutt shaneutt closed this Oct 6, 2025
@github-project-automation github-project-automation bot moved this from Review to Done in Release v1.4.0 Oct 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. do-not-merge/invalid-commit-message Indicates that a PR should not merge because it has an invalid commit message. do-not-merge/release-note-label-needed Indicates that a PR should not merge because it's missing one of the release note labels. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. do-not-merge DEPRECATED. Indicates that a PR should not merge. Label can only be manually applied/removed. kind/gep PRs related to Gateway Enhancement Proposal(GEP) size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

0