-
Notifications
You must be signed in to change notification settings - Fork 437
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
feat: Expose initial_metadata in GrpcHealthCheck #8511
feat: Expose initial_metadata in GrpcHealthCheck #8511
Conversation
a78bd04
to
a626dc8
Compare
171d379
to
009db75
Compare
Issues linked to changelog: |
projects/gloo/api/external/envoy/api/v2/core/health_check.proto
Outdated
Show resolved
Hide resolved
// Specifies a list of key-value pairs that should be added to the metadata of each GRPC call | ||
// that is sent to the health checked cluster. | ||
repeated .solo.io.envoy.api.v2.core.HeaderValueOption initial_metadata = 3 | ||
[(validate.rules).repeated .max_items = 1000]; |
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.
Similar to http health check
https://github.com/solo-io/gloo/blob/main/projects/gloo/api/external/envoy/api/v2/core/health_check.proto#L122
Visit the preview URL for this PR (updated for commit 78932d2): https://gloo-edge--pr8511-expose-initialmetada-8qa4vxzw.web.app (expires Mon, 07 Aug 2023 19:22:44 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 77c2b86e287749579b7ff9cadb81e099042ef677 |
Co-authored-by: Nathan Fudenberg <nathan.fudenberg@solo.io>
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 if we can definitively resolve the question about protected metadata fields
47ed3cd
to
78932d2
Compare
* feat: Expose initial_metadata in GrpcHealthCheck * Trigger CI * Fix uneven whitespace Co-authored-by: Nathan Fudenberg <nathan.fudenberg@solo.io> * Adding changelog file to new location * Avoid code duplcation * Removing duplicate changelog * fix test * Add check for forbidden custom headers --------- Co-authored-by: David Jumani <djumani@Solo-System-DJumani.local> Co-authored-by: soloio-bulldozer[bot] <48420018+soloio-bulldozer[bot]@users.noreply.github.com> Co-authored-by: Nathan Fudenberg <nathan.fudenberg@solo.io> Co-authored-by: changelog-bot <changelog-bot>
* feat: Expose initial_metadata in GrpcHealthCheck (#8511) * feat: Expose initial_metadata in GrpcHealthCheck * Trigger CI * Fix uneven whitespace Co-authored-by: Nathan Fudenberg <nathan.fudenberg@solo.io> * Adding changelog file to new location * Avoid code duplcation * Removing duplicate changelog * fix test * Add check for forbidden custom headers --------- Co-authored-by: David Jumani <djumani@Solo-System-DJumani.local> Co-authored-by: soloio-bulldozer[bot] <48420018+soloio-bulldozer[bot]@users.noreply.github.com> Co-authored-by: Nathan Fudenberg <nathan.fudenberg@solo.io> Co-authored-by: changelog-bot <changelog-bot> * Move changelog * Update changelog --------- Co-authored-by: David Jumani <djumani@Solo-System-DJumani.local> Co-authored-by: soloio-bulldozer[bot] <48420018+soloio-bulldozer[bot]@users.noreply.github.com> Co-authored-by: Nathan Fudenberg <nathan.fudenberg@solo.io>
Description
Exposes the
initial_metadata
field inGrpcHealthCheck
Context
#8490
Checklist:
make -B install-go-tools generated-code
to ensure there will be no code diffBOT NOTES:
resolves #8490