CLOUDP-317911 Add pprof integration in operator #101
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
pprof can be configured by two new environment variables:
MDB_OPERATOR_PPROF_ENABLED
- together withOPERATOR_ENV
controls enabling of pprof server. Basically the rule for enabling pprof is defined inIsPprofEnabled
function:mongodb-kubernetes/pkg/pprof/pprof.go
Lines 66 to 70 in 94fc9ea
MDB_OPERATOR_PPROF_PORT
- by default it is set to 10081It's more than
_ "net/http/pprof"
one liner for a couple of reasons:standard way of starting pprof by is discouraged for lack of configurability and security issues
Proof of Work
pprof debug page is available at default

localhost:10081
portAdded unit tests that verify
IsPprofEnabled
function. Shutdown is also working:Checklist
Reminder (Please remove this when merging)