Skip to content

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

Merged
kyhavlov merged 1 commit intorelease/1.14.xfrom
backport/envoy-bootstrap-logging/forcibly-steady-loon
Feb 1, 2023
Merged

Backport of Add a flag for enabling debug logs to the connect envoy command into release/1.14.x#16121
kyhavlov merged 1 commit intorelease/1.14.xfrom
backport/envoy-bootstrap-logging/forcibly-steady-loon

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.14.

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

@hc-github-team-consul-core hc-github-team-consul-core force-pushed the backport/envoy-bootstrap-logging/forcibly-steady-loon branch from 5d9ec5f to ea9b104 Compare January 31, 2023 21:31
@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

…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/forcibly-steady-loon branch from 924e138 to deaf477 Compare February 1, 2023 01:28
@kyhavlov kyhavlov marked this pull request as ready for review February 1, 2023 01:38
@kyhavlov kyhavlov requested a review from a team as a code owner February 1, 2023 01:38
@kyhavlov kyhavlov merged commit 757aab8 into release/1.14.x Feb 1, 2023
@kyhavlov kyhavlov deleted the backport/envoy-bootstrap-logging/forcibly-steady-loon branch February 1, 2023 01:39
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