Skip to content

internal/provider/kubernetes: add HTTPRoute controller#179

Merged
skriss merged 1 commit intoenvoyproxy:mainfrom
skriss:pr-httproute-controller
Aug 3, 2022
Merged

internal/provider/kubernetes: add HTTPRoute controller#179
skriss merged 1 commit intoenvoyproxy:mainfrom
skriss:pr-httproute-controller

Conversation

@skriss
Copy link
Copy Markdown
Contributor

@skriss skriss commented Jul 29, 2022

Adds an initial version of the HTTPRoute
controller to the Kubernetes provider.
Integration with the message service/the
IR to follow.

Closes #163.

Signed-off-by: Steve Kriss [email protected]

There's not really anything to test yet, so deferred adding tests until things start getting integrated.

@skriss skriss requested a review from a team as a code owner July 29, 2022 15:11
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 29, 2022

Codecov Report

Merging #179 (06517c5) into main (c83b7b0) will decrease coverage by 0.79%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##             main     #179      +/-   ##
==========================================
- Coverage   61.74%   60.94%   -0.80%     
==========================================
  Files          20       21       +1     
  Lines        1814     1854      +40     
==========================================
+ Hits         1120     1130      +10     
- Misses        632      659      +27     
- Partials       62       65       +3     
Impacted Files Coverage Δ
internal/provider/kubernetes/kubernetes.go 61.11% <0.00%> (-5.56%) ⬇️
internal/provider/kubernetes/httproute.go 27.02% <27.02%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us.

Adds an initial version of the HTTPRoute
controller to the Kubernetes provider.
Integration with the message service/the
IR to follow.

Closes envoyproxy#163.

Signed-off-by: Steve Kriss <[email protected]>
@skriss skriss force-pushed the pr-httproute-controller branch from 732ee41 to 06517c5 Compare August 2, 2022 15:25
@skriss skriss merged commit f692761 into envoyproxy:main Aug 3, 2022
@skriss skriss deleted the pr-httproute-controller branch August 3, 2022 14:59
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 HTTPRoute Controller

3 participants