Skip to content

Conversation

@Xunzhuo
Copy link
Member

@Xunzhuo Xunzhuo commented Oct 20, 2022

Resolves: #624

image

Signed-off-by: bitliu [email protected]

@Xunzhuo Xunzhuo requested a review from a team as a code owner October 20, 2022 07:24
@Xunzhuo Xunzhuo added this to the 0.3.0-rc.1 milestone Oct 20, 2022
@codecov-commenter
Copy link

codecov-commenter commented Oct 20, 2022

Codecov Report

Merging #627 (bb390e3) into main (4ad8ff6) will decrease coverage by 0.24%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #627      +/-   ##
==========================================
- Coverage   63.34%   63.10%   -0.25%     
==========================================
  Files          47       47              
  Lines        5754     5754              
==========================================
- Hits         3645     3631      -14     
- Misses       1881     1893      +12     
- Partials      228      230       +2     
Impacted Files Coverage Δ
internal/provider/kubernetes/httproute.go 59.25% <0.00%> (-8.24%) ⬇️
internal/provider/kubernetes/gateway.go 49.68% <0.00%> (-2.56%) ⬇️
internal/provider/kubernetes/tlsroute.go 67.40% <0.00%> (+7.92%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

docs/index.rst Outdated
This project is under active development. Many, many features are not
complete. We would love for you to :doc:`get involved<get_involved>`.

This project is now listed in official
Copy link
Contributor

Choose a reason for hiding this comment

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

s/in official/as an official/

IMO this information is better conveyed on the compatibility page. Thoughts @arkodg @LukeShu @skriss @AliceProxy?

docs/index.rst Outdated
complete. We would love for you to :doc:`get involved<get_involved>`.

This project is now listed in official
`Gateway API implementations <https://gateway-api.sigs.k8s.io/implementations/#envoy-gateway>`_ .
Copy link
Contributor

Choose a reason for hiding this comment

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

s/implementations/implementation/

Signed-off-by: bitliu <[email protected]>
@arkodg
Copy link
Contributor

arkodg commented Oct 24, 2022

imho if we we are linking back to the Gateway API docs, there should be a good reason why it matters to the user

  • either we are encouraging them to check out other implementations :)
  • or the Gateway API docs has stated the level of conformance/support which Envoy Gateway provides (currently core conformance) reassuring the user of feature support.
    I think we should highlight the latter point, raised update envoy gateway status kubernetes-sigs/gateway-api#1475 to highlight that EG passes core conformance tests and we should include that verbiage here as well.

@danehans
Copy link
Contributor

danehans commented Nov 7, 2022

@Xunzhuo do you mind resolving the review feedback or closing this PR?

@Xunzhuo
Copy link
Member Author

Xunzhuo commented Nov 8, 2022

Yes @danehans, I think we can close this PR.

@Xunzhuo Xunzhuo closed this Nov 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Reference to Gateway API Implementations Page

4 participants