Skip to content

Backport of Add a flag for enabling debug logs to the connect envoy command into release/1.12.x#16119

Merged
kyhavlov merged 1 commit intorelease/1.12.xfrom
backport/envoy-bootstrap-logging/factually-brave-bass
Feb 1, 2023
Merged

Backport of Add a flag for enabling debug logs to the connect envoy command into release/1.12.x#16119
kyhavlov merged 1 commit intorelease/1.12.xfrom
backport/envoy-bootstrap-logging/factually-brave-bass

Conversation

@hc-github-team-consul-core
Copy link
Copy Markdown
Collaborator

Backport

This PR is auto-generated from #15988 to be assessed for backporting due to the inclusion of the label backport/1.12.

WARNING automatic cherry-pick of commits failed. Commits will require human attention.

merge conflict error: POST https://api.github.com/repos/hashicorp/consul/merges: 409 Merge conflict []

The below text is copied from the body of the original PR.


This PR adds a flag, -enable-config-gen-logging, to the connect envoy command to display some log messages when generating the bootstrap config. It can be hard to troubleshoot slowness that happens during the multiple api requests during this command due to the current lack of logs, so this flag enables outputting some messages (on stderr) to help with that.


Overview of commits

@hashicorp-cla
Copy link
Copy Markdown

hashicorp-cla commented Jan 31, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Auto approved Consul Bot automated PR

@kyhavlov kyhavlov force-pushed the backport/envoy-bootstrap-logging/factually-brave-bass branch from 8659270 to 66ab129 Compare February 1, 2023 01:54
@kyhavlov kyhavlov marked this pull request as ready for review February 1, 2023 02:21
@kyhavlov kyhavlov requested a review from a team as a code owner February 1, 2023 02:21
…5988)

* Add a flag for enabling debug logs to the `connect envoy` command

* Update website/content/commands/connect/envoy.mdx

Co-authored-by: Jeff Boruszak <104028618+boruszak@users.noreply.github.com>

* Add changelog note

* Add debug log note to envoy proxy doc page

* Update website/content/docs/connect/proxies/envoy.mdx

Co-authored-by: Kendall Strautman <36613477+kendallstrautman@users.noreply.github.com>

* Wording tweak in envoy bootstrap section

---------

Co-authored-by: Jeff Boruszak <104028618+boruszak@users.noreply.github.com>
Co-authored-by: Kendall Strautman <36613477+kendallstrautman@users.noreply.github.com>
@kyhavlov kyhavlov force-pushed the backport/envoy-bootstrap-logging/factually-brave-bass branch from 66ab129 to 5076ced Compare February 1, 2023 02:49
@kyhavlov kyhavlov merged commit 563d044 into release/1.12.x Feb 1, 2023
@kyhavlov kyhavlov deleted the backport/envoy-bootstrap-logging/factually-brave-bass branch February 1, 2023 02:59
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.

4 participants