-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 with geonode-selenium #4240
Conversation
Travis is failing because I am using Ubuntu 16.04 instead of the default 14.04. |
dee27a0
to
023da23
Compare
Ok, we are getting closer to it. 5 jobs out of 9 are now working as expected; 3 GeoServer jobs are full of Java errors (could someone please have a look at them?), while the new |
Seems like a GeoFence initialization issue
trying to restart the job in order to see if this is a temporary issue. I will dig into it otherwise. |
I guess those addings to .travis.yml
are causing issues. |
a3e1877
to
f9e1ae3
Compare
f9e1ae3
to
9c8d1b2
Compare
Moving from having a script |
weird... let me take a look! |
@frafra I suspect the issue is the Java version it is using. We should install the Oracle JDK 8 instead. |
Ideally we should do something like the following:
|
You are right, it is using a different JVM:
The current code seems very similar to what you proposed, and almost identical to the one in master (I just call debconf to accept the license):
It seems that installing |
@frafra almost there... I see this error on geonode-selenium test
not sure why it is trying to run two nginx instances |
@afabiani I updated |
Please try one last time. |
@frafra few progresses but still errors
|
:(
|
083c7aa
to
9ae8535
Compare
Ok, tests are now running, but it looks that there is a timing issue. I am going to fix it. I propose to squash the "fix" commits after that and merge. Thanks for your help. |
188598d
to
6f1bb7d
Compare
I tested the modifications alone, but not the merged
.travis-ci.yml
file because I have issue with Travis CI (it says that my geonode repository is legacy, they do not support migration to the new system at the moment, so it seems that I cannot use it nor disable it).This pull request fixes #4234.