Skip to content

feat: use zap.AtomicLevel for dynamic logging levels#26182

Merged
davidby-influx merged 3 commits intomaster-1.xfrom
DSB_zap_atomiclevel
Apr 17, 2025
Merged

feat: use zap.AtomicLevel for dynamic logging levels#26182
davidby-influx merged 3 commits intomaster-1.xfrom
DSB_zap_atomiclevel

Conversation

@davidby-influx
Copy link
Copy Markdown
Contributor

Use the zap.AtomicLevel struct for log levels
which allows the level to be changed dynamically.
Enterprise will use this feature

@davidby-influx davidby-influx marked this pull request as ready for review April 16, 2025 23:43
Copy link
Copy Markdown
Member

@gwossum gwossum left a comment

Choose a reason for hiding this comment

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

LGTM

@davidby-influx davidby-influx merged commit 53329a3 into master-1.x Apr 17, 2025
9 checks passed
@davidby-influx davidby-influx deleted the DSB_zap_atomiclevel branch April 17, 2025 17:21
devanbenz pushed a commit that referenced this pull request Sep 23, 2025
Use the zap.AtomicLevel struct for log levels
which allows the level to be changed dynamically.
Enterprise will use this feature.

(cherry picked from commit 53329a3)
devanbenz added a commit that referenced this pull request Sep 23, 2025
Co-authored-by: davidby-influx <72418212+davidby-influx@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants