Mailing-List: contact cloudstack-users-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: cloudstack-users@incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of Jessica.Wang@citrix.com
 designates 66.165.176.63 as permitted sender)
From: Jessica Wang <Jessica.Wang@citrix.com>
To: "Olga Smola (olya.smola@gmail.com)" <olya.smola@gmail.com>
CC: Brian Federle <Brian.Federle@citrix.com>, Sonny Chhen
	<Sonny.Chhen@citrix.com>, Will Chan <will.chan@citrix.com>, Pranav Saxena
	<pranav.saxena@citrix.com>, "cloudstack-users@incubator.apache.org"
	<cloudstack-users@incubator.apache.org>, "David Nalley (david@gnsa.us)"
	<david@gnsa.us>, Nitin Mehta <Nitin.Mehta@citrix.com>
Date: Tue, 12 Jun 2012 14:27:58 -0700
Subject: RE: CloudStack bugs, CS-14896 (& is not encoded correctly when user
 tries to create an instance)
Thread-Topic: CloudStack bugs, CS-14896 (& is not encoded correctly when
 user tries to create an instance)
Thread-Index: Ac1IovSV+U54gOyGRjWsDG9H7dWQLQAE6g1AAAOYx3AAAxxusAACV7WQ
Message-ID: 
 <8EC081586F1D7C41931517E802E94732013042392699@SJCPMAILBOX01.citrite.net>
References: 
 <CAJkj-8E1HUcoOmrQR0Z_BG_pjV7tytMkDJs6u1v8wCUkYrwvcw@mail.gmail.com>
 <CE52B3BAA8221C4A91FA40D3C8194ED2E91A2E7799@BANPMAILBOX01.citrite.net>
 <CAJkj-8HyufB3RM4UdVRDc_1DRewYN=0uW4iZv5UW9PVRHQrOkg@mail.gmail.com>
 <CE52B3BAA8221C4A91FA40D3C8194ED2E91A2E77CA@BANPMAILBOX01.citrite.net>
 <CAJkj-8G2xx_tygvp-b-yACNvt2-mT+nYu8xJPT0KHjQpjpnJ6Q@mail.gmail.com>
 <CAJkj-8GC9b0wXfZjYDCBJ8xq5p0jFdkZht3jT3xVWwOQLvTTRw@mail.gmail.com>
 <CE52B3BAA8221C4A91FA40D3C8194ED2E91A2E79C4@BANPMAILBOX01.citrite.net>
 <67EF18FDCA335F489B366120481AB6C5EE3A77B767@BANPMAILBOX01.citrite.net>
 <CAKprHVaik6G0osVDTA3WjPSesGPJiWUbKsH1Ta=n9YUj3UzeUA@mail.gmail.com>
 <67EF18FDCA335F489B366120481AB6C5EE3A77B7B4@BANPMAILBOX01.citrite.net>
 <8EC081586F1D7C41931517E802E947320130423925C6@SJCPMAILBOX01.citrite.net>
 <67EF18FDCA335F489B366120481AB6C5EE3A77B7D1@BANPMAILBOX01.citrite.net>
In-Reply-To: 
 <67EF18FDCA335F489B366120481AB6C5EE3A77B7D1@BANPMAILBOX01.citrite.net>
Accept-Language: en-US
Content-Language: en-US
acceptlanguage: en-US
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0

Olga,

http://bugs.cloudstack.org/browse/CS-14896

(1) CS-14896 is a cloudstack 2.2.x bug, NOT a cloudstack 3.0.x bug.

cloudstack 2.2.x UI is totally different from cloudstack 3.0.x UI (UI appea=
rance is different , UI architecture is different, UI code is different).

The bug exists only in cloudstack 2.2.x UI, but not in cloudstack 3.0.x UI.

Cloudstack 2.2.x is not in apache. Only cloudstack 3.0.x is in apache.=20
Please do not fix bugs whose version is 2.2.x.

(2) I did check whether cloudstack 3.0.x UI happens to have the same bug.

The bug says ampersand is not encoded correctly when trying to create an in=
stance, but ampersand is encoded correctly when trying to create an account=
 (in 2.2.x UI).

In 3.0.x UI, it's the opposite =3D> ampersand is encoded correctly when try=
ing to create an instance, but ampersand is not encoded correctly when tryi=
ng to create an account.=20

So, I tried to fix account creation in 3.0.x UI.
Your fix (which is for 3.0.x UI) is to forbid special characters like amper=
sand being used in account name when creating an account.
But cloudstack's server-side (API) does accept special characters in accoun=
t name.
Therefore, UI shouldn't forbid special characters in account name.
Instead, UI should encode account name before it's being passed to API call=
.

