chore(#10455): update CouchDB version to 3.5.1#10501
chore(#10455): update CouchDB version to 3.5.1#10501shashi-sah2003 wants to merge 13 commits intomedic:masterfrom
Conversation
|
@dianabarsan this PR is ready for review |
|
This PR is now marked "stale" after 30 days without activity. It will be closed automatically in 10 days unless you add a comment, push new changes or remove the "stale" label. |
dianabarsan
left a comment
There was a problem hiding this comment.
As stated in this comment, we should make an assessment at least on Nouveau performance changes in the new version.
|
@shashi-sah2003, would you be available to work on the requirements mentioned by @dianabarsan above? We would love to include this issue in the upcoming CHT release. |
|
@dianabarsan I have added benchmarking setup, didn't get desirable results (maybe I am missing something). Go through the README file for setting up the setup. lmk what do you think about this? If there is anything wrong in my setup do lmk |
|
All the benchmarking tests already had a framework to be run: https://github.com/medic/cht-core/blob/master/tests/scalability/run_suite.sh#L111 . They are run through the scalability suite, so there was no need to setup a test data generator or anything like that. If it's easier, I can setup the Nouveau benchmark on In terms of favorable results, can you share what was unfavorable? |
|
@dianabarsan for the results what I saw locally with my current setup is below, is it possible that I am using less data or something else? also is the Nouveau benchmark merged on master? |
The scalability suite (run_suite.sh) already has framework for: - Setting up CHT environment - Using test-data-generator for population - Running benchmarks via node index.js - Uploading results Removed custom Docker infrastructure as benchmarks only need to run against the actual CHT deployment, not standalone instances.
this is taking too long, I have left it for around 40 minutes still it didn't complete, is it expected to happen? Except this CouchDb Performance benchmarkDatabase InfoCouchDb version: 3.5.1 Database doc count: 903030 _changes benchmark
_all_docs benchmark
_bulk_get benchmark
view benchmark
|
No, that should not happen, but to be honest I don't know what your setup is. What I believe the next steps should be:
Once the build is green, I will create a new branch off of your branch and run scalability tests on it - these tests need to be manually triggered on a branch that is in the Thanks a lot for your patience! |
|
@dianabarsan I have pushed the setup which I am trying to do fyr: https://github.com/medic/cht-core/pull/10501/changes Steps that I am following:
was already running with the above mentioned steps locally
I will see what is making the build to fail tho it's just a version upgrade. EDIT: Hmm interesting the build are failing due to couchdb containers aren't able to spin up with version 3.5.1, I believe this is happening due to the support of HTTP/2 in this version? Maybe we have to reconfigure it to use a nouveau configuration yaml file (attached in this pr) for this to spin up properly, what do you think? |
The is the reported error: If you believe additional nouveau config will help, this is a quick way to add it (see the configs section): https://docs.couchdb.org/en/stable/install/nouveau.html#compose The files to change would be: Let's give it a go to see if tests pass and then we can polish it. |
yes because of these errors only I have to add this config https://github.com/medic/cht-core/pull/10501/changes#diff-8b77ae47b921cddcb6cad23386ce670c0bf1caa17a7e37df394c21ce756a5aed
will add these now, and let you know EDIT: have pushed the changes using noveau config |
…e cluster and single-node templates to include new config structure
…de new config structure
…e obsolete config file
|
@dianabarsan have added more detailed config to avoid |
from here: https://docs.couchdb.org/en/stable/install/nouveau.html, I thought increasing the value of
Yeah we could use the default values here as well, idk if that would help but let's give it a try. PS: what is this UI you are using? looks good for visualization :) |
|
didn't see any effect locally, checked the logs seems like indexing is being done correctly
|
The doc count is the same as when I ran the tests multiple times. Indexing 1kk docs should not take so long, there's something definitely wrong happening here. |
I'm wondering how removing that optional config is leading to ci failure 🤔. Also they all are failing with different issues. |
|
I kept restarting it. 🤞🏻 |
|
I don't think they are flaky, failing everytime with same error. ;( Btw did you try running scalability tests on master? Just wanted to see if this issue is related to this feature branch only |
|
|
Hmm this is tricky, according to this it shouldn't take much time for 3.5.1 version, there is something very wrong with our configs |
|
This PR is now marked "stale" after 30 days without activity. It will be closed automatically in 10 days unless you add a comment, push new changes or remove the "stale" label. |
|
closing due to #10455 (comment) |

fixes #10455
Code review checklist
can_view_old_navigationpermission to see the old design. Test it has appropriate design for RTL languages.License
The software is provided under AGPL-3.0. Contributions to this project are accepted under the same license.