Yes that's fine -----Original Message----- From: Alena Prokharchyk Sent: Wednesday, January 29, 2014 11:14 AM To: dev@cloudstack.apache.org Cc: Murali Reddy; Animesh Chaturvedi Subject: Re: assignCertToLB/removeCertToLb commands
Animesh, should it go directly to 4.3 branch? Thanks, Alena. On 1/29/14, 9:58 AM, "Syed Ahmed" <syed1.mush...@gmail.com> wrote: >Thanks Alena, > >I have the review request at https://reviews.apache.org/r/17504/ > >-Syed > > >On Wed 29 Jan 2014 12:47:54 PM EST, Alena Prokharchyk wrote: >> The commands that should be fixed are: >> >> assignCertToLoadBalancer >> removeCertFromLoadBalancer >> >> >> >> Thanks, >> Alena. >> >> >> On 1/29/14, 8:33 AM, "Syed Ahmed" <syed1.mush...@gmail.com> wrote: >> >>> Hi Alena, >>> >>> I was checking the code and I see that the response for >>> uploadSslCert is uploadsslcertresponse and deleteSslCert is >>> deletesslcertresponse are you expecting something different? >>> >>> Thanks, >>> -Syed >>> >>> >>> On Wed 29 Jan 2014 11:06:23 AM EST, Syed Ahmed wrote: >>>> Yes Animesh. >>>> >>>> -Syed >>>> >>>> On Wed 29 Jan 2014 10:54:02 AM EST, Animesh Chaturvedi wrote: >>>>> Can you provide the fix today? >>>>> >>>>> Thanks >>>>> Animesh >>>>> >>>>> On Jan 29, 2014, at 7:36 AM, "Syed Ahmed" >>>>> <syed1.mush...@gmail.com> >>>>> wrote: >>>>> >>>>>> Hi, >>>>>> >>>>>> I'll check this and fix it. This was in the 4.3 release. >>>>>> >>>>>> Thanks, >>>>>> -Syed >>>>>> >>>>>> On Tue 28 Jan 2014 07:31:51 PM EST, Alena Prokharchyk wrote: >>>>>>> Hi Murali, >>>>>>> >>>>>>> Doing some integration work with ssl cert stuff, and found that >>>>>>> Apis you¹ve submitted on behalf of Syed, have incorrect response >>>>>>> name format (should be all lower cased, and end up with >>>>>>> ³response²). Can you please check? Also what release it went to? >>>>>>> If it was released like that, and people started using it, we >>>>>>> can¹t change it as we will break api compatibility. >>>>>>> >>>>>>> Thanks, >>>>>>> Alena. >>>>>> >>>>>> >>>> >>>> >>> >>> >> > >