[devel] [PATCH 3/3] fmd: enable reload of configuration without restart [#2923]

2018-09-18 Thread Gary Lee
Only FMS_PROMOTE_ACTIVE_TIMER, FMS_NODE_ISOLATION_TIMEOUT and FMS_ACTIVATION_SUPERVISION_TIMER are currently supported. These values can be changed in fmd.conf and take effect by sending SIGHUP to fmd. --- src/fm/fmd/fm_cb.h | 2 ++ src/fm/fmd/fm_main.cc | 70

[devel] [PATCH 0/3] Review Request for fmd: enable reload of configuration without restart V2 [#2923]

2018-09-18 Thread Gary Lee
Summary: fmd: enable reload of configuration without restart [#2923] Review request for Ticket(s): 2923 Peer Reviewer(s): Hans Pull request to: *** LIST THE PERSON WITH PUSH ACCESS HERE *** Affected branch(es): develop Development branch: ticket-2923 Base revision:

[devel] [PATCH 1/3] base: remove use of SIGHUP to toggle INFO messages [#2923]

2018-09-18 Thread Gary Lee
We need to use SIGHUP for reload of configuration --- src/base/daemon.c| 5 +++-- src/base/logtrace.cc | 22 -- 2 files changed, 3 insertions(+), 24 deletions(-) diff --git a/src/base/daemon.c b/src/base/daemon.c index 4a37c4174..cdde7fde0 100644 --- a/src/base/daemon.c

[devel] [PATCH 2/3] base: add config file reader [#2923]

2018-09-18 Thread Gary Lee
Some configuration attribute are read by OpenSAF daemons as environment variables. eg. export FMS_PROMOTE_ACTIVE_TIMER=0 There is no easy way to reload these values without a restart. ConfigFileReader will parse these files looking for 'export VAR=VAL' and store them into a map, so a daemon can

[devel] [PATCH 0/2] Review Request for amfd: Split-brain recovery for NwayActive [#2926]

2018-09-18 Thread Minh Chau
Summary: amfd: Reboot the node that has SU over-assigned after split brain [#2926] Review request for Ticket(s): 2926 Peer Reviewer(s): *** Hans, Gary, Nagu Pull request to: *** LIST THE PERSON WITH PUSH ACCESS HERE *** Affected branch(es): develop Development branch: ticket-2926 Base revision:

[devel] [PATCH 1/2] amfd: Reboot the node that has SU over-assigned after split brain [#2926]

2018-09-18 Thread Minh Chau
After split-brain, the assignments can go over the preferred number. Reboot the node once at time until the current active assignments is equal to preferred number. --- src/amf/amfd/sg_nwayact_fsm.cc | 20 +--- src/amf/amfd/siass.cc | 5 - 2 files changed, 21

[devel] [PATCH 0/1] Review Request for plmd: cleanup current admin operation on EE if EE uninstantiates [#198]

2018-09-18 Thread Jones, Alex
Summary: plmd: cleanup current admin operation on EE if EE uninstantiates [#198] Review request for Ticket(s): 198 Peer Reviewer(s): Mathi, Ravi Pull request to: Affected branch(es): develop Development branch: ticket-198 Base revision: 6f4f5ddc776a7e566b6958ecd4d4a3502bccfa5e Personal

[devel] [PATCH 1/1] plmd: cleanup current admin operation on EE if EE uninstantiates [#198]

2018-09-18 Thread Jones, Alex
Application can receive START step after responding to VALIDATE step after EE has uninstantiated. Ongoing admin operations are not cleaned up after EE uninstantiates. Clean up ongoing admin operations and tracking when an EE uninstantiates. --- src/plm/plmd/plms_plmc.c | 42

[devel] [PATCH 1/1] ntf: Fix NtfReader filter with searchcriteria is not correct [#2925]

2018-09-18 Thread Canh Van Truong
NtfReader filter with searchcriteria then create the cached list notification. But the filter is not correct. The patch update the the filter of search criteria and the read next of notification. --- src/ntf/apitest/ntf_search_criteria_test.sh | 26 +- src/ntf/ntfd/NtfReader.cc

[devel] [PATCH 0/1] Review Request for ntf: Fix NtfReader filter with searchcriteria is not correct [#2925]

2018-09-18 Thread Canh Van Truong
Summary: ntf: Fix NtfReader filter with searchcriteria is not correct [#2925] Review request for Ticket(s): 2925 Peer Reviewer(s): Minh Pull request to: *** LIST THE PERSON WITH PUSH ACCESS HERE *** Affected branch(es): develop Development branch: ticket-2925 Base revision:

Re: [devel] [PATCH 1/1] uml: add support for plm to run under uml [#2922]

2018-09-18 Thread Mathi N P
Good work. A very nice value add for PLM users. Mathi. On Tue, Sep 18, 2018 at 9:22 AM Hans Nordebäck wrote: > ack, review only. (Run ok with the legacy uml without plm enabled). > > Minor comment, the function cmd_build/install_container_testprog() is > part of ticket #70? > > /BR HansN > >

Re: [devel] [PATCH 1/1] plmd: fix adding and removing of invocation id to list [#197]

2018-09-18 Thread Mathi N P
Ack, Not tested. Mathi. On Fri, Sep 14, 2018 at 3:33 PM Alex Jones wrote: > Jan 22 11:09:03 localhost osafplmd[3988]: Invocation id mentioned in the > resp, is not found in the grp->inocation_list. inv_id: 9 > > If multiple entities are part of the same entity group, and START or > VALIDATE >

Re: [devel] [PATCH 0/3] Review Request for fmd: enable reload of configuration without restart [#2923]

2018-09-18 Thread Gary Lee
Sorry, I will send out a V2 soon. Please ignore. I haven't handled the case where you send SIGHUP to a process that doesn't explicitly handle it. On 18/09/18 18:13, Gary Lee wrote: Summary: base: remove use of SIGHUP to toggle INFO messages [#2923] Review request for Ticket(s): 2923 Peer

[devel] [PATCH 3/3] fmd: enable reload of configuration without restart [#2923]

2018-09-18 Thread Gary Lee
Only FMS_PROMOTE_ACTIVE_TIMER, FMS_NODE_ISOLATION_TIMEOUT and FMS_ACTIVATION_SUPERVISION_TIMER are currently supported. These values can be changed in fmd.conf and take effect by sending SIGHUP to fmd. --- src/fm/fmd/fm_cb.h | 2 ++ src/fm/fmd/fm_main.cc | 70

[devel] [PATCH 2/3] base: add config file reader [#2923]

2018-09-18 Thread Gary Lee
Some configuration attribute are read by OpenSAF daemons as environment variables. eg. export FMS_PROMOTE_ACTIVE_TIMER=0 There is no easy way to reload these values without a restart. ConfigFileReader will parse these files looking for 'export VAR=VAL' and store them into a map, so a daemon can

[devel] [PATCH 0/3] Review Request for fmd: enable reload of configuration without restart [#2923]

2018-09-18 Thread Gary Lee
Summary: base: remove use of SIGHUP to toggle INFO messages [#2923] Review request for Ticket(s): 2923 Peer Reviewer(s): Hans Pull request to: *** LIST THE PERSON WITH PUSH ACCESS HERE *** Affected branch(es): develop Development branch: ticket-2923 Base revision:

[devel] [PATCH 1/3] base: remove use of SIGHUP to toggle INFO messages [#2923]

2018-09-18 Thread Gary Lee
We need to use SIGHUP for reload of configuration --- src/base/logtrace.cc | 22 -- 1 file changed, 22 deletions(-) diff --git a/src/base/logtrace.cc b/src/base/logtrace.cc index c1a194f60..8908c1ff3 100644 --- a/src/base/logtrace.cc +++ b/src/base/logtrace.cc @@ -75,22 +75,6

Re: [devel] [PATCH 1/1] uml: add support for plm to run under uml [#2922]

2018-09-18 Thread Hans Nordebäck
ack, review only. (Run ok with the legacy uml without plm enabled). Minor comment, the function cmd_build/install_container_testprog() is part of ticket #70? /BR HansN On 09/17/2018 09:53 PM, Alex Jones wrote: > Add support for plm to run under uml. > --- > src/plm/config/openhpi.conf