Skip to content

Move diagnostics server running outside of manager #7285

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

Open
2 of 4 tasks
czeslavo opened this issue Mar 26, 2025 · 0 comments
Open
2 of 4 tasks

Move diagnostics server running outside of manager #7285

czeslavo opened this issue Mar 26, 2025 · 0 comments
Labels
area/maintenance Cleanup, refactoring, and other maintenance improvements that don't change functionality. release/not-required this is not a hard requirement for the release, it can be considered a "nice to have"
Milestone

Comments

@czeslavo
Copy link
Contributor

Is there an existing issue for this?

  • I have searched the existing issues

Does this enhancement require public documentation?

  • I have added an Acceptance Criteria item for adding and/or adjusting public documentation (if applicable)

Problem Statement

To make it possible to not run the diagnostics server as part of the manager, we've added DisableRunningDiagnosticsServer flag to the Config struct. It's needed in KGO so that we can run a manager without the diagnostics server as otherwise it wouldn't be possible to run multiple instances of the manager.

The goal of this issue is to remove this flag and move the logic responsible for running the diagnostics server outside of the manager (from

to
health.NewHealthCheckServer(
).

Proposed Solution

Move code that runs the diagnostics server out of manager.Manager. It can be done similarly to the health check server that's run in

health.NewHealthCheckServer(
.

Additional information

No response

Acceptance Criteria

  • DisableRunningDiagnosticsServer in Config struct is removed
  • By default, manager.Manager is run without a diagnostics server
@czeslavo czeslavo added the area/maintenance Cleanup, refactoring, and other maintenance improvements that don't change functionality. label Mar 26, 2025
@czeslavo czeslavo added this to the KO v2.0.0 milestone Mar 26, 2025
@czeslavo czeslavo added the release/not-required this is not a hard requirement for the release, it can be considered a "nice to have" label Apr 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/maintenance Cleanup, refactoring, and other maintenance improvements that don't change functionality. release/not-required this is not a hard requirement for the release, it can be considered a "nice to have"
Projects
None yet
Development

No branches or pull requests

1 participant