Skip to content
This repository was archived by the owner on Nov 25, 2025. It is now read-only.

[Merge After #102][PATH] Add debug endpoints to PATH chart#103

Open
adshmh wants to merge 8 commits into
mainfrom
feat-path-debug-endpoints
Open

[Merge After #102][PATH] Add debug endpoints to PATH chart#103
adshmh wants to merge 8 commits into
mainfrom
feat-path-debug-endpoints

Conversation

@adshmh
Copy link
Copy Markdown
Contributor

@adshmh adshmh commented Jun 30, 2025

Summary

Add debug endpoints routing with API key authentication and refactor health check configuration

Primary Changes:

  • Add debug_endpoints authentication configuration with dedicated API key secret for secure debug access
  • Create path-pprof service to expose pprof endpoints on port 6060 for internal cluster access
  • Enable debug endpoints routing through guard with separate authentication from main API

Secondary Changes:

  • Refactor health check configuration structure in values.yaml with clearer defaults and documentation
  • Bump chart versions: path from 0.1.26 to 0.1.27, guard dependency from 0.1.24 to 0.1.32
  • Add TODO comment for future API key security improvements using K8s secrets

@adshmh adshmh self-assigned this Jun 30, 2025
@adshmh adshmh added the path label Jun 30, 2025
@Olshansk Olshansk requested review from Olshansk and okdas July 15, 2025 22:57
Copy link
Copy Markdown
Contributor

@Olshansk Olshansk left a comment

Choose a reason for hiding this comment

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

@okdas Can you pick up this PR and push it over the finish line?

Direction (not code) is a 👍 from my end EVEN if it doesn't work on prod for now.

Copy link
Copy Markdown
Contributor

@okdas okdas left a comment

Choose a reason for hiding this comment

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

Same as #102 - looks good, but would appreciate additional functionality behind the values flag.

Copy link
Copy Markdown
Contributor

@Olshansk Olshansk left a comment

Choose a reason for hiding this comment

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

@adshmh Now that we merged in #102, can you:

  1. Resolve conflicts and merge this in
  2. end to the comment from @okdas regarding switches here in a followup commit?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants