-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Dist tests started to fail for Ubuntu 20 #3894
Comments
Fixes JanusGraph#3894 Signed-off-by: Oleksandr Porunov <alexandr.porunov@gmail.com>
Notice, I restarted previously passed Now not only they failed for Java 11, but also for Java 8 as well. See failed job: |
Fixes JanusGraph#3894 Signed-off-by: Oleksandr Porunov <alexandr.porunov@gmail.com>
Fixes JanusGraph#3894 Signed-off-by: Oleksandr Porunov <alexandr.porunov@gmail.com>
Fixes JanusGraph#3894 Signed-off-by: Oleksandr Porunov <alexandr.porunov@gmail.com>
Fixes JanusGraph#3894 Signed-off-by: Oleksandr Porunov <alexandr.porunov@gmail.com>
Fixes JanusGraph#3894 Signed-off-by: Oleksandr Porunov <alexandr.porunov@gmail.com>
Fixes JanusGraph#3894 Signed-off-by: Oleksandr Porunov <alexandr.porunov@gmail.com>
@JanusGraph/committers if anyone can take a look at this issue it would be really helpful. This issue blocks any other PRs from being merged. The test fails when we are executing the line 155 ( janusgraph/janusgraph-dist/src/test/java/org/janusgraph/pkgtest/AbstractJanusGraphAssemblyIT.java Line 155 in 029b4d2
Observations:
Any ideas or thoughts are very welcome. Thank you! |
Issue which could be related: #1826 |
@farodin91 @FlorianHockmann looks like the tests are failing due to some environment changes (probably due to docker updates as noted here: #1826 (comment)). You can see that dist tests are passing in the next branch where I reverted those 2 commits: As I understand, the work of moving Docker builds into the main repository is still Work In Progress. As such, would you be OK if I revert those two commits to unblock other PRs, and instead create a separate branch with those commits? After the work of moving Docker builds into the main repo is finished and all tests are passing then we can merge those commits back into the “master” branch again. |
@farodin91 I will later today look into this issue. |
@porunov I will try get out a log output which would help to debug. Currently I'm thinking of out of memory or out of storage. |
Make sense. It's just strange that with no any code changes these tests started to fail in |
Not sure, but could also be related to the file janusgraph-dist/docker/Dockerfile
Potentially, something could be changed in the |
It looks like updating runs-on Ubuntu from 20.04 to 22.04 seems to work. |
Interesting why suddenly JanusGraph Docker container stopped to work in Ubuntu 20.04 but works in Ubuntu 22.04. Do you know if we start Docker container |
eclipse-temurin:11-jre is just used for the docker image. The image is used for the test. This is the last executed test. |
This issue was fixed in #3842 by upgrading from Ubuntu 20 to Ubuntu 22. I also think that this is the main problem for users commented in #1826 . I don't think there is anyone working on this issue, thus, I'm removing milestone from this ticket because it doesn't block the builds anymore and all tests pass in |
Fixes JanusGraph#3894 Signed-off-by: Oleksandr Porunov <alexandr.porunov@gmail.com>
Please include configurations and logs if available.
For confirmed bugs, please report:
master
Stack Trace (if you have one)
The text was updated successfully, but these errors were encountered: