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

Increase Indicator Queue Count Variable #6237

Merged
merged 2 commits into from
Mar 4, 2024

Conversation

zandre-eng
Copy link
Contributor

Link to ticket here.

This change is part of an urgent request to rebuild a UCR datasource. To help speed up the process, we are increasing the amount of indicators that can be queued to 20k (default is 10k).

Environments Affected

production

@mkangia
Copy link
Contributor

mkangia commented Mar 4, 2024

Hey @calellowitz
Based on our recent discussions, we are trying to run the async indicators with a bumped up number while not pushing it too far ahead as well.
Let me know if this sounds like a safe update to you.

@@ -256,3 +256,5 @@ shared_dir_gid: 1500
# for better time syncronization on aws servers
# replace ntp with chrony.
use_chrony: True

ASYNC_INDICATORS_TO_QUEUE: 20000
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs to be part of the localsettings object for it to be included in the localsettings https://github.com/dimagi/commcare-cloud/blob/master/src/commcare_cloud/ansible/roles/commcarehq/templates/localsettings.py.j2#L882

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, good catch. I've moved it under the localsettings object (c8c14a8).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for catching that @calellowitz

Copy link
Contributor

@calellowitz calellowitz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: these are mostly in alphabetical order and it would be nicer to add this in its alphabetical place, but i recognize there is another that has been added out of order at the end so not blocking

@zandre-eng zandre-eng merged commit 0adcbad into master Mar 4, 2024
2 checks passed
@zandre-eng zandre-eng deleted the ze/increase-indicator-queue-count branch March 4, 2024 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants