Skip to content
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

release log gate if disable-gated-logs flag is set #24280

Merged
merged 4 commits into from
Dec 11, 2023

Conversation

hghaf099
Copy link
Contributor

@hghaf099 hghaf099 commented Nov 28, 2023

Vault wouldn’t output any logs at all on startup right now as the logs are gated to be flushed until after lot of initialization is done. This PR introduces a hidden flag which is mostly used for troubleshooting that allows to flush the logs as they become available.
Addresses https://hashicorp.atlassian.net/browse/VAULT-20508

@github-actions github-actions bot added the hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed label Nov 28, 2023
Copy link

Build Results:
All builds succeeded! ✅

Copy link

CI Results:
All Go tests succeeded! ✅

Copy link
Contributor

@raskchanky raskchanky left a comment

Choose a reason for hiding this comment

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

This seems like a challenging thing to write tests for, but I'm assuming you tested this in a terminal?

changelog/24280.txt Outdated Show resolved Hide resolved
Co-authored-by: Josh Black <raskchanky@gmail.com>
@hghaf099
Copy link
Contributor Author

hghaf099 commented Dec 4, 2023

This seems like a challenging thing to write tests for, but I'm assuming you tested this in a terminal?

68747470733a2f2f6d65646961312e67697068792e636f6d2f6d656469612f336f4564754d33375065564b536b6d736a432f67697068792e676966

Yes, I tried it manually in the terminal.

@hghaf099 hghaf099 merged commit 57b6b74 into main Dec 11, 2023
114 of 118 checks passed
@hghaf099 hghaf099 deleted the controlled-gated-logger-flush branch December 11, 2023 16:08
hghaf099 added a commit that referenced this pull request Dec 11, 2023
* release log gate if disable-gated-logs flag is set

* CL

* Update changelog/24280.txt

Co-authored-by: Josh Black <raskchanky@gmail.com>

---------

Co-authored-by: Peter Wilson <peter.wilson@hashicorp.com>
Co-authored-by: Josh Black <raskchanky@gmail.com>
hghaf099 added a commit that referenced this pull request Dec 11, 2023
* release log gate if disable-gated-logs flag is set

* CL

* Update changelog/24280.txt



---------

Co-authored-by: Peter Wilson <peter.wilson@hashicorp.com>
Co-authored-by: Josh Black <raskchanky@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants