generator-jhipster: ngx-couchbase intermittent Test Failures
Overview of the issue
This issue was created from the discussion at https://github.com/jhipster/generator-jhipster/issues/10589. It seems that ngx-couchbase tests on our CI seems to hang and fail sometimes. Here are some examples;
https://github.com/jhipster/generator-jhipster/pull/10498/checks?check_run_id=254018117
https://dev.azure.com/jhipster/jhipster/_build/results?buildId=6316
Motivation for or Use Case
Getting the CI tests in working condition without problems.
Reproduce the error
Submitting a PR or making a commit so that the CI is triggered might recreate the issue.
Related issues
Suggest a Fix
Currently not known.
- Checking this box is mandatory (this is just to show you read everything)
About this issue
- Original URL
- State: closed
- Created 5 years ago
- Comments: 48 (48 by maintainers)
@SudharakaP Are you downgrading the couchbase java-client version? The dependency is already defined in jhipster bom project : see https://github.com/jhipster/jhipster/blob/master/jhipster-dependencies/pom.xml#L274 Current version defined is 2.7.9 : https://github.com/jhipster/jhipster/blob/master/jhipster-dependencies/pom.xml#L46
@vishal423 I try to upgrade to this version but it was not working…
@SudharakaP : the first run failed 😦
Ok, let’s close this. Plz @SudharakaP, claim the bounty, it’s well deserved !
I’m restarting it, as you need to see the result, right ?
I’m reopening this, and wait few days our daily builds before closing this definitely
Hello, sorry guys I couldn’t reproduce the problem on my end, I’ve updated Couchmove and Couchbase image version, but I don’t think it will change anything
@SudharakaP, I have noticed a few things while looking into this failure (maybe these can help):
v6.0.1. It would be good to try to upgrade the couchbase image (considering we already have18.04in our build infrastructure). I do see some couchbase image upgrade attempts by @DanielFran, but, not sure on why they were reverted.v6.0.3. We can try to see if that helps to improve the situation on this part.v5.5.1. I don’t see any change on that part, so, not very sure if that code will work flawlessly with the latest couchbase images.@SudharakaP : do you want me to merge #10628, then restart the daily builds for OpenJDK8.Official ? Am I correct ?
Ok, restarting the Official.openjdk8 right now
Just tried tonight and I manage to reproduce the issue only 1 time. Then, in Azure Daily builds:
cc @tchlyah if you can help here For me, I gave up, as I’m not a Couchbase user
I’m reopening this, as we don’t have the real reason of the failures
See https://twitter.com/whichrich/status/1183348440803221504?s=19
I’ll priorize it and will investigate.
@pascalgrimaud : I’ve done three PRs. In the first two I’ve made OpenJDK 8 the default on Azure pipelines. Given our experience with Couchbase I don’t think the Azul JDK that ships with Azure can be trusted, and we should test everything with OpenJDK.
My third PR reintroduces Couchbase docker start. We’ll merge this and see how it goes. In the worst case we can just revert it back so that everything will work as usual 😄
yes, let’s try !
if I remember well, the problem with Couchbase was:
I am not very sure on this solution. However, if it solves current issue, then, there is no harm to try this out.
@pascalgrimaud @vishal423 : I believe I found the problem here. It seems that the javadoc (via the maven javadoc plugin) runs on java 1.8.0_22; because we manually set JAVA_HOME in
00-init-env.sh. But notice that on Github Actions when Java 11 is installed (via the initialsetup-java@v1) JAVA_HOME is already set to/opt/hostedtoolcache/Java/11.0.3/x64and this value is overwritten by our00-init-env.shscript and thus the javadoc ultimately uses the default java version installed in the build machine; which seems to be 1.8.0_22.What we need to do is for GitHub CI not change the JAVA_HOME value. I’ve done some tests with this and it works. I will do this modification to the current PR soon. 😄
The best would be to test both: dev and prod But it would be too much configurations to test.
There are no specific reason, that
couchbaseis tested inprodprofile. It could be indevand Cassandra could be inprodThanks. I’m adding a bounty on this as it is important to be fixed quickly.