Skip to content

Commit 894e9d1

Browse files
authored
Merge pull request #53 from BlaineEXE/prow-debug-gomaxprocs
prow: debug GOMAXPROCS instead of setting it
2 parents de74a7e + 2c2687b commit 894e9d1

File tree

2 files changed

+4
-2
lines changed

2 files changed

+4
-2
lines changed

hack/prow-lint.sh

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,11 @@ set -o errexit
33
set -o nounset
44
set -o xtrace
55

6+
echo "GOMAXPROCS: $GOMAXPROCS" # debug prow CPU limit to ensure job not being throttled
7+
68
GOLANGCI_LINT_RUN_OPTS=""
79
GOLANGCI_LINT_RUN_OPTS="$GOLANGCI_LINT_RUN_OPTS --verbose" # debug linter timing and mem usage
8-
GOLANGCI_LINT_RUN_OPTS="$GOLANGCI_LINT_RUN_OPTS --concurrency=2" # prow job lags if too many threads
10+
GOLANGCI_LINT_RUN_OPTS="$GOLANGCI_LINT_RUN_OPTS --concurrency=$GOMAXPROCS" # golangci-lint seems to do a bad job obeying GOMAXPROCS
911
export GOLANGCI_LINT_RUN_OPTS
1012

1113
make lint

hack/prow-test.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,6 @@ set -o errexit
33
set -o nounset
44
set -o xtrace
55

6-
export GOMAXPROCS=2 # prow job lags if too many threads
6+
echo "GOMAXPROCS: $GOMAXPROCS" # debug prow CPU limit to ensure job not being throttled
77

88
make test

0 commit comments

Comments
 (0)