Skip to content

cds: add debug log when cluster is skipped#8294

Merged
zuercher merged 1 commit intoenvoyproxy:masterfrom
ramaraochavali:fix/add_debug_logs
Sep 19, 2019
Merged

cds: add debug log when cluster is skipped#8294
zuercher merged 1 commit intoenvoyproxy:masterfrom
ramaraochavali:fix/add_debug_logs

Conversation

@ramaraochavali
Copy link
Copy Markdown
Contributor

Description: Adds a debug log line when a cluster is skipped from CDS response.
Risk Level: None
Testing: N/A
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: Rama Chavali <rama.rao@salesforce.com>
@zuercher zuercher self-assigned this Sep 19, 2019
@zuercher
Copy link
Copy Markdown
Member

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

@zuercher
Copy link
Copy Markdown
Member

Github had a couple of the jobs stuck in the queued state, but Azure that they succeeded. Hopefully the re-run clears this up.

Copy link
Copy Markdown
Member

@zuercher zuercher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks.

@zuercher zuercher merged commit 3c4e5bf into envoyproxy:master Sep 19, 2019
@ramaraochavali ramaraochavali deleted the fix/add_debug_logs branch September 20, 2019 02:35
danzh2010 pushed a commit to danzh2010/envoy that referenced this pull request Sep 24, 2019
Adds a debug log line when a cluster is skipped from CDS response.

Risk Level: Low (debug logging)
Testing: N/A
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: Rama Chavali <rama.rao@salesforce.com>
danzh2010 pushed a commit to danzh2010/envoy that referenced this pull request Oct 4, 2019
Adds a debug log line when a cluster is skipped from CDS response.

Risk Level: Low (debug logging)
Testing: N/A
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: Rama Chavali <rama.rao@salesforce.com>
danzh2010 pushed a commit to danzh2010/envoy that referenced this pull request Oct 4, 2019
Adds a debug log line when a cluster is skipped from CDS response.

Risk Level: Low (debug logging)
Testing: N/A
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: Rama Chavali <rama.rao@salesforce.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants