Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0052445Openbravo ERPA. Platformpublic2023-05-15 11:452023-05-22 15:55
ReporterAugustoMauch 
Assigned ToAugustoMauch 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionPR23Q3 
Merge Request Statusapproved
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0052445: The margin when checking if we need to replace a cluster instance is too small
DescriptionOne of the responsibilities of the ClusterService is to ensure a new node takes control of a service if its current node is not available. It checks that condition by comparing when was the last time the current node notified its availability (by making an update in the database), adding a 10% margin to the notification periodicity.

We have found that the current 10% margin is too small. It should be made configurable and by default it should account for the possibility of missing at least one ping (i.e. because the current node did not have db connections available for a few seconds).
Steps To ReproduceThere are no steps to reproduce but we have seen it happen several times. We can check with CSU to get the details.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
has duplicate defect 0052457 closed Triage Platform Conn On cluster environments processing node can be switch even if current node is alive 
related to defect 0052456 closed caristu Import Entry post process hook not executed when processing node is changed 
related to defect 0052455 closed caristu Import Entry processed twice when processing node is changed 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2023-05-15 11:45AugustoMauchNew Issue
2023-05-15 11:45AugustoMauchAssigned To => Triage Platform Base
2023-05-15 11:45AugustoMauchOBNetwork customer => No
2023-05-15 11:45AugustoMauchModules => Core
2023-05-15 11:45AugustoMauchTriggers an Emergency Pack => No
2023-05-15 11:45AugustoMauchAssigned ToTriage Platform Base => AugustoMauch
2023-05-16 09:45AugustoMauchSummaryThe margin when checking => The margin when checking if we need to replace a cluster instance is too small
2023-05-16 09:45AugustoMauchAssigned ToAugustoMauch => meriem_azaf
2023-05-16 09:56AugustoMauchAssigned Tomeriem_azaf => AugustoMauch
2023-05-16 09:56AugustoMauchStatusnew => acknowledged
2023-05-16 10:31gorkaionRelationship addedhas duplicate 0052457
2023-05-16 10:31gorkaionRelationship addedrelated to 0052456
2023-05-16 10:31gorkaionRelationship addedrelated to 0052455
2023-05-18 14:33ngarciaIssue Monitored: ngarcia
2023-05-19 13:52AugustoMauchStatusacknowledged => scheduled
2023-05-19 13:58hgbotMerge Request Status => open
2023-05-19 13:58hgbotNote Added: 0149985
2023-05-22 12:08hgbotMerge Request Statusopen => approved
2023-05-22 12:22hgbotResolutionopen => fixed
2023-05-22 12:22hgbotStatusscheduled => closed
2023-05-22 12:22hgbotFixed in Version => PR23Q3
2023-05-22 12:22hgbotNote Added: 0150057
2023-05-22 12:22hgbotNote Added: 0150058
2023-05-22 14:35hgbotNote Added: 0150072
2023-05-22 15:55hgbotNote Added: 0150083

Notes
(0149985)
hgbot   
2023-05-19 13:58   
Merge Request created: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/888 [^]
(0150057)
hgbot   
2023-05-22 12:22   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/openbravo [^]
Changeset: 73afbd0f589b19d3fbe6796d2c34129a620635e8
Author: Augusto Mauch <augusto.mauch@openbravo.com>
Date: 22-05-2023 10:22:40
URL: https://gitlab.com/openbravo/product/openbravo/-/commit/73afbd0f589b19d3fbe6796d2c34129a620635e8 [^]

Fixes ISSUE-52445: Makes cluster timeout threshold configurable

---
M src-db/database/model/tables/AD_CLUSTER_SERVICE_SETTINGS.xml
M src-db/database/sourcedata/AD_COLUMN.xml
M src-db/database/sourcedata/AD_ELEMENT.xml
M src-db/database/sourcedata/AD_FIELD.xml
M src/org/openbravo/cluster/ClusterService.java
M src/org/openbravo/cluster/ClusterService_data.xsql
---
(0150058)
hgbot   
2023-05-22 12:22   
Merge request merged: https://gitlab.com/openbravo/product/openbravo/-/merge_requests/888 [^]
(0150072)
hgbot   
2023-05-22 14:35   
Repository: https://gitlab.com/openbravo/product/openbravo [^]
Changeset: 6110813aa83e4205bdbfe4b0fd71599451e3b841
Author: Carlos Aristu <carlos.aristu@openbravo.com>
Date: 22-05-2023 14:34:00
URL: https://gitlab.com/openbravo/product/openbravo/-/commit/6110813aa83e4205bdbfe4b0fd71599451e3b841 [^]

related to ISSUE-52445: add missing on create default

---
M src-db/database/model/tables/AD_CLUSTER_SERVICE_SETTINGS.xml
---
(0150083)
hgbot   
2023-05-22 15:55   
Repository: https://gitlab.com/openbravo/ci/backoffice-api [^]
Changeset: 536022c95b5f1f39701ae89d388643be35d14ba2
Author: Carlos Aristu <carlos.aristu@openbravo.com>
Date: 22-05-2023 15:54:59
URL: https://gitlab.com/openbravo/ci/backoffice-api/-/commit/536022c95b5f1f39701ae89d388643be35d14ba2 [^]

related to ISSUE-52445: add missing check constraint for new column

---
M model/src-db/database/model/tables/AD_CLUSTER_SERVICE_SETTINGS.xml
---