I just checked in a fix(to encode account name before it's being passed to =
API call).

Jessica


-----Original Message-----
From: Pranav Saxena=20
Sent: Tuesday, June 12, 2012 12:34 PM
To: Jessica Wang; Olga Smola (olya.smola@gmail.com); cloudstack-users@incub=
ator.apache.org; David Nalley (david@gnsa.us)
Cc: Brian Federle; Sonny Chhen; Will Chan
Subject: RE: CloudStack bugs, CS-14896

Hi Olga,=20

I have reviewed your patch for this bug as well - http://bugs.cloudstack.or=
g/browse/CS-14230 . This is what I have to say -=20

1) The patch looks good but since you had attached the diff file , there wa=
s misplacement of the curly braces which was making the situation a little =
tedious for us to put your patch at the appropriate places , eventually lea=
ding to unnecessary errors . So will it be possible for you to follow the g=
it workflow instead of attaching diffs ? This would help us in reviewing yo=
ur code more easily and would make the process even more faster.

2) Is it possible for you to make your own private branch ? ( I am not sure=
 about this )

Otherwise , the patch is perfect and works absolutely fine . Good work !:)

Thanks & Regards,
Pranav=20

-----Original Message-----
From: Jessica Wang=20
Sent: Tuesday, June 12, 2012 11:31 PM
To: Pranav Saxena; Olga Smola (olya.smola@gmail.com); cloudstack-users@incu=
bator.apache.org; David Nalley (david@gnsa.us)
Cc: Brian Federle; Sonny Chhen; Will Chan
Subject: RE: CloudStack bugs, CS-14896

CS-14111 (There is no option to Edit IP ranges of Management Traffic) is in=
valid.
I've written down my comment and close the bug as "By design".

http://bugs.cloudstack.org/browse/CS-14111

-----Original Message-----
From: Pranav Saxena
Sent: Tuesday, June 12, 2012 9:39 AM
To: Olga Smola (olya.smola@gmail.com); cloudstack-users@incubator.apache.or=
g; David Nalley (david@gnsa.us)
Cc: Jessica Wang; Brian Federle; Sonny Chhen; Will Chan
Subject: RE: CloudStack bugs, CS-14896

Hi Olga ,

I have reviewed the following bugs fixed by you and this is what I have fou=
nd -=20

1) http://bugs.cloudstack.org/browse/CS-14511 - This bug seems to be alread=
y fixed in 3.0.3 release . Though your patch of code was perfect.
2) http://bugs.cloudstack.org/browse/CS-13535 - This patch of code works pe=
rfect. The bug was reproducible and the spinner never stopped once you crea=
ted a child domain with the same name. (Brian you could verify it once more=
).
3) http://bugs.cloudstack.org/browse/CS-12478 - This has also been verified=
. The patch looks good according to me .

The patches for the other bugs have already been merged or are getting revi=
wed.

Thanks & Regards,
Pranav=20


-----Original Message-----
From: David Nalley [mailto:david@gnsa.us]
Sent: Tuesday, June 12, 2012 7:25 PM
To: Pranav Saxena
Cc: Jessica Wang; Brian Federle; Sonny Chhen; Will Chan
Subject: Re: CloudStack bugs, CS-14896

Ohhh please do this!

If you publicly review all of her existing patches I'll happily propose you=
 as a committer. (and I'll happily take care of merges too should Jessica/B=
rian/Sonny not have issues with this process.)

--David

