Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Initial Gateway API Conformance #65

Closed
danehans opened this issue May 13, 2022 · 11 comments
Closed

Initial Gateway API Conformance #65

danehans opened this issue May 13, 2022 · 11 comments
Labels
help wanted Extra attention is needed release-note Indicates a required release note
Milestone

Comments

@danehans
Copy link
Contributor

danehans commented May 13, 2022

The community has established Gateway API conformance as the target of the 0.0.2 release. This is a meta issue, in that issues, PRs, etc. that target this goal should be linked to this issue.

@danehans danehans added the help wanted Extra attention is needed label May 13, 2022
@danehans danehans added this to the 0.0.2 milestone May 13, 2022
@danehans danehans changed the title Gateway API Conformance Initial Gateway API Conformance May 13, 2022
@danehans
Copy link
Contributor Author

Since conformance tests are a moving target and being fully conformant may be a considerable undertaking, we should identify a subset of tests that are the v0.2.0 target.

xref: #11 (comment)
xref conformance tests: https://github.com/kubernetes-sigs/gateway-api/tree/master/conformance

@youngnick feel free to provide input.

@youngnick
Copy link
Contributor

Yes, I think we should pick a Gateway API release as our first milestone target, we could choose either v0.4.3, the final v1alpha2 release, or the forthcoming v0.5.0, the first v1beta1 release. I'm inclined toward the latter.

@danehans
Copy link
Contributor Author

@skasisom6 the v0.2.0 goal is for Envoy Gateway to pass Gateway API conformance tests. This means that Envoy Gateway will provide a set of core API Gateway functionality. When you have a moment, please review Gateway API conformance, along with the conformance tests to learn more about conformance. Let us know if these features are not consistent with your requirements for the release.

1 similar comment
@danehans
Copy link
Contributor Author

@skasisom6 the v0.2.0 goal is for Envoy Gateway to pass Gateway API conformance tests. This means that Envoy Gateway will provide a set of core API Gateway functionality. When you have a moment, please review Gateway API conformance, along with the conformance tests to learn more about conformance. Let us know if these features are not consistent with your requirements for the release.

@skriss
Copy link
Contributor

skriss commented Jul 15, 2022

I can work on getting the Gateway API conformance tests running in CI so as soon as we have something integrated, we can start running tests against it. I'll file a specific issue for it.

@danehans
Copy link
Contributor Author

v0.2.0 will focus on httproute conformance, e.g. not tlsroute, tcproute, etc. @kflynn we should make sure the docs state this.

xref #17

@skriss
Copy link
Contributor

skriss commented Sep 6, 2022

Large blockers for getting Gateway API conformance tests to run:

Specific "core" conformance features still to be implemented:

It's although worth noting that the Gateway API conformance tests are still a work in progress -- not every feature tagged as "core" in the API spec has tests yet, but they are being continually added. So -- we should try to get all the core features implemented, whether they currently have a conformance test or not, so we're not behind the ball when corresponding tests do get added.

@danehans danehans added the release-note Indicates a required release note label Sep 8, 2022
@danehans
Copy link
Contributor Author

danehans commented Oct 3, 2022

Note that #168 and #411 are not required to pass conformance tests.

@danehans
Copy link
Contributor Author

danehans commented Oct 3, 2022

@danehans
Copy link
Contributor Author

2 remaining referencegrant tests should pass when #588 is fixed.

@danehans
Copy link
Contributor Author

Closing since all core conformance tests are passing. All but one RefernceGrant test (tracked by #539) is passing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed release-note Indicates a required release note
Projects
None yet
Development

No branches or pull requests

3 participants