-
Notifications
You must be signed in to change notification settings - Fork 674
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
Add conditions to HTTPProxy and TLSCertificateDelegation #2706
Add conditions to HTTPProxy and TLSCertificateDelegation #2706
Conversation
b756734
to
8ce7308
Compare
Codecov Report
@@ Coverage Diff @@
## master #2706 +/- ##
==========================================
- Coverage 76.93% 76.90% -0.04%
==========================================
Files 72 72
Lines 5715 5715
==========================================
- Hits 4397 4395 -2
- Misses 1227 1228 +1
- Partials 91 92 +1
|
5b7bec3
to
b288839
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, looks great. Just a few nits, and I'd recommend using make site-devel
to view the generated API reference and reviewing that it reads and formats correctly.
0c0ae92
to
0bbee0b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Nice additional doc additions.
Updates projectcontour#2495. This adds a new `DetailedCondition` type to HTTPProxy, that allows for sub-conditions. This allows Contour to have a `Valid` condition, that has further information about how that conditions `status` has been reached, or in other words, to expose more than one error or warning about a HTTPProxy to the user. Also adds the Conditions block to TLSCertificateDelegation. Signed-off-by: Nick Young <ynick@vmware.com>
0bbee0b
to
a44ed82
Compare
Signed-off-by: Nick Young <ynick@vmware.com>
a44ed82
to
1c68384
Compare
…our#2706) Updates projectcontour#2495. This adds a new `DetailedCondition` type to HTTPProxy, that allows for sub-conditions. This allows Contour to have a `Valid` condition, that has further information about how that conditions `status` has been reached, or in other words, to expose more than one error or warning about a HTTPProxy to the user. Also adds the Conditions block to TLSCertificateDelegation. Signed-off-by: Nick Young <ynick@vmware.com>
Updates #2495.
This adds a new
DetailedCondition
type to HTTPProxy, that allows for sub-conditions.This allows Contour to have a
Valid
condition, that has further information abouthow that conditions
status
has been reached, or in other words, to expose morethan one error or warning about a HTTPProxy to the user.
Also adds the Conditions block to TLSCertificateDelegation.
Signed-off-by: Nick Young ynick@vmware.com