-
Notifications
You must be signed in to change notification settings - Fork 32
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
Refer to the DNS HealthCheckSpeck from dns-operator repo #723
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #723 +/- ##
==========================================
+ Coverage 80.20% 82.56% +2.35%
==========================================
Files 64 77 +13
Lines 4492 6144 +1652
==========================================
+ Hits 3603 5073 +1470
- Misses 600 710 +110
- Partials 289 361 +72
Flags with carried forward coverage won't be shown. Click here to find out more.
|
the listeners assigned to the target gateway | ||
properties: | ||
endpoint: | ||
description: |- |
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.
it doesn't seem great to have lost the field descriptions, can they be re-added to the Spec in the DNS Operator to keep them?
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.
added those to the DNS Operator bu we will see them here only after the dns-operator PR gets merged
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.
@philbrookes now should be fine
3574261
to
94b7386
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 fine to me, maybe @philbrookes could take a quick look since its health check related.
Signed-off-by: Maskym Vavilov <mvavilov@redhat.com>
Stop redefining DNSRecord HealthCheckSpec from dns-operator repo.
Also, push to a fresher commit of the
dns-operator
so we have the latest CRD