Re: Review Request 12427: Fix for CLOUDSTACK-3365 and CLOUDSTACK-2536

2013-07-19 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12427/#review23492 --- The changes related to CLOUDSTACK-2536 is not required.

Re: Review Request 12427: Fix for CLOUDSTACK-3365 and CLOUDSTACK-2536

2013-07-17 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12427/ --- (Updated July 17, 2013, 7:37 a.m.) Review request for cloudstack, Abhinandan

Re: Review Request 12427: Fix for CLOUDSTACK-3365 and CLOUDSTACK-2536

2013-07-17 Thread Harikrishna Patnala
On July 17, 2013, 5:38 a.m., Nitin Mehta wrote: server/src/com/cloud/configuration/Config.java, line 202 https://reviews.apache.org/r/12427/diff/3/?file=319814#file319814line202 Please clarify - Are they not at cluster level anymore ? These are cluster level parameters but can be

Re: Review Request 12427: Fix for CLOUDSTACK-3365 and CLOUDSTACK-2536

2013-07-16 Thread Nitin Mehta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12427/#review23239 --- server/src/com/cloud/alert/AlertManagerImpl.java

Re: Review Request 12427: Fix for CLOUDSTACK-3365 and CLOUDSTACK-2536

2013-07-10 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12427/ --- (Updated July 10, 2013, 10:14 a.m.) Review request for cloudstack, Abhinandan

Re: Review Request 12427: Fix for CLOUDSTACK-3365 and CLOUDSTACK-2536

2013-07-10 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12427/ --- (Updated July 10, 2013, 10:19 a.m.) Review request for cloudstack, Abhinandan