envoyproxy/envoy

Add optional error message field to DiscoveryResponse

Open

#18,887 创建于 2021年11月3日

在 GitHub 查看
 (4 评论) (0 反应) (0 负责人)C++ (5,373 fork)batch import
area/xdshelp wanted

仓库指标

Star
 (27,997 star)
PR 合并指标
 (平均合并 8天) (30 天内合并 378 个 PR)

描述

Title: Add optional error message field to DiscoveryResponse

Description: Engineers at Square are allowed to configure their envoy's behavior through certain key/value pairs that we add to the bootstrap node metadata. Occasionally, they submit incorrect values. They may use incorrect key names or they may use sets of configuration options that are logically incorrect. We could try to identify these errors through some process that runs alongside the Envoy sidecar, but then we'd need to update that (i.e. update our fleet of envoys) every time we wanted to expose a new configuration option.

Instead, I'd like to propose that we allow adding an error message to a DiscoveryResponse. If set, Envoy will log the error message to stdout when it receives the DiscoveryResponse. This will allow our engineers to see why their sidecar is failing to become ready. My team can then try and return actionable feedback that empowers these engineers to correct their configuration mistake!

Let me know if this is reasonable or if there is another way to accomplish what I am trying to do.

贡献者指南