Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@skriss
Copy link
Member

@skriss skriss commented Feb 27, 2023

Replaces custom condition types/reasons with
upstream ones, and cleans up filter processing
for consistency.

Closes #5103.

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

Replaces custom condition types/reasons with
upstream ones, and cleans up filter processing
for consistency.

Closes projectcontour#5103.

Signed-off-by: Steve Kriss <[email protected]>
@skriss skriss force-pushed the pr-fix-route-conditions branch from 0bc03dd to f468cfb Compare February 28, 2023 20:08
@skriss skriss marked this pull request as ready for review February 28, 2023 20:08
@skriss skriss requested a review from a team as a code owner February 28, 2023 20:08
@skriss skriss requested review from stevesloka and sunjayBhatia and removed request for a team February 28, 2023 20:08
Signed-off-by: Steve Kriss <[email protected]>
@skriss skriss added the release-note/small A small change that needs one line of explanation in the release notes. label Feb 28, 2023
@codecov
Copy link

codecov bot commented Feb 28, 2023

Codecov Report

Merging #5125 (7c03197) into main (e7ff728) will decrease coverage by 0.04%.
The diff coverage is 79.50%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #5125      +/-   ##
==========================================
- Coverage   78.02%   77.98%   -0.04%     
==========================================
  Files         138      138              
  Lines       17517    17510       -7     
==========================================
- Hits        13668    13656      -12     
- Misses       3583     3588       +5     
  Partials      266      266              
Impacted Files Coverage Δ
internal/status/routeconditions.go 51.04% <ø> (ø)
internal/dag/gatewayapi_processor.go 95.09% <79.50%> (-0.41%) ⬇️

@skriss skriss merged commit c900f48 into projectcontour:main Mar 1, 2023
@skriss skriss deleted the pr-fix-route-conditions branch March 1, 2023 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-note/small A small change that needs one line of explanation in the release notes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

set condition of Accepted: false with reason UnsupportedValue for unknown HTTPRoute path match types

2 participants