Merge pull request #1904 from ruflin/fix-flaky-mysql-build

Update mysql version dependency to 5.7.12
elastic · Jun 24, 2016 · 139705b · 139705b
2 parents 08960d1 + b170061
commit 139705b
Show file tree

Hide file tree

Showing 7 changed files with 17 additions and 10 deletions.
diff --git a/libbeat/scripts/Makefile b/libbeat/scripts/Makefile
@@ -135,9 +135,7 @@ integration-tests: prepare-tests
 
 # Runs the integration inside a virtual environment. This can be run on any docker-machine (local, remote)
 .PHONY: integration-tests-environment
-integration-tests-environment:
-	$(MAKE) prepare-tests
-	$(MAKE) build-image
+integration-tests-environment: prepare-tests build-image
 	${DOCKER_COMPOSE} run beat make integration-tests RACE_DETECTOR=$(RACE_DETECTOR)
 
 # Runs the system tests
@@ -148,9 +146,7 @@ system-tests: buildbeat.test prepare-tests python-env
 
 # Runs the system tests
 .PHONY: system-tests-environment
-system-tests-environment:
-	$(MAKE) prepare-tests
-	$(MAKE) build-image
+system-tests-environment: prepare-tests build-image
 	${DOCKER_COMPOSE} run beat make system-tests;
 
 # Runs system tests without coverage reports and in parallel
@@ -283,6 +279,12 @@ write-environment:
 	echo "ES_USER=beats" >> ${BUILD_DIR}/test.env
 	echo "ES_PASS=testing" >> ${BUILD_DIR}/test.env
 
+# Tails the environment logs
+.PHONY: env-logs
+env-logs:
+	${DOCKER_COMPOSE} logs -f
+
+
 ### Packaging targets ####
 
 # Installs the files that need to get to the home path on installations

diff --git a/metricbeat/docker-compose.yml b/metricbeat/docker-compose.yml
@@ -41,7 +41,7 @@ mongodb:
   image: mongo:3.0
 
 mysql:
-  image: mysql:5.7.10
+  image: mysql:5.7.12
   environment:
     - MYSQL_ROOT_PASSWORD=test
 

diff --git a/metricbeat/docs/modules/mysql.asciidoc b/metricbeat/docs/modules/mysql.asciidoc
@@ -21,7 +21,7 @@ DSN is as following:
 [float]
 === Compatibility
 
-The mysql MetricSets were tested with mysql 5.7.10 and are expected to work with all version
+The mysql MetricSets were tested with mysql 5.7.12 and are expected to work with all version
 >= 5.7.0.
 
 

diff --git a/metricbeat/include/list.go b/metricbeat/include/list.go
@@ -20,6 +20,7 @@ import (
 	_ "github.com/elastic/beats/metricbeat/module/redis/info"
 	_ "github.com/elastic/beats/metricbeat/module/redis/keyspace"
 	_ "github.com/elastic/beats/metricbeat/module/system"
+	_ "github.com/elastic/beats/metricbeat/module/system/common"
 	_ "github.com/elastic/beats/metricbeat/module/system/core"
 	_ "github.com/elastic/beats/metricbeat/module/system/cpu"
 	_ "github.com/elastic/beats/metricbeat/module/system/diskio"

diff --git a/metricbeat/module/mysql/_meta/docs.asciidoc b/metricbeat/module/mysql/_meta/docs.asciidoc
@@ -16,5 +16,5 @@ DSN is as following:
 [float]
 === Compatibility
 
-The mysql MetricSets were tested with mysql 5.7.10 and are expected to work with all version
+The mysql MetricSets were tested with mysql 5.7.12 and are expected to work with all version
 >= 5.7.0.
diff --git a/metricbeat/module/mysql/status/status_test.go b/metricbeat/module/mysql/status/status_test.go
@@ -22,7 +22,7 @@ func TestConfigValidation(t *testing.T) {
 				"module":     "mysql",
 				"metricsets": []string{"status"},
 			},
-			err: "missing required field accessing config",
+			err: "missing required field accessing 'hosts'",
 		},
 		{
 			// Invalid DSN

diff --git a/testing/environments/Makefile b/testing/environments/Makefile
@@ -14,3 +14,7 @@ stop:
 clean:
 	docker stop $(shell docker ps -a -q)
 	docker rm -v $(shell docker ps -a -q)
+
+# Tails the environment log files
+logs:
+	${BASE_COMMAND} logs -f