Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
92233: envutil: add GOMEMLIMIT to safeVarRegistry r=ajwerner a=nvanbenschoten

This commit adds the GOMEMLIMIT environment variable to the list of go runtime env vars that Cockroach will report about on the Node Diagnostics page. The original set was added in f6e2313. Since then, this is the only env var added to the go runtime.

Release note: None

Epic: None

Co-authored-by: Nathan VanBenschoten <[email protected]>
  • Loading branch information
craig[bot] and nvanbenschoten committed Nov 21, 2022
2 parents f694132 + 797b866 commit f0554bc
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions pkg/util/envutil/env.go
Original file line number Diff line number Diff line change
Expand Up @@ -186,10 +186,12 @@ func GetEnvVarsUsed() (result []redact.RedactableString) {
// the name and the value safely: the value is known to never contain
// sensitive information.
var safeVarRegistry = map[redact.SafeString]struct{}{
// Go runtime.
"GOGC": {},
"GODEBUG": {},
"GOMAXPROCS": {},
"GOTRACEBACK": {},
"GOMEMLIMIT": {},
// gRPC.
"GRPC_GO_LOG_SEVERITY_LEVEL": {},
"GRPC_GO_LOG_VERBOSITY_LEVEL": {},
Expand Down

0 comments on commit f0554bc

Please sign in to comment.