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

Bug: Duplication in outbound push #9854

Open
jonathanbataire opened this issue Mar 24, 2025 · 4 comments · May be fixed by #9855
Open

Bug: Duplication in outbound push #9854

jonathanbataire opened this issue Mar 24, 2025 · 4 comments · May be fixed by #9855
Assignees
Labels
Type: Bug Fix something that isn't working as intended
Milestone

Comments

@jonathanbataire
Copy link
Contributor

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

  • create out bound push with atleast 2 configurations to 2 target end points eg push-target1 and push-target2
  • Configure both outbound pushes to be triggered by the same form type i.e relevant_to: doc.form === malaria_screening
  • Leave the "cron" empty so pushes occur immediately as soon as the report is submitted
  • Login as a user and submit the sample form (malaria_screening)
  • Outbound push will push records to both push-target1 and push-target2 and create another task:outbound for push-target2
  • After about 5 mins the push-target2 task will be executed and the same payload will be pushed to target2 again
    Expected behavior
  • Target2 task shouldnt be created as it causes payload to be pushed twice

Logs

Screenshots

Environment

  • Instance: dev instance on v4.9.0, local environment with v4.18.0
  • Browser: Firefox and Chrome
  • Client platform: Linux
  • Version: 4.9.0 and the latest code 4.18.0

Additional context

@jonathanbataire jonathanbataire added the Type: Bug Fix something that isn't working as intended label Mar 24, 2025
@jonathanbataire jonathanbataire linked a pull request Mar 24, 2025 that will close this issue
6 tasks
@andrablaj andrablaj added this to the 4.19.0 milestone Mar 24, 2025
@jonathanbataire
Copy link
Contributor Author

@andrablaj can we back port the solution to 4.9 as this affects MoH-UG deployment

@andrablaj
Copy link
Member

andrablaj commented Mar 27, 2025

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?

@jonathanbataire
Copy link
Contributor Author

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

@dianabarsan
Copy link
Member

Hi @jonathanbataire
The advised path is to upgrade, but I understand it's very hard to get MOH signoff.

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?
What is the logistical aspect of a potential upgrade that is preventing MOH approval?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Fix something that isn't working as intended
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants