(chore) Site config updates - #445
Open
pirupius wants to merge 6 commits into
Open
Conversation
…ners Docker's default 10s stop timeout is not enough for Tomcat plus ~40 modules and the embedded ActiveMQ broker. A SIGKILL partway through shutdown leaves a torn record in the KahaDB journal, and the next boot fails in recoverProducerAudit with "Invalid location ... NegativeArraySizeException". Give openmrs 120s and mysql 60s. InnoDB recovers from a hard kill on its own, but a clean stop avoids a long redo replay on the next start and keeps the binlog tail intact for CDC.
The AutoCloseVisitsTask row was inserted directly into scheduler_task_config by a per-site changeset in bunia, matsapha and mosul. All three guard on the same task UUID, so the duplication served no purpose beyond drift between sites. In mosul the changeset was also mis-identified as "unvoid_arabic_translations-202411271458" while carrying the auto-close SQL, which made the site history misleading. Existing deployments are unaffected - the changesets have already run and the scheduler row persists. The visits.autoCloseVisitType global property stays under initializer_config/globalproperties.
Auto-close only stopped OPD Visit, leaving every other visit type open indefinitely. Extend the property to the visit types each site actually uses: matsapha: OPD Visit, Facility Visit, Home Visit, Offline Visit, Group Session mosul: OPD Visit, Home Visit, Telehealth Bunia is left on OPD Visit only.
Drop @openmrs/esm-patient-task-list-app from modulesToRemove so the frontend assembly keeps it in the Matsapha build.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Requirements
LIME2-123)Summary
Screenshots
Related Issue
Other