Great! Thanks for the feedback Reka! On Sun, Dec 21, 2014 at 9:07 AM, Reka Thirunavukkarasu <r...@wso2.com> wrote:
> +1 to use cluster instance id. It makes more sense.. > > Thanks, > Reka > > On Sun, Dec 21, 2014 at 3:11 AM, Imesh Gunaratne <im...@apache.org> wrote: > >> May be we could call this property "clusterInstanceId" because it will >> always get the cluster instance id value: >> >> MemberContext/Member { >> serviceName // Service name >> clusterId // Cluster id >> clusterInstanceId // Cluster instance id >> memberId // Cloud controller generated member id >> instanceId // IaaS generated instance id >> } >> >> On Sun, Dec 21, 2014 at 12:14 AM, Imesh Gunaratne <im...@apache.org> >> wrote: >> >>> Hi Devs, >>> >>> It seems like we have reused the existing Instance Id property in Member >>> Context for defining the Application Hierarchy Instance Id. >>> >>> In cloud controller MemberContext.InstanceId was used earlier to store a >>> unique id based on the node id returned from jclouds. Now there seems to be >>> a conflict here. >>> >>> Shall we introduce a new property to store the Application Hierarchy >>> Instance Id in MemberContext/Member called "hierarchyInstanceId"? >>> >>> MemberContext/Member { >>> serviceName // Service name >>> clusterId // Cluster id >>> memberId // Cloud controller generated id >>> instanceId // IaaS generated id >>> hierarchyInstanceId // Instance id of the hierarchy that member >>> belongs to >>> } >>> >>> Thanks >>> >>> >>> -- >>> Imesh Gunaratne >>> >>> Technical Lead, WSO2 >>> Committer & PMC Member, Apache Stratos >>> >> >> >> >> -- >> Imesh Gunaratne >> >> Technical Lead, WSO2 >> Committer & PMC Member, Apache Stratos >> > > > > -- > Reka Thirunavukkarasu > Senior Software Engineer, > WSO2, Inc.:http://wso2.com, > Mobile: +94776442007 > > > -- Imesh Gunaratne Technical Lead, WSO2 Committer & PMC Member, Apache Stratos