On Tue, Jun 12, 2012 at 9:43 AM, Pranav Saxena <pranav.saxena@citrix.com> w=
rote:
> Hi Olga,
>
> I am also a =A0UI developer here. I have reviewed most of your diffs whic=
h you have attached with the respective bug tickets and will look at the re=
maining ones very soon .They look good to me. I 'll also ask =A0Jessica /Br=
ian ( Senior UI developers here ) to merge your patches once they are also =
satisfied with your code changes .In case there are any issues , they'll le=
t you know regarding any further modifications required.
>
> Thanks & Regards,
> Pranav Saxena
>
> -----Original Message-----
> From: Nitin Mehta [mailto:Nitin.Mehta@citrix.com]
> Sent: Tuesday, June 12, 2012 5:40 PM
> To: Olga Smola; cloudstack-dev@incubator.apache.org; Jessica Wang
> Subject: RE: CloudStack bugs, CS-14896
>
> This is good work Olga. Were you able to subscribe to the lists I=20
> forwarded you below ? =A0If not, I advise you to do so asap. There is a=20
> way for you to submit your work in form of patches. Refer to the wiki=20
> http://wiki.cloudstack.org/display/dev/Git+workflow+in+the+brave+new+w
> orld#Gitworkflowinthebravenewworld-Non-committerworkflow
> I am copying you on the dev list to get you going and Jessica is the main=
tainer of the UI component so she should be able to help review your work b=
ut, you need to follow the procedure in the wiki above to submit your work.
> Also once again I request you to subscribe to the Distribution lists belo=
w.
>
>
> Thanks,
> -Nitin
>
> From: Olga Smola [mailto:olya.smola@gmail.com]
> Sent: Tuesday, June 12, 2012 4:09 PM
> To: Nitin Mehta
> Subject: Re: CloudStack bugs, CS-14896
>
> Hi Nitin,
>
> could you please help me to ask somebody to review the tickets I've alrea=
dy completed?
> I've started to work on CloudStack bugs a week ago and I have solutions f=
or:
> http://bugs.cloudstack.org/browse/CS-14511
> http://bugs.cloudstack.org/browse/CS-13535
> http://bugs.cloudstack.org/browse/CS-14111
> http://bugs.cloudstack.org/browse/CS-14010 (was reviewed yesterday)
> http://bugs.cloudstack.org/browse/CS-12478
> http://bugs.cloudstack.org/browse/CS-14230
> http://bugs.cloudstack.org/browse/CS-14896
> and now I'm working on http://bugs.cloudstack.org/browse/CS-4460.
>
> I can spend some time to work on CloudStack, so if there is actual UI tic=
ket, I can work on it. But I don't know to whom should I write to get a fee=
dback.
>
> Best regards, Olga.
> On Mon, Jun 11, 2012 at 3:35 PM, Olga Smola <olya.smola@gmail.com<mailto:=
olya.smola@gmail.com>> wrote:
> Nitin,
>
> thanks, I'll write to you if any problems. Have a good day!
>
> On Mon, Jun 11, 2012 at 1:58 PM, Nitin Mehta <Nitin.Mehta@citrix.com<mail=
to:Nitin.Mehta@citrix.com>> wrote:
> Olga - Please do let me know in case you have a problem catching=20
> somebody to review your code
>
> From: Olga Smola
> [mailto:olya.smola@gmail.com<mailto:olya.smola@gmail.com>]
> Sent: Monday, June 11, 2012 4:26 PM
> To: Nitin Mehta
> Subject: Re: CloudStack bugs, CS-14896
>
> Hi Nitin,
>
> thanks a lot.
> On Mon, Jun 11, 2012 at 12:45 PM, Nitin Mehta <Nitin.Mehta@citrix.com<mai=
lto:Nitin.Mehta@citrix.com>> wrote:
> Olga - This is superb. Since CS has been donated to Apache Server foundat=
ion now you can more easily =A0contribute your code (initially as patch) an=
d become a committer.
> I would suggest you to subscribe to the following lists to get going.
>
> cloudstack-commits-subscribe@incubator.apache.org<mailto:cloudstack-co
> mmits-subscribe@incubator.apache.org>
> cloudstack-dev-subscribe@incubator.apache.org<mailto:cloudstack-dev-su
> bscribe@incubator.apache.org>
> cloudstack-users-subscribe@incubator.apache.org<mailto:cloudstack-user
> s-subscribe@incubator.apache.org>
>
> Just send an email to them to subscribe.
>
> Refer to this for creating a patch
> http://wiki.cloudstack.org/display/dev/Git+workflow+in+the+brave+new+w
> orld#Gitworkflowinthebravenewworld-Non-committerworkflow
>
>
> Thanks again,
> -Nitin
>
> From: Olga Smola
> [mailto:olya.smola@gmail.com<mailto:olya.smola@gmail.com>]
> Sent: Monday, June 11, 2012 2:59 PM
> To: Nitin Mehta
> Subject: CloudStack bugs, CS-14896
>
> Hi Nitin,
>
> excuse me to bother you. My name is Olga, and I'm interested in work on C=
loudStack. I'm UI developer and I can help to fix, extend project's UI. I s=
aw your post on http://bugs.cloudstack.org/browse/CS-14896 and I think that=
 my fix should help to you. Also I've fixed several other bugs (http://bugs=
.cloudstack.org (http://bugs.cloudstack.org/browse/CS-14511, http://bugs.cl=
oudstack.org/browse/CS-13535, http://bugs.cloudstack.org/browse/CS-14111), =
but I don't know how can I get the feedback. Maybe you know whom can I writ=
e to review my fixes?
>
> Best regards, Olga Smola.
>
>
>