devtron
devtron copied to clipboard
Feature: Expose heap dump api internally on port other than 80 / 443 and remove authentication for orchestrator
🔖 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?
- [X] I checked and didn't find similar issue
🏢 Have you read the Code of Conduct?
- [X] I have read the Code of Conduct
AB#8360
❌ There was a problem linking to Azure Boards work item(s):
- 8360
Please check the IDs and try again using the AB# syntax. Learn more
❌ There was a problem linking to Azure Boards work item(s):
- 8360
Please check the IDs and try again using the AB# syntax. Learn more
❌ There was a problem linking to Azure Boards work item(s):
- 8360
Please check the IDs and try again using the AB# syntax. Learn more
❌ There was a problem linking to Azure Boards work item(s):
- 8360
Please check the IDs and try again using the AB# syntax. Learn more
- We won't be removing authentication from endpoint (Please discuss before picking)
- Different port is required or not is purely tech call
Not critical anymore as per discussion with @tayalrishabh96 @nishant-d
We can take heap dumps for microservices other than orchestrator using proxy but authentication is not removed yet. No plans for the same.