devtron-labs / devtron

Tool integration platform for Kubernetes
https://devtron.ai
Apache License 2.0
4.24k stars 472 forks source link

Feature: Expose heap dump api internally on port other than 80 / 443 and remove authentication for orchestrator #4633

Open tayalrishabh96 opened 7 months ago

tayalrishabh96 commented 7 months ago

🔖 Feature description

Expose heap dump api internally on port other than 80 / 443 and remove authentication for the same for orchestrator

🎤 Pitch / Usecases

As of now heap dump api requires super admin token, the use-case is to be able to take heap dump without generating super admin token. current api to take heap dump : curl -H 'token: <apiToken>' https://<orchestrator_service_url.devtroncd>/orchestrator/debug/pprof/heap requirement is something like below : curl https://<orchestrator_service_url.devtroncd>:<some_port>/orchestrator/debug/pprof/heap

🔄️ Alternative

NA

👀 Have you spent some time to check if this issue has been raised before?

🏢 Have you read the Code of Conduct?

AB#8360

azure-boards[bot] commented 7 months ago

❌ There was a problem linking to Azure Boards work item(s):

Please check the IDs and try again using the AB# syntax. Learn more

azure-boards[bot] commented 7 months ago

❌ There was a problem linking to Azure Boards work item(s):

Please check the IDs and try again using the AB# syntax. Learn more

azure-boards[bot] commented 7 months ago

❌ There was a problem linking to Azure Boards work item(s):

Please check the IDs and try again using the AB# syntax. Learn more

azure-boards[bot] commented 7 months ago

❌ There was a problem linking to Azure Boards work item(s):

Please check the IDs and try again using the AB# syntax. Learn more

abhibhaw commented 7 months ago
abhibhaw commented 6 months ago

Not critical anymore as per discussion with @tayalrishabh96 @nishant-d

tayalrishabh96 commented 1 month ago

We can take heap dumps for microservices other than orchestrator using proxy but authentication is not removed yet. No plans for the same.