Before going ahead, Following is the explanation for the arch_word of
the MDS.
Arch word(4bits) is combination of architecture and bit size of the
machine. 3 bits are allocated
for architecture and 1 bit is allocated for bit size.
architecture of value 0 means unspecified.
Message encoding is
osaf/tools/saflog/saflogger/saf_logger.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
saflogger is not returning appropriate exit code to the shell when
SaLogWriteLogCallbackT returns error.
The patch exits with EXIT_FAILURE when the SaLogWriteLogCallbackT returns error
diff --git
Summary: log: saflogger to return EXIT_FAILURE when SaLogWriteLogCallbackT
reports error [#884]
Review request for Trac Ticket(s): #884
Peer Reviewer(s): lennart.l...@ericsson.com
Pull request to: LIST THE PERSON WITH PUSH ACCESS HERE
Affected branch(es): opensaf-4.3.x, 4.4.x, default
Development
Have tested and it seems Ok. ACK
Thanks
Lennart
-Original Message-
From: mathi.naic...@oracle.com [mailto:mathi.naic...@oracle.com]
Sent: den 2 maj 2014 23:45
To: Lennart Lund
Cc: opensaf-devel@lists.sourceforge.net
Subject: [PATCH 0 of 2] Review Request for log: ignore environment
Summary: amfd: instantiate mw sus when node is joining [#863]
Review request for Trac Ticket(s): #863
Peer Reviewer(s): Hans F, Hans N praveen
Pull request to: LIST THE PERSON WITH PUSH ACCESS HERE
Affected branch(es): All
Development branch: Default
Impacted
osaf/services/saf/amf/amfd/su.cc | 11 +--
1 files changed, 9 insertions(+), 2 deletions(-)
Problem:
When mw su is in locked-in state and opensaf is started, amfnd hangs.
When mw su is unlocked-in, amfnd still doesn't instantiate the mw su.
Analysis:
Amfd doesn't send instantiate
I don't get the use/test case. In the ticket node lock is performed. Node lock
should not affect the mw SU meaning after mw restart the mw SU should be
instantiated. The patch seems to indicate that SU admin ops are done.
Thanks,
Hans
-Original Message-
From: nagendr...@oracle.com
Hi
This seems Ok but there is one more problem in the same area that should be
fixed as well.
if (cb_invocation != invocation) {
fprintf(stderr, logWriteLogCallbackT FAILED: wrong
invocation\n);
return errorCode;
}
Should be changed to:
Please check the ticket, it is updated now.
Thanks
-Nagu
-Original Message-
From: Hans Feldt [mailto:hans.fe...@ericsson.com]
Sent: 06 May 2014 17:22
To: Nagendra Kumar; Hans Nordebäck; Praveen Malviya
Cc: opensaf-devel@lists.sourceforge.net
Subject: RE: [PATCH 1 of 1] amfd:
In #493 the assignments were given to SU5 without checking its presence
state. But this was not because of some admin operation, but due to the
continuous faults in
in all other SUs which were receiving the assignments. So ideally such a
check of Presence sate should be added in assignment
First I don't think we should bring in cluster and application state just like
this. It is not relevant for this case and we anyway don't support such ops.
Second this macro is out of control and should be changed into a (inline)
function instead.
Haven't I already done that in my refactoring
osaf/services/saf/amf/amfd/sg.cc | 13 +
1 files changed, 13 insertions(+), 0 deletions(-)
Problem:
SG admin lock-in return immediately.
Next admin unlock-in also succeed even if sus are not terminated yet.
Analysis:
If Amf could return try again to unlock-in admin op, then
another
Summary: amfd: return try_again to sg admin unlock-in if any su is in
terminating state [#854]
Review request for Trac Ticket(s): #854
Peer Reviewer(s): Hans F, Hans N, Praveen
Pull request to: LIST THE PERSON WITH PUSH ACCESS HERE
Affected branch(es): All
Development branch: Default
Summary: log: saflogger to return EXIT_FAILURE when SaLogWriteLogCallbackT
fails - v2 [#884]
Review request for Trac Ticket(s): #884
Peer Reviewer(s): Lennart
Pull request to: LIST THE PERSON WITH PUSH ACCESS HERE
Affected branch(es): opensaf-4.3.x, 4.4.x, default
Development branch: IF ANY GIVE
Ack from me.
Reviewed the code, but didn't test.
Best regards,
Zoran
-Original Message-
From: Anders Björnerstedt
Sent: den 29 april 2014 16:37
To: reddy.neelaka...@oracle.com; Zoran Milinkovic
Cc: opensaf-devel@lists.sourceforge.net
Subject: [PATCH 0 of 1] Review Request for imm: Allow
osaf/tools/saflog/saflogger/saf_logger.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
saflogger is not returning appropriate exit code to the shell when
SaLogWriteLogCallbackT fails either because of an error reported in the
callback or because
of an invalid invocationId.
The
tests/logsv/saflogtest.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
saflogtest is not returning appropriate exit code to the shell when
SaLogWriteLogCallbackT fails either because of an error reported in the
callback or because
of an invalid invocationId.
The patch exits with
One comment below.
mathi.naic...@oracle.com wrote:
Summary: log: saflogger to return EXIT_FAILURE when SaLogWriteLogCallbackT
fails - v2 [#884]
Review request for Trac Ticket(s): #884
Peer Reviewer(s): Lennart
Pull request to: LIST THE PERSON WITH PUSH ACCESS HERE
Affected branch(es):
There is a print message there.
fprintf(stderr, logWriteLogCallbackT FAILED: wrong invocation\n);
I can add if you are referring to a missing message.
Cheers,
Mathi.
- anders.bjornerst...@ericsson.com wrote:
One comment below.
mathi.naic...@oracle.com wrote:
Summary: log: saflogger
Comments inline...
On 05/06/2014 09:08 AM, Hans Feldt wrote:
First I don't think we should bring in cluster and application state just
like this. It is not relevant for this case and we anyway don't support such
ops.
[Alex] It is relevant in that this macro is a test for SU in-service.
Hi Praveen,
I don't think this is necessary because avd_sg_2n_su_chose_asgn()
(and others) checks the readiness state. And the readiness state is only
set to in-service in other parts of the code by first testing this
macro which I've modified.
So, if an SU is not instantiated, the
Surya,
Thank for reiterating arch_word of the MDS feature ,we all in sync.
On 5/6/2014 3:55 PM, SuryaNarayana Garlapati wrote:
MDS version unless we get alternate bits/variables used for MDS
version.
[Surya] Thats the reason i am asking for some time.
[AVM] If we get some alternate
Summary: v3 amfnd: saAmfSGCompRestartProb timer is not stopped after the
relevant model had been deleted [#831]
Review request for Trac Ticket(s): 831
Peer Reviewer(s): Hans F, Hans N, Nagendra, Praveen
Pull request to:
Affected branch(es): default
Development branch: default
Since there can be many more reasons why the SI is not assigned, I don't think
you should log any of what you have proposed in this part of the code. SG, SU
can be locked. Node can be locked etc.
In other places in the code we could log more when we actually know more things.
/Hans
Ack except for the commit message which just (again) describes the problem
instead of what the patch is changing.
/Hans
-Original Message-
From: Minh Hon Chau [mailto:minh.c...@dektech.com.au]
Sent: den 7 maj 2014 06:39
To: Hans Nordebäck; Hans Feldt; praveen.malv...@oracle.com;
25 matches
Mail list logo