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.

Sign up for GitHub

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

Jump to bottom

Configurable Control Plane Log-Level #554

Closed
ja20222 opened this issue Apr 5, 2023 · 2 comments · Fixed by #943
Closed

Configurable Control Plane Log-Level #554

ja20222 opened this issue Apr 5, 2023 · 2 comments · Fixed by #943
Assignees
@sjberman
Labels
area/control-plane General control plane issues enhancement New feature or request refined Requirements are refined and the issue is ready to be implemented. size/medium Estimated to be completed within a week
Milestone
v0.6.0

Comments

@ja20222
Copy link

ja20222 commented Apr 5, 2023 •
edited by mpstefan
Loading

As a user of NKG
I want the ability to assign a log-level at startup of the control plane at runtime without restarting
So that I can select the level of logs that I want for my environment
And so that I do not lose the state of what is causing the issue.

Acceptance:

Control plane container:

Allow setting log-level at startup OR during runtime without NGINX reload
Document and define supported log-levels

Allow runtime configuration of log-level; raising and lowering the level dynamically

Enhancement Proposal: #928
@pleshakov pleshakov added enhancement New feature or request blocked Blocked by other issue area/control-plane General control plane issues labels Apr 7, 2023
@pleshakov pleshakov added this to the v1.0.0 milestone Apr 7, 2023
@mpstefan mpstefan modified the milestones: v1.0.0, v0.6.0 Jun 5, 2023
@mpstefan mpstefan changed the title Support: Provide way to set log-level Configurable Log-Level Jun 5, 2023
@mpstefan mpstefan changed the title Configurable Log-Level Configurable Control Plane Log-Level Jul 5, 2023
@mpstefan mpstefan mentioned this issue Jul 5, 2023
Closed
@mpstefan
Copy link
Collaborator

Blocked by discussion #855

@mpstefan mpstefan added enhancement-proposal Enhancement Proposal issue and removed enhancement New feature or request labels Aug 1, 2023
@sjberman sjberman self-assigned this Aug 1, 2023
@pleshakov pleshakov mentioned this issue Aug 2, 2023
Closed
This was referenced Aug 2, 2023
Merged
Closed
@mpstefan mpstefan added refined Requirements are refined and the issue is ready to be implemented. enhancement New feature or request and removed blocked Blocked by other issue enhancement-proposal Enhancement Proposal issue labels Aug 7, 2023
@mpstefan
Copy link
Collaborator

mpstefan commented Aug 7, 2023

Blocker resolved, moving forward with implementation.

@mpstefan mpstefan added the size/medium Estimated to be completed within a week label Aug 7, 2023
@sjberman sjberman mentioned this issue Aug 8, 2023
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sjberman sjberman

Labels
area/control-plane General control plane issues enhancement New feature or request refined Requirements are refined and the issue is ready to be implemented. size/medium Estimated to be completed within a week
Projects
None yet
Milestone
v0.6.0
Development

Successfully merging a pull request may close this issue.

Dynamic control plane configuration support sjberman/nginx-gateway-fabric
4 participants
@pleshakov @mpstefan @sjberman @ja20222