Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[CI] org.elasticsearch.repositories.azure.AzureSnapshotRestoreTests fails #26812

Closed
hendrikmuhs opened this issue Sep 28, 2017 · 0 comments
Closed

[CI] org.elasticsearch.repositories.azure.AzureSnapshotRestoreTests fails #26812

hendrikmuhs opened this issue Sep 28, 2017 · 0 comments
Assignees
@dadoonet
Labels
>test Issues or PRs that are addressing/adding tests >test-failure Triaged test failures from CI v6.1.0 v7.0.0-beta1

Comments

@hendrikmuhs
Copy link

Reproduce: gradle :plugins:repository-azure:test

Blame: 1ccb497

Log:

==> Test Info: seed=529C3C6977F695FC; jvms=3; suites=6
Suite: org.elasticsearch.repositories.azure.AzureSnapshotRestoreTests
ERROR   0.00s J2 | AzureSnapshotRestoreTests (suite) <<< FAILURES!
   > Throwable #1: java.lang.IllegalStateException: to run integration tests, you need to set -Dtests.thirdparty=true and -Dtests.azure.account=azure-account -Dtests.azure.key=azure-key
   >    at org.elasticsearch.cloud.azure.AzureTestUtils.generateMockSecureSettings(AzureTestUtils.java:37)
   >    at org.elasticsearch.repositories.azure.AzureSnapshotRestoreTests.generateMockSettings(AzureSnapshotRestoreTests.java:81)
   >    at org.elasticsearch.repositories.azure.AzureSnapshotRestoreTests.<clinit>(AzureSnapshotRestoreTests.java:84)
   >    at java.lang.Class.forName0(Native Method)
   >    at java.lang.Class.forName(Class.java:348)
Completed [1/6] on J2 in 2.21s, 0 tests, 1 error <<< FAILURES!
@hendrikmuhs hendrikmuhs added >test-failure Triaged test failures from CI >test Issues or PRs that are addressing/adding tests v6.0.0 v6.1.0 labels Sep 28, 2017
@dadoonet dadoonet self-assigned this Sep 28, 2017
hendrikmuhs pushed a commit that referenced this issue Sep 28, 2017
disable AzureSnapshotRestoreTests, see #26812
4c9b029
@hendrikmuhs hendrikmuhs added v7.0.0 and removed v6.0.0 labels Sep 28, 2017
hendrikmuhs pushed a commit that referenced this issue Sep 28, 2017
disable AzureSnapshotRestoreTests, see #26812
bf4d312
hendrikmuhs pushed a commit that referenced this issue Sep 28, 2017
fix of disabling #26812
0358bb5
hendrikmuhs pushed a commit that referenced this issue Sep 28, 2017
fix of disabling #26812
4aec42b
dadoonet added a commit that referenced this issue Sep 28, 2017
@dadoonet
Don't use create static storage service
eb6d714 
Even though you annotate the Test class with `@ThirdParty` the static
code is initialized.

In that case it fails with:

```
==> Test Info: seed=529C3C6977F695FC; jvms=3; suites=6
Suite: org.elasticsearch.repositories.azure.AzureSnapshotRestoreTests
ERROR   0.00s J2 | AzureSnapshotRestoreTests (suite) <<< FAILURES!
   > Throwable #1: java.lang.IllegalStateException: to run integration tests, you need to set -Dtests.thirdparty=true and -Dtests.azure.account=azure-account -Dtests.azure.key=azure-key
   >    at org.elasticsearch.cloud.azure.AzureTestUtils.generateMockSecureSettings(AzureTestUtils.java:37)
   >    at org.elasticsearch.repositories.azure.AzureSnapshotRestoreTests.generateMockSettings(AzureSnapshotRestoreTests.java:81)
   >    at org.elasticsearch.repositories.azure.AzureSnapshotRestoreTests.<clinit>(AzureSnapshotRestoreTests.java:84)
   >    at java.lang.Class.forName0(Native Method)
   >    at java.lang.Class.forName(Class.java:348)
Completed [1/6] on J2 in 2.21s, 0 tests, 1 error <<< FAILURES!
```

Closes #26812.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dadoonet dadoonet

Labels
>test Issues or PRs that are addressing/adding tests >test-failure Triaged test failures from CI v6.1.0 v7.0.0-beta1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@colings86 @dadoonet @hendrikmuhs