That's fine then. Especially in the loong run we might support individual 
component upgrades too (for middleware services) with appropriate version 
adjustments.

Also, as per my previous comment, I think the ETF could have the elements that 
are defined as "mandatory" in the SMF spec!

Thanks,
Mathi.

>-----Original Message-----
>From: Bertil Engelholm [mailto:[email protected]]
>Sent: Wednesday, April 30, 2014 11:15 AM
>To: Mathivanan Naickan Palanivelu; Hans Feldt
>Cc: [email protected]
>Subject: RE: [PATCH 2 of 2] clm: add ETF file for CLMD [#848]
>
>Normally it is an ETF per bundle but since the packaging of the SW is decided
>by someone building OpenSAF we can't decide the bundle. But there is a
>need for input to this bundle ETF and that's what we try and add to OpenSAF.
>If we create one ETF per CompType we'll have max flexibility regarding the
>packaging. And since the node directors are usually packaged separately from
>the directors we believe it is better to separate them also on ETF file level.
>
>/Bertil
>
>-----Original Message-----
>From: Mathivanan Naickan Palanivelu [mailto:[email protected]]
>Sent: den 29 april 2014 17:18
>To: Hans Feldt
>Cc: [email protected]; Bertil Engelholm
>Subject: Re: [PATCH 2 of 2] clm: add ETF file for CLMD [#848]
>
>Comments inline:
>Thanks,
>Mathi.
>
>----- [email protected] wrote:
>
>> See inline
>>
>> Thanks,
>>
>> Hans
>>
>>
>>
>>
>>
>>
>>
>>
>> From: Mathivanan Naickan Palanivelu [mailto:[email protected]]
>> Sent: den 25 april 2014 12:08
>> To: Hans Feldt; Bertil Engelholm
>> Cc: [email protected]
>> Subject: RE: [PATCH 2 of 2] clm: add ETF file for CLMD [#848]
>>
>>
>>
>> Nice start.
>>
>>
>>
>> Like Bertil commented, yes CLMNA should also be part of the same ETF
>> for the CLM ‘software’!
>>
>> [Hans] sure this was just an RFC, the same or another file?
>>
>
>I don't know which is better, i.e. ETF per middleware service or ETF per
>middleware component.
>I'm inclining towards ETF per middleware service!
>
>The SMF spec talks of "ETF w.r.t software bundles", not sure if it should be
>seen as ETF 'per' software bundle!
>
>To quote:
>"When a software vendor provides a new software bundle, an entity types
>file describ- ing the content of the software bundle shall be supplied with the
>bundle."
>..."One entity types file may refer to multiple software bundles, each of which
>must be identified according to the XML schema...."
>
>
>
>> And,
>>
>>
>>
>> What about the below, will they be part of some OpenSAF common ETF?
>>
>> +<xs:element name="SUType">++----------------
>>
>> +<xs:element name="SGType">-+---------------
>>
>> +<xs:element name="AppType">++----
>>
>> +<xs:element name="CSType">++--------
>>
>> +<xs:element name="ServiceType">
>>
>> [Hans] I currently don’t see any need for that.
>>
>>
>
>The SMF spec/schema already defines which elements should be mandatory
>element, optional or alternative.
>We could use that as the guideline!
>
>
>>
>> And did you not think of the need for defining any appropriate item
>> from below?
>>
>> [Hans] Sorry I don’t get this list, what exactly is missing from this
>> ETF to generate the same imm.xml as today
>>
>
>While there is a rule/guideline from the SMF spec, it also depends on the
>scope for the tool that is going to use this ETF file.
>I'm not sure about the scope of this activity(patch request)!?
>
>You could weigh all your comments below w.r.t the guideline for ETF from the
>SMF spec.
>
>>
>>
>> -<xs:element name="swBundle" maxOccurs="unbounded" minOccurs="0">
>>
>> +<xs:element name="removal">++----------
>>
>> +<xs:element name="installation">-+-------- </xs:sequence>
>>
>> [Hans] I currently don’t see a need for the bundle part
>>
>>
>>
>> +<xs:element name="quiescingComplete" type="BoundedTimeT">-
>>
>> [Hans] already in there
>>
>> +<xs:element name="numInstances" type="UnsignedInt32T">-
>>
>> +<xs:element name="numMaxStandbyCsi" type="BoundedUInt32T">-
>>
>> [Hans] could add that
>>
>> +<xs:element name="instantiateCmd">--
>>
>> [Hans] already in there
>>
>> +<xs:element name="numMaxActiveCsi" type="BoundedUInt32T">-
>>
>> [Hans] could add that
>>
>> +<xs:attributeGroup name="valueRange">-
>>
>> +<xs:complexType name="UnsignedInt32T">-
>>
>> +<xs:complexType name="BoundedUInt32T">-
>>
>> +<xs:complexType name="BoundedTimeT">-
>>
>> +<xs:complexType name="callbackOptionsT">----------
>>
>> +<xs:complexType name="ProbationT">-
>>
>> +<xs:simpleType name="SaTimeT">-
>>
>> +<xs:simpleType name="SaStringT">-
>>
>> +<xs:simpleType name="VersionT">--
>>
>> +<xs:simpleType name="SwBundleNameT">--
>>
>> +<xs:simpleType name="CompTypeNameT">--
>>
>> +<xs:simpleType name="SUTypeNameT">--
>>
>> +<xs:simpleType name="SGTypeNameT">--
>>
>> +<xs:simpleType name="AppTypeNameT">--
>>
>> +<xs:simpleType name="CSTypeNameT">--
>>
>> +<xs:simpleType name="SrvTypeNameT">--
>>
>> +<xs:simpleType name="RecommendedRecoveryT">--
>>
>> [Hans] already in there
>>
>> +<xs:simpleType name="HealthcheckVariantT">--
>>
>> +<xs:simpleType name="RedModelT">--
>>
>> +<xs:group name="CompCapability">--------------------
>>
>> +<xs:group name="CompCallbackTimeouts">----------
>>
>> +<xs:group name="CliParameters">------
>>
>> +<xs:group name="CommonCompAttributes">-----------------
>>
>> [Hans] yes the relevant ones are configured
>>
>> +<xs:group name="ScopeOfImpact">
>>
>>
>>
>>
>>
>> Thanks,
>>
>> Mathi.
>>
>>
>>
>>
>>
>> >-----Original Message-----
>>
>> >From: Hans Feldt [ mailto:[email protected] ]
>>
>> >Sent: Thursday, April 24, 2014 7:10 PM
>>
>> >To: [email protected] ; Mathivanan Naickan Palanivelu
>>
>> >Cc: [email protected]
>>
>> >Subject: [PATCH 2 of 2] clm: add ETF file for CLMD [#848]
>>
>> >
>>
>> > osaf/services/saf/clmsv/config/ETF-CLMD.xml | 35
>>
>> >+++++++++++++++++++++++++++++
>>
>> > osaf/services/saf/clmsv/config/Makefile.am | 2 +
>>
>> > 2 files changed, 37 insertions(+), 0 deletions(-)
>>
>> >
>>
>> >
>>
>> >Add an ETF file for CLMD. The contents corresponds to what we have in
>> the
>>
>> >IMM XML file.
>>
>> >
>>
>> >diff --git a/osaf/services/saf/clmsv/config/ETF-CLMD.xml
>>
>> >b/osaf/services/saf/clmsv/config/ETF-CLMD.xml
>>
>> >new file mode 100644
>>
>> >--- /dev/null
>>
>> >+++ b/osaf/services/saf/clmsv/config/ETF-CLMD.xml
>>
>> >@@ -0,0 +1,35 @@
>>
>> >+<?xml version="1.0" encoding="utf-8"?>
>>
>> >+<entityTypesFile xmlns:xsi=" http://www.w3.org/2001/XMLSchema-
>>
>> >instance " name="CLMD" provider="OSAF">
>>
>> >+ <AmfEntityType>
>>
>> >+ <CompType name="safCompType=OSAF-CLMD"
>>
>> >version="safVersion=1">
>>
>> >+ <providesCSType name="safCSType=OSAF-CLMD"
>>
>> >version="safVersion=1">
>>
>> >+ <oneactiveoronestandby/>
>>
>> >+ </providesCSType>
>>
>> >+ <saAware>
>>
>> >+ <healthCheck
>>
>> >key="safHealthcheckKey=Default">
>>
>> >+ <period lowerBound="240000000000"
>>
>> >default="240000000000"/>
>>
>> >+ <maxDuration
>>
>> >lowerBound="180000000000"/>
>>
>> >+ </healthCheck>
>>
>> >+ <quiescingComplete
>>
>> >lowerBound="10000000000" default="10000000000"/>
>>
>> >+ <independent>
>>
>> >+ <instantiateCmd>
>>
>> >+
>>
>> > <command>xxCLCCLIDIRxx/osaf-clmd</command>
>>
>> >+ <args>instantiate</args>
>>
>> >+ </instantiateCmd>
>>
>> >+ <cleanupCmd>
>>
>> >+
>>
>> > <command>xxCLCCLIDIRxx/osaf-clmd</command>
>>
>> >+ <args>cleanup</args>
>>
>> >+ </cleanupCmd>
>>
>> >+ </independent>
>>
>> >+ </saAware>
>>
>> >+ <disableRestart>true</disableRestart>
>>
>> >+
>>
>> > <recoveryOnError>SA_AMF_NODE_FAILFAST</recoveryOnError>
>>
>> >+ <defaultClcCliTimeOut lowerBound="10000000000"/>
>>
>> >+ <defaultCallbackTimeOut
>>
>> >lowerBound="40000000000"/>
>>
>> >+ <bundleReference name="safSmfBundle=OSAF"/>
>>
>> >+ </CompType>
>>
>> >+ </AmfEntityType>
>>
>> >+ <AmfEntityType>
>>
>> >+ <CSType name="safCSType=OSAF-CLMD"
>>
>> >version="safVersion=1" />
>>
>> >+ </AmfEntityType>
>>
>> >+</entityTypesFile>
>>
>> >diff --git a/osaf/services/saf/clmsv/config/Makefile.am
>>
>> >b/osaf/services/saf/clmsv/config/Makefile.am
>>
>> >--- a/osaf/services/saf/clmsv/config/Makefile.am
>>
>> >+++ b/osaf/services/saf/clmsv/config/Makefile.am
>>
>> >@@ -38,6 +38,8 @@ dist_pkgimmxml_svc_DATA += \
>>
>> > clmsv_pl_template.xml
>>
>> > endif
>>
>> >
>>
>> >+dist_pkgetfxml_svc_DATA = \
>>
>> >+ ETF-CLMD.xml
>>
>> > endif
>>
>> >
>>
>> > install-data-hook:

------------------------------------------------------------------------------
"Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE
Instantly run your Selenium tests across 300+ browser/OS combos.  Get 
unparalleled scalability from the best Selenium testing platform available.
Simple to use. Nothing to install. Get started now for free."
http://p.sf.net/sfu/SauceLabs
_______________________________________________
Opensaf-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to