-
Notifications
You must be signed in to change notification settings - Fork 239
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
Bug: Duplication in outbound push #9854
Comments
@andrablaj can we back port the solution to 4.9 as this affects MoH-UG deployment |
4.9 is not supported anymore, so backporting to 4.9 is not a recommended approach. The ideal would be to upgrade MoH Uganda to a more recent, supported version. Is the upgrade option viable for MoH Uganda? |
That's very unfortunate getting moh to upgrade especially a minor/major upgrade is very hard given the logistical aspect of it and the changes the upgrade will bring |
Hi @jonathanbataire I think we should not do selective backports (for example release a 4.9.x with the fix, but not release a 4.10.x with the fix), and there are technical limitations to these backports - like fixes for "external" issues that you would need to identify and also cherry pick in order to get your build to pass (one example quickly comes to mind: 5d50e06 ) and I am sure this is not the only one. Creating hotpatch branches with fixes is risky, because it further decreases the chances for future upgrades. We have done this very sparingly (maybe twice?) and never pushed more than one change, and it was always to fix absolutely critical bugs (and I don't believe this bug is critical). How badly is the instance affected by this bug? |
Describe the bug
When configuring more than one out bound pushes without a cron configuration eg obp1 and obp2 the second configuration (obp2) will be pushed twice. Once immediately and another after about 5 mins
To Reproduce
push-target1
andpush-target2
relevant_to: doc.form === malaria_screening
push-target1
andpush-target2
and create another task:outbound forpush-target2
Expected behavior
Logs
Screenshots
Environment
Additional context
The text was updated successfully, but these errors were encountered: