Skip to content

Commit

Permalink
Yet more logging for S3SnapshotRepoTestKitIT
Browse files Browse the repository at this point in the history
Moves the logging added in elastic#102299 onto the test cluster where it
belongs, and adds wire logging to this test suite too.

Relates elastic#102294
  • Loading branch information
DaveCTurner committed Nov 17, 2023
1 parent 8573571 commit 7d250a1
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 5 deletions.
6 changes: 6 additions & 0 deletions x-pack/plugin/snapshot-repo-test-kit/qa/s3/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,12 @@ testClusters.matching { it.name == "javaRestTest" }.configureEach {
println "Using an external service to test " + project.name
}
setting 'xpack.security.enabled', 'false'
// Additional tracing related to investigation into https://github.com/elastic/elasticsearch/issues/102294
setting 'logger.org.elasticsearch.repositories.s3', 'TRACE'
setting 'logger.org.elasticsearch.repositories.blobstore.testkit', 'TRACE'
setting 'logger.com.amazonaws.request', 'DEBUG'
setting 'logger.org.apache.http.wire', 'DEBUG'
}
tasks.register("s3ThirdPartyTest") {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
package org.elasticsearch.repositories.blobstore.testkit;

import org.elasticsearch.common.settings.Settings;
import org.elasticsearch.test.junit.annotations.TestIssueLogging;

import static org.hamcrest.Matchers.blankOrNullString;
import static org.hamcrest.Matchers.not;
Expand All @@ -31,10 +30,6 @@ protected Settings repositorySettings() {
}

@Override
@TestIssueLogging(
issueUrl = "https://github.com/elastic/elasticsearch/issues/102294",
value = "org.elasticsearch.repositories.s3:TRACE,org.elasticsearch.repositories.blobstore.testkit:TRACE"
)
public void testRepositoryAnalysis() throws Exception {
super.testRepositoryAnalysis();
}
Expand Down

0 comments on commit 7d250a1

Please sign in to comment.