To be more clear, I am not telling we need to validate "network partitions" while deploying deployment policy. I am telling we should validate whether all the network partitions given in the deployment policy are already *deployed/added or not*. This validation is different from network partitions validation.
Thanks. On Mon, Feb 16, 2015 at 1:01 PM, Rajkumar Rajaratnam <rajkum...@wso2.com> wrote: > Hi Gayan, > > Please find my comments inline. > > On Mon, Feb 16, 2015 at 12:41 PM, Gayan Gunarathne <gay...@wso2.com> > wrote: > >> Hi Raj, >> >> I think we don't need to validate the network partition stuff with the >> deployment policy.It should validate through network partition management >> API. >> > > The validation I am talking is different from the validation you are > referring. While deploying deployment policy, we need to validate whether > the network partitions given in the deployment policy is already deployed > or not. Otherwise, one can refer a dummy network partition id in the > deployment policy. > > >> >> Deployment policy has only the reference to the network >> partitions.Network partition management is done through separate REST API >> methods. >> >> Yeah currently network partition management API reside in AS. IMO we can >> move that network partition management to the CC as well. >> > > I also feel CC is the best place to have network partition management > APIs. Lets hear others' opinion as well. > > Thanks. > >> >> Thanks, >> Gayan >> >> On Mon, Feb 16, 2015 at 12:18 PM, Rajkumar Rajaratnam <rajkum...@wso2.com >> > wrote: >> >>> Hi Devs, >>> >>> I guess $Subject is not done at the moment. I am in the process of >>> adding the validation. This is the sample deployment policy. >>> >>> { >>> "id": "deployment-policy-2", >>> "networkPartition": [ >>> { >>> "id": "network-partition-1", >>> "partitionAlgo": "one-after-another", >>> "partitions": [ >>> { >>> "id": "partition-1", >>> "max": 5 >>> } >>> ] >>> } >>> ] >>> } >>> >>> AFAIU, we need to validate two things. >>> >>> - whether all network partitions are already deployed >>> - whether all partitions are defined in relevant network partitions >>> >>> Are there any other stuffs to be validated? >>> >>> Further, deployment policy management APIs are at CC side. Network >>> partitions management APIs are at AS side. >>> >>> I am not sure about this. Do we have a convention that we shouldn't call >>> AS service from CC? >>> Thanks. >>> >>> -- >>> Rajkumar Rajaratnam >>> Committer & PMC Member, Apache Stratos >>> Software Engineer, WSO2 >>> >>> Mobile : +94777568639 >>> Blog : rajkumarr.com >>> >> >> >> >> -- >> >> Gayan Gunarathne >> Technical Lead >> WSO2 Inc. (http://wso2.com) >> email : gay...@wso2.com | mobile : +94 766819985 >> >> > > > > -- > Rajkumar Rajaratnam > Committer & PMC Member, Apache Stratos > Software Engineer, WSO2 > > Mobile : +94777568639 > Blog : rajkumarr.com > -- Rajkumar Rajaratnam Committer & PMC Member, Apache Stratos Software Engineer, WSO2 Mobile : +94777568639 Blog : rajkumarr.com