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.

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

[TEST] Packaging Test Failures #44

Closed
adnapibar opened this issue Feb 4, 2021 · 4 comments
Closed

[TEST] Packaging Test Failures #44

adnapibar opened this issue Feb 4, 2021 · 4 comments
Labels
>FORK Related to the fork process >test-failure Test failure from CI, local build, etc.

Comments

@adnapibar
Copy link
Contributor

adnapibar commented Feb 4, 2021

The qa directory contains tests which run against several operating systems. This Issue lists the current test failures.

Task: ./gradlew packagingTest

  • org.elasticsearch.packaging.test.CronEvalCliTests > test10Install FAILED
@adnapibar adnapibar added >FORK Related to the fork process >test-failure Test failure from CI, local build, etc. labels Feb 4, 2021
@tlfeng
Copy link
Collaborator

tlfeng commented Feb 5, 2021

Can be reproduced by me, and I tested the error doesn't appear in Elasticsearch v7.10.2 .

| org.elasticsearch.packaging.test.CronEvalCliTests > test10Install FAILED
|     java.lang.AssertionError: 
|     Expected: "file/directory: "<File> with owner "root" with group "root" with posix permissions [<OWNER_READ>,<OWNER_WRITE>,<OWNER_EXECUTE>,<GROUP_READ>,<GROUP_EXECUTE>,<OTHERS_READ>,<OTHERS_EXECUTE>]
|          but: path </usr/share/elasticsearch/bin/elasticsearch-certgen>Does not exist
|         at __randomizedtesting.SeedInfo.seed([98141E2552261F4:E5DB4D1A09B2DEAE]:0)
|         at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:18)
|         at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:6)
|         at org.elasticsearch.packaging.util.Packages.lambda$verifyDefaultInstallation$12(Packages.java:242)
|         at java.base/java.util.Spliterators$ArraySpliterator.forEachRemaining(Spliterators.java:948)
|         at java.base/java.util.stream.ReferencePipeline$Head.forEach(ReferencePipeline.java:658)
|         at org.elasticsearch.packaging.util.Packages.verifyDefaultInstallation(Packages.java:242)
|         at org.elasticsearch.packaging.util.Packages.verifyPackageInstallation(Packages.java:167)
|         at org.elasticsearch.packaging.test.PackagingTestCase.install(PackagingTestCase.java:220)
|         at org.elasticsearch.packaging.test.CronEvalCliTests.test10Install(CronEvalCliTests.java:39)

@adnapibar
Copy link
Contributor Author

This seems to be related to #49

@adnapibar
Copy link
Contributor Author

Before closing this issue, we need to verify no more new packaging tests are failing after we removed the default distribution #50 .

@nknize
Copy link
Collaborator

nknize commented Mar 22, 2021

can this now be closed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>FORK Related to the fork process >test-failure Test failure from CI, local build, etc.
Projects
None yet
Development

No branches or pull requests

3 participants