Skip to content

Commit

Permalink
Remove debug logging in full cluster restart tests (#33612)
Browse files Browse the repository at this point in the history
These logs are incredibly verbose, and it makes chasing normal failures
burdensome. This commit removes the debug logging, which can be
reenabled again if needed.
  • Loading branch information
jasontedor committed Sep 11, 2018
1 parent 8bcd6d3 commit acd6214
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 12 deletions.
6 changes: 0 additions & 6 deletions qa/full-cluster-restart/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -57,9 +57,6 @@ for (Version version : bwcVersions.indexCompatible) {
// some tests rely on the translog not being flushed
setting 'indices.memory.shard_inactive_time', '20m'

// debug logging for testRecovery
setting 'logger.level', 'DEBUG'

if (version.onOrAfter('5.3.0')) {
setting 'http.content_type.required', 'true'
}
Expand All @@ -79,9 +76,6 @@ for (Version version : bwcVersions.indexCompatible) {
// some tests rely on the translog not being flushed
setting 'indices.memory.shard_inactive_time', '20m'

// debug logging for testRecovery
setting 'logger.level', 'DEBUG'

numNodes = 2
dataDir = { nodeNum -> oldClusterTest.nodes[nodeNum].dataDir }
cleanShared = false // We want to keep snapshots made by the old cluster!
Expand Down
6 changes: 0 additions & 6 deletions x-pack/qa/full-cluster-restart/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -160,9 +160,6 @@ subprojects {
// some tests rely on the translog not being flushed
setting 'indices.memory.shard_inactive_time', '20m'

// debug logging for testRecovery see https://github.com/elastic/x-pack-elasticsearch/issues/2691
setting 'logger.level', 'DEBUG'

setting 'xpack.security.enabled', 'true'
setting 'xpack.security.transport.ssl.enabled', version.onOrAfter("5.2.0").toString()
setting 'xpack.ssl.keystore.path', 'testnode.jks'
Expand Down Expand Up @@ -213,9 +210,6 @@ subprojects {
setupCommand 'setupTestUser', 'bin/elasticsearch-users', 'useradd', 'test_user', '-p', 'x-pack-test-password', '-r', 'superuser'
waitCondition = waitWithAuth

// debug logging for testRecovery see https://github.com/elastic/x-pack-elasticsearch/issues/2691
setting 'logger.level', 'DEBUG'

// some tests rely on the translog not being flushed
setting 'indices.memory.shard_inactive_time', '20m'
setting 'xpack.security.enabled', 'true'
Expand Down

0 comments on commit acd6214

Please sign in to comment.