Skip to content

Commit

Permalink
MM-26584: Bump up Go patch version (mattermost#15156)
Browse files Browse the repository at this point in the history
* MM-26584: Bump up Go patch version

This contains a patch fix for golang/go#38023
which have been encountered in our load tests and some users.

I have verified on such a stuck instance and the thread dump seem to match
with what is seen on the issue. All threads stuck on runtime.futex, except one,
which is at runtime.osyield.

And of all the times load tests were run which _did not_ show this issue,
it was run with a version greater than 1.14.1, which further points to the
theory that this bug is the culprit.

* Replace docker image versions.

* Update build/README.md

Co-authored-by: Elisabeth Kulzer <[email protected]>
Co-authored-by: Mattermod <[email protected]>
  • Loading branch information
3 people authored and krishna sindhur maruvala committed Oct 9, 2020
1 parent 33180f9 commit e62e3b5
Show file tree
Hide file tree
Showing 6 changed files with 16 additions and 15 deletions.
12 changes: 6 additions & 6 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ jobs:

check-app-layers:
docker:
- image: mattermost/mattermost-build-server:20200322_golang-1.14.1
- image: mattermost/mattermost-build-server:20200731_golang-1.14.6
working_directory: ~/mattermost
steps:
- attach_workspace:
Expand All @@ -114,7 +114,7 @@ jobs:
check-store-layers:
docker:
- image: mattermost/mattermost-build-server:20200322_golang-1.14.1
- image: mattermost/mattermost-build-server:20200731_golang-1.14.6
working_directory: ~/mattermost
steps:
- attach_workspace:
Expand Down Expand Up @@ -146,7 +146,7 @@ jobs:

build:
docker:
- image: mattermost/mattermost-build-server:20200322_golang-1.14.1
- image: mattermost/mattermost-build-server:20200731_golang-1.14.6
resource_class: xlarge
working_directory: ~/mattermost
steps:
Expand Down Expand Up @@ -231,7 +231,7 @@ jobs:
--env GOBIN=/mattermost/mattermost-server/bin \
-v ~/mattermost:/mattermost \
-w /mattermost/mattermost-server \
mattermost/mattermost-build-server:20200322_golang-1.14.1 \
mattermost/mattermost-build-server:20200731_golang-1.14.6 \
bash -c 'ulimit -n 8096; make test-server BUILD_NUMBER="$CIRCLE_BRANCH-$CIRCLE_PREVIOUS_BUILD_NUM" TESTFLAGS= TESTFLAGSEE='
no_output_timeout: 1h
- run:
Expand Down Expand Up @@ -304,7 +304,7 @@ jobs:
--env MM_ELASTICSEARCHSETTINGS_CONNECTIONURL=http://elasticsearch:9200 \
-v ~/mattermost:/mattermost \
-w /mattermost/mattermost-server \
mattermost/mattermost-build-server:20200322_golang-1.14.1 \
mattermost/mattermost-build-server:20200731_golang-1.14.6 \
bash -c 'ulimit -n 8096; make ARGS="version" run-cli && make MM_SQLSETTINGS_DATASOURCE="postgres://mmuser:mostest@postgres:5432/latest?sslmode=disable&connect_timeout=10" ARGS="version" run-cli'
echo "Generating dump"
docker-compose --no-ansi exec -T postgres pg_dump --schema-only -d migrated -U mmuser > migrated.sql
Expand Down Expand Up @@ -340,7 +340,7 @@ jobs:
--env MM_ELASTICSEARCHSETTINGS_CONNECTIONURL=http://elasticsearch:9200 \
-v ~/mattermost:/mattermost \
-w /mattermost/mattermost-server \
mattermost/mattermost-build-server:20200322_golang-1.14.1 \
mattermost/mattermost-build-server:20200731_golang-1.14.6 \
bash -c 'ulimit -n 8096; make ARGS="version" run-cli && make MM_SQLSETTINGS_DATASOURCE="mmuser:mostest@tcp(mysql:3306)/latest?charset=utf8mb4,utf8&readTimeout=30s&writeTimeout=30s" ARGS="version" run-cli'
echo "Ignoring known MySQL mismatch: ChannelMembers.SchemeGuest"
Expand Down
2 changes: 1 addition & 1 deletion build/Dockerfile.buildenv
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
FROM golang:1.14.1
FROM golang:1.14.6

RUN apt-get update && apt-get install -y make git apt-transport-https ca-certificates curl software-properties-common build-essential zip xmlsec1 jq
2 changes: 1 addition & 1 deletion build/Jenkinsfile.branch
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ def platformStages = new org.mattermost.PlatformStages()

def rndEE = UUID.randomUUID().toString()
def rndTE = UUID.randomUUID().toString()
def mmBuilderServer = 'mattermost/mattermost-build-server:20200322_golang-1.14.1'
def mmBuilderServer = 'mattermost/mattermost-build-server:20200731_golang-1.14.6'
def mmBuilderWebapp = 'mattermost/mattermost-build-webapp:oct-2-2018'

pipeline {
Expand Down
12 changes: 6 additions & 6 deletions build/Jenkinsfile.pr
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,7 @@ pipeline {
}

steps {
withDockerContainer(args: '-u root --privileged -v ${WORKSPACE}/src:/go/src/', image: 'mattermost/mattermost-build-server:20200322_golang-1.14.1') {
withDockerContainer(args: '-u root --privileged -v ${WORKSPACE}/src:/go/src/', image: 'mattermost/mattermost-build-server:20200731_golang-1.14.6') {
ansiColor('xterm') {
sh """
cd /go/src/github.com/mattermost/mattermost-server
Expand All @@ -98,7 +98,7 @@ pipeline {
}

steps {
withDockerContainer(args: '-u root --privileged -v ${WORKSPACE}/src:/go/src/', image: 'mattermost/mattermost-build-server:20200322_golang-1.14.1') {
withDockerContainer(args: '-u root --privileged -v ${WORKSPACE}/src:/go/src/', image: 'mattermost/mattermost-build-server:20200731_golang-1.14.6') {
ansiColor('xterm') {
sh """
cd /go/src/github.com/mattermost/mattermost-server
Expand Down Expand Up @@ -286,7 +286,7 @@ pipeline {
}
}

withDockerContainer(args: "-u root --privileged --net ${COMPOSE_PROJECT_NAME}_mm-test -v ${WORKSPACE}/src:/go/src/", image: 'mattermost/mattermost-build-server:20200322_golang-1.14.1') {
withDockerContainer(args: "-u root --privileged --net ${COMPOSE_PROJECT_NAME}_mm-test -v ${WORKSPACE}/src:/go/src/", image: 'mattermost/mattermost-build-server:20200731_golang-1.14.6') {
ansiColor('xterm') {
sh """
cd /go/src/github.com/mattermost/mattermost-server
Expand Down Expand Up @@ -333,7 +333,7 @@ pipeline {
}
}

withDockerContainer(args: "-u root --privileged --net ${COMPOSE_PROJECT_NAME}_mm-test -v ${WORKSPACE}/src:/go/src/", image: 'mattermost/mattermost-build-server:20200322_golang-1.14.1') {
withDockerContainer(args: "-u root --privileged --net ${COMPOSE_PROJECT_NAME}_mm-test -v ${WORKSPACE}/src:/go/src/", image: 'mattermost/mattermost-build-server:20200731_golang-1.14.6') {
ansiColor('xterm') {
sh """
cd /go/src/github.com/mattermost/mattermost-server
Expand Down Expand Up @@ -404,7 +404,7 @@ pipeline {
}
}

withDockerContainer(args: "-u root --privileged --net ${COMPOSE_PROJECT_NAME}_mm-test -v ${WORKSPACE}/src-copy:/go/src/", image: 'mattermost/mattermost-build-server:20200322_golang-1.14.1') {
withDockerContainer(args: "-u root --privileged --net ${COMPOSE_PROJECT_NAME}_mm-test -v ${WORKSPACE}/src-copy:/go/src/", image: 'mattermost/mattermost-build-server:20200731_golang-1.14.6') {
ansiColor('xterm') {
sh """
cd /go/src/github.com/mattermost/mattermost-server
Expand Down Expand Up @@ -448,7 +448,7 @@ pipeline {
}
}

withDockerContainer(args: "-u root --privileged --net ${COMPOSE_PROJECT_NAME}_mm-test -v ${WORKSPACE}/src-copy:/go/src/", image: 'mattermost/mattermost-build-server:20200322_golang-1.14.1') {
withDockerContainer(args: "-u root --privileged --net ${COMPOSE_PROJECT_NAME}_mm-test -v ${WORKSPACE}/src-copy:/go/src/", image: 'mattermost/mattermost-build-server:20200731_golang-1.14.6') {
ansiColor('xterm') {
sh """
cd /go/src/github.com/mattermost/mattermost-server
Expand Down
1 change: 1 addition & 0 deletions build/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -18,3 +18,4 @@ In our Docker Hub Repository we have the following images:
- `mattermost/mattermost-build-server:feb-28-2019` which is based on Go 1.12 you can use for MM versions >= `5.9.0` <= `5.15.0`
- `mattermost/mattermost-build-server:sep-17-2019` which is based on Go 1.12.9 you can use for MM versions >= `5.16.0`
- `mattermost/mattermost-build-server:20200322_golang-1.14.1` which is based on Go 1.14.1 you can use for MM versions >= `5.24.0`
- `mattermost/mattermost-build-server:20200731_golang-1.14.6` which is based on Go 1.14.6 you can use for MM versions >= `5.26.0`
2 changes: 1 addition & 1 deletion build/local-test-env.sh
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ up()
-e MM_EMAILSETTINGS_SMTPSERVER="inbucket" \
-e MM_EMAILSETTINGS_SMTPPORT="10025" \
-e MM_ELASTICSEARCHSETTINGS_CONNECTIONURL="http://elasticsearch:9200" \
mattermost/mattermost-build-server:20200322_golang-1.14.1 /bin/bash
mattermost/mattermost-build-server:20200731_golang-1.14.6 /bin/bash
}

down()
Expand Down

0 comments on commit e62e3b5

Please sign in to comment.