Repository: cloudstack-git
Description
---
CLOUDSTACK-4686: Fixed volume limit for domain
Diffs
-
test/integration/component/test_resource_limits.py 833723c
Diff: https://reviews.apache.org/r/14200/diff/
Testing
---
Thanks,
Girish Shilamkar
.
Repository: cloudstack-git
Description
---
Storage type was set to local due to which tests failed.
Diffs
-
test/integration/component/test_reset_ssh_keypair.py 8b499d0
Diff: https://reviews.apache.org/r/14230/diff/
Testing
---
Thanks,
Girish Shilamkar
: https://reviews.apache.org/r/14335/diff/
Testing
---
Thanks,
Girish Shilamkar
Hello,
Egress rules tests rely on accessing virtual router VR after creating a
network. I have often seen that VR is not immediately accessible.
A ssh to VR fails, I think it takes a while for the network to come up and VR
can be ssh'd. This happens even though the VR is in "Running"
state.
So
, Marcus Sorensen wrote:
> In the past, with KVM, the agent doesn't show the router as running until
> it can ssh to it. You can watch the agent poll if it is debug logging. Does
> this issue not apply to KVM, or is it a regression?
> On Sep 29, 2013 11:38 PM, "Girish Shilamka
router boot up.
> Take the max interval as 10 times of boot up time.
> Repeat the ssh for each boot up interval.
>
> Thanks,
> Jayapal
>
> On 30-Sep-2013, at 1:38 PM, Girish Shilamkar
> wrote:
>
>> Marcus,
>>
>> I guess it no longer applies to even KVM
budumuru.
Bugs: CLOUDSTACK-4637
Repository: cloudstack-git
Description
---
Add logic to wait for router to boot.
Diffs
-
test/integration/component/test_egress_fw_rules.py 5c18f9c
Diff: https://reviews.apache.org/r/14469/diff/
Testing
---
Thanks,
Girish Shilamkar
for network gc
LB rules should be cleared. Added that check instead of router
being in stopped state.
Diffs (updated)
-
test/integration/component/test_vpc_network.py 970a625
Diff: https://reviews.apache.org/r/14335/diff/
Testing
---
Thanks,
Girish Shilamkar
value remains same i.e. 100.
- Girish
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14469/#review26761
---
On Oct. 3, 2013, 2:33 p.m., Giris
onError: The virtual machine 974d1275-b747-441c-83b2-1795de4d87df failed
to start even after 10 minutes
------
Thanks,
Girish Shilamkar
Description
---
Renamed testcase name and also initialised _cleanup so that
it does not break on non-NS Cloudstack setup.
Diffs
-
test/integration/component/test_netscaler_nw_off.py 3139257
Diff: https://reviews.apache.org/r/14557/diff/
Testing
---
Thanks,
Girish Shilamkar
d
output.
- Girish
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14556/#review26816
---
On Oct. 9, 2013, 9:50 a.m., Girish Shilam
--
Ran 11 tests in 2154.059s
OK
Thanks,
Girish Shilamkar
3139257
Diff: https://reviews.apache.org/r/14557/diff/
Testing
---
Thanks,
Girish Shilamkar
Hello Prasanna,
+1 to the new marvin architecture.
The proposed architecture will make marvin lot more stable, remove defining
data in code and will be easier to add testcases.
Can we have a run in jenkins.cloudstack.org to ensure that the existing tests
won't break with new marvin ?
How much
I have been hitting this issue quite often since couple of weeks.
I just switched the management server and so far haven't seen it.
Maybe updating httplib might help.
Regards,
Girish
On 16-Oct-2013, at 11:04 AM, Prasanna Santhanam wrote:
> On Wed, Oct 16, 2013 at 05:11:46AM +
Lets track this issue here https://issues.apache.org/jira/browse/CLOUDSTACK-4846
Regards,
Girish
On 16-Oct-2013, at 11:30 AM, Prasanna Santhanam wrote:
> httplib is used indirectly through requests. I'd try reproduce this
> by throttling 1000s of requests through a single mgmt server. It is
> i
Hello,
Sometimes while running automation testcases tests just fail with following
error:
ConnectionError: HTTPConnectionPool(host='10.147.59.65', port=8080): Max
retries exceeded with url: /client/api?
apiKey=lcgWQ9XrjyIyPjs
: Master failover ... ok
--
Ran 26 tests in 15127.540s
OK
Thanks,
Girish Shilamkar
> On Oct. 8, 2013, 11:14 a.m., abhinav roy wrote:
> > Ship It!
Can someone commit this patch.
- Girish
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14334/#review26770
-
cause for this issue. Adding complete server calls and debug information
> will help to debug more i believe?
>
> http://stackoverflow.com/questions/1767934/why-am-i-getting-this-error-in-python-httplib
>
> Check above and you can get more information for this issue to delve further.
>
test/integration/component/test_mm_project_limits.py PRE-CREATION
Diff: https://reviews.apache.org/r/14872/diff/
Testing
---
Thanks,
Girish Shilamkar
/component/test_mm_project_limits.py PRE-CREATION
Diff: https://reviews.apache.org/r/14873/diff/
Testing
---
Thanks,
Girish Shilamkar
On Oct. 23, 2013, 9:08 a.m., Gaurav Aradhye wrote:
> > The naming of the tests should be improved. test_01/02/03 doesn't tell much
> > about the test. Also is the ordering strictly required? Can I reorder the
> > tests and still expect the tests to pass? If not, we should remove the 01 ,
> > 0
gt;> From: Prasanna Santhanam [mailto:t...@apache.org]
>> Sent: Wednesday, October 23, 2013 4:08 PM
>> To: CloudStack Dev
>> Subject: [ANNOUNCE] New Committer: Shilamkar, Girish
>>
>> The Project Management Committee (PMC) for Apache CloudStack has asked
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14459/#review27595
---
Ship it!
Ship It!
- Girish Shilamkar
On Oct. 28, 2013, 6:02 a.m
automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/14199/
> ---
>
> (Updated Oct. 28, 2013, 5:53 a.m.)
>
>
> Review request for cloudstack, Girish Shilamkar and suresh sadhu.
>
>
://reviews.apache.org/r/15082/diff/
Testing (updated)
---
Yes
Thanks,
Girish Shilamkar
---
Thanks,
Girish Shilamkar
://reviews.apache.org/r/15082/diff/
Testing
---
Yes
Thanks,
Girish Shilamkar
/
Testing
---
Yes
Thanks,
Girish Shilamkar
error insufficient capacity. So
remove this test which is invalid.
Diffs
-
test/integration/component/test_shared_network_offering.py 85f0892
Diff: https://reviews.apache.org/r/15115/diff/
Testing
---
Thanks,
Girish Shilamkar
tps://reviews.apache.org/r/15127/#comment54310>
If we are specifying vmid then no need to specify listall=True. Rest looks
good.
- Girish Shilamkar
On Oct. 31, 2013, 12:41 p.m., Gaurav Aradhye wrote:
>
> ---
> This is a
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15127/#review27906
---
Ship it!
Ship It!
- Girish Shilamkar
On Oct. 31, 2013, 1:48 p.m
test_createSharedNetwork_usedVlan (test_shared_networks.TestSharedNetworks)
Thanks,
Girish Shilamkar
> On Oct. 24, 2013, 9:24 a.m., Prasanna Santhanam wrote:
> > 0780604 master
Shouldn't this fix also go to 4.2 branch ?
- Girish
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14874/#review27
to 4.2 i
> believe.
>
> Santhosh
> ____
> From: Girish Shilamkar [nore...@reviews.apache.org] on behalf of Girish
> Shilamkar [gir...@clogeny.com]
> Sent: Friday, November 01, 2013 3:07 AM
> To: Prasanna Santhanam
> Cc: Santhos
rs/santhoshe/
>
> Right now it seems that everything has been reviewed, so Santosh will need to
> submit his patches through review board and assign you as reviewer.
>
> Hope that helps,
>
> -sebastien
>
>
> On Nov 1, 2013, at 3:37 AM, Girish Shilamkar wrote:
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15307/#review28360
---
Ship it!
Ship It!
- Girish Shilamkar
On Nov. 7, 2013, 10:23 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15305/#review28362
---
Ship it!
Ship It!
- Girish Shilamkar
On Nov. 7, 2013, 10:22 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15342/#review28507
---
Ship it!
Ship It!
- Girish Shilamkar
On Nov. 8, 2013, 7:07 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14874/#review28510
---
4.2: ae852adabb70f43d6d349dd4ab56dc3116e10aed
- Girish Shilamkar
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15021/#review28512
---
4.2: e7b6ee10eda85a90c594c9de5d36cc062e069576
- Girish Shilamkar
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15175/#review28513
---
4.2: b18e730108e6a24e00c372e3c56c655ff51106ba
- Girish Shilamkar
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14648/#review28520
---
4.2: fba874120bb40ddeceacbc7ef51217f34d3749de
- Girish Shilamkar
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14595/#review28521
---
4.2: 6e1821f585df5c643873ab0ce1e7d91d1d94f00d
- Girish Shilamkar
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14869/#review28524
---
4.2: 87983e09e21c51dbef6b2a8a0252b039e32915ad
- Girish Shilamkar
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14875/#review28525
---
4.2: 3a31a7e6
- Girish Shilamkar
On Oct. 23, 2013, 1:57 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14925/#review28526
---
4.2: fba874120bb40ddeceacbc7ef51217f34d3749de
- Girish Shilamkar
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15208/#review28527
---
4.2: c76da53123f4130ac59bdf112171bc2e39af91d4
- Girish Shilamkar
different file.
- Girish Shilamkar
On Nov. 8, 2013, 7:16 a.m., Santhosh Edukulla wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
> On Nov. 8, 2013, 10:11 a.m., Girish Shilamkar wrote:
> > Santhosh,
> >
> > Please update the diff instead of uploading them as different file.
Also the commit header should be "CLOUDSTACK-5099: " This will ensure
that the jira ticket also gets
> On Nov. 8, 2013, 9:15 a.m., Girish Shilamkar wrote:
> > 4.2: fba874120bb40ddeceacbc7ef51217f34d3749de
>
> Sebastien Goasguen wrote:
> if these patches apply to master, can you make sure to also apply to
> master at the same time. thanks
They were already applied to
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15395/#review28638
---
Ship it!
Ship It!
- Girish Shilamkar
On Nov. 10, 2013, 7:28 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13841/#review28783
---
Ship it!
Ship It!
- Girish Shilamkar
On Oct. 30, 2013, 7:01 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14058/#review28782
---
Ship it!
Ship It!
- Girish Shilamkar
On Nov. 8, 2013, 1:48 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15453/#review28787
---
Ship it!
Ship It!
- Girish Shilamkar
On Nov. 12, 2013, 1:09 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15454/#review28793
---
Ship it!
Ship It!
- Girish Shilamkar
On Nov. 12, 2013, 1:39 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15475/#review28796
---
Ship it!
Ship It!
- Girish Shilamkar
On Nov. 13, 2013, 11:48
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15478/#review28805
---
Ship it!
Ship It!
- Girish Shilamkar
On Nov. 13, 2013, 2:06 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15478/#review28803
---
Ship it!
Ship It!
- Girish Shilamkar
On Nov. 13, 2013, 2:06 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13841/#review28810
---
Committed to 4.2, 4.3 and master
- Girish Shilamkar
On Oct. 30
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14058/#review28811
---
Committed to 4.2, 4.3 and master.
- Girish Shilamkar
On Nov. 8
Congrats Sanjeev !!
Regards,
Girish
On 14-Nov-2013, at 12:11 PM, Suresh Sadhu wrote:
> Congrats Sanjeev
>
> -Original Message-
> From: Sailaja Mada [mailto:sailaja.m...@citrix.com]
> Sent: 14 November 2013 11:38
> To: dev@cloudstack.apache.org
> Subject: RE: New Committer: Sanjeev Nee
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15513/#review28935
---
Ship it!
Ship It!
- Girish Shilamkar
On Nov. 14, 2013, 11:46
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15513/#review28937
---
Ship it!
Committed to 4.2, 4.3 and master
- Girish Shilamkar
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15514/#review28940
---
Ship it!
Committed to 4.2, 4.3 and master
- Girish Shilamkar
On
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15516/#review28944
---
Ship it!
Committed to 4.2, 4.3 and master
- Girish Shilamkar
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15570/#review28954
---
Ship it!
COmmitted to 4.2, 4.3 and master
- Girish Shilamkar
On
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15452/#review28958
---
Ship it!
Committed to 4.2, 4.3 and master
- Girish Shilamkar
On
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15569/#review28963
---
Ship it!
COmmitted to 4.2, 4.3 and master
- Girish Shilamkar
On
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15610/#review29010
---
Ship it!
Committed to 4.2, 4.3 and master
- Girish Shilamkar
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15610/#review29011
---
Ship it!
Committed to 4.2, 4.3 and master
- Girish Shilamkar
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15611/#review29015
---
Ship it!
Committed to 4.2, 4.3 and master
- Girish Shilamkar
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15645/#review29101
---
Ship it!
Committed to 4.2, 4.3 and master
- Girish Shilamkar
On
Hello Prasanna,
Tests using listportforwardingrules are failing with latest marvin.
listportforwardingrules failed, due to: errorCode: 431, errorText:Unable to
execute API command listportforwardingrules due to invalid value. Invalid
parameter id value=c9373d55-3d01-480d-98dc-2365d3927378 due to
/marvin/marvin/integration/lib/base.py ec1c34e
Diff: https://reviews.apache.org/r/11067/diff/
Testing
---
The tests which are not skipped are working.
Thanks,
Girish Shilamkar
one? Should the setting be
> > false? It looks like you are testing for the negative result?
>
> Girish Shilamkar wrote:
> The flag is true but we don't provide the hostname so it should pick up
> the configured instance name. There are different set of tests where fla
hostname for the instance without internal name
This addresses bug CLOUDSTACK-778.
Diffs (updated)
-
test/integration/component/test_custom_hostname.py PRE-CREATION
Diff: https://reviews.apache.org/r/10793/diff/
Testing
---
Tested and works fine.
Thanks,
Girish Shilamkar
unresolved max_value in test_project_limits.py
This addresses bug CLOUDSTACK-2472.
Diffs
-
test/integration/component/test_project_limits.py 17ddfc6
Diff: https://reviews.apache.org/r/11303/diff/
Testing
---
Thanks,
Girish Shilamkar
a typo in test_vpc_network.py
This addresses bug CLOUDSTACK-2473.
Diffs
-
test/integration/component/test_vpc_network.py dc53585
Diff: https://reviews.apache.org/r/11304/diff/
Testing
---
Thanks,
Girish Shilamkar
---
Remove garbage code which was added while resolving merge conflicts.
This addresses bug CLOUDSTACK-2474.
Diffs
-
test/integration/component/test_vpc_routers.py 763a4cb
Diff: https://reviews.apache.org/r/11306/diff/
Testing
---
Thanks,
Girish Shilamkar
Santhanam.
Description
---
Remove garbage code which was added while resolving merge conflicts.
This addresses bug CLOUDSTACK-2474.
Diffs
-
test/integration/component/test_vpc_routers.py 763a4cb
Diff: https://reviews.apache.org/r/11306/diff/
Testing
---
Thanks,
Girish
Hello,
While writing automation tests Changing account membership for VMs,
assignVirtualMachine API fails with
errorCode: 530, errorText:Failed to move vm Remove the Port forwarding
rules for this VM before assigning to another user.
This happens even though vm wasn't set any PF rule. We saw thi
---
listPortForwarding API returns an exception if the PF is deleted.
Changed testcases to handle this exception.
This addresses bug CLOUDSTACK-2577.
Diffs
-
test/integration/smoke/test_network.py 4a7bb44
Diff: https://reviews.apache.org/r/11328/diff/
Testing
---
Thanks,
Girish
is the expected behavior.
>
> ** **
>
> Thanks,
>
> Likitha ****
>
> ** **
>
> *From:* Girish Shilamkar [mailto:gir...@clogeny.com]
> *Sent:* Wednesday, May 22, 2013 12:40 PM
> *To:* dev@cloudstack.apache.org
> *Cc:* Likitha Shetty; Parth Jagirdar
>
,
Girish Shilamkar
/test_affinity_groups.py PRE-CREATION
Diff: https://reviews.apache.org/r/11067/diff/
Testing
---
The tests which are not skipped are working.
Thanks,
Girish Shilamkar
/test_affinity_groups.py PRE-CREATION
Diff: https://reviews.apache.org/r/11067/diff/
Testing
---
The tests which are not skipped are working.
Thanks,
Girish Shilamkar
---
Update ssh key location for vmware
This addresses bug CLOUDSTACK-1758.
Diffs
-
tools/marvin/marvin/integration/lib/utils.py 6892c41
Diff: https://reviews.apache.org/r/11956/diff/
Testing
---
Thanks,
Girish Shilamkar
---
Additional fixes were needed which were found during further testing.
This addresses bug CLOUDSTACK-1758.
Diffs
-
test/integration/smoke/test_ssvm.py d637f96
Diff: https://reviews.apache.org/r/11962/diff/
Testing
---
Thanks,
Girish Shilamkar
/diff/
Testing
---
Thanks,
Girish Shilamkar
-CREATION
tools/marvin/marvin/integration/lib/base.py 503ed64
Diff: https://reviews.apache.org/r/11067/diff/
Testing
---
The tests which are not skipped are working.
Thanks,
Girish Shilamkar
Hello Prasanna,
While adding rest of the affinity rules tests I had removed affinity rules test
from smoke. But while
going through various iterations on the review board this hunk to remove test
from smoke was dropped.
The same test is now included in component.
I think we should remove test_a
-git
Description
---
Tests in test_network_offering.py failed as Vlan option was false. Changed it
to true.
Diffs
-
test/integration/component/test_network_offering.py a3e87f6
Diff: https://reviews.apache.org/r/12241/diff/
Testing
---
Thanks,
Girish Shilamkar
Hello,
Some of the BVT test cases verify if load balancing rule set to round robin is
working or not.
It is seen that often LBing is not perfect round robin. If first request goes
to one vm the next does not always go to second,
when there are just two vms added to LB rule.
I vaguely recall di
Hello Likitha,
Some of the regression tests for changing domain membership failed.
http://jenkins.buildacloud.org/job/test-regression-matrix/34/suite=test_assign_vm/testReport/
Could you please confirm my following assumptions:
A domain admin can move vms only amongst its subdomains or between t
the move should be successful and not raise an exception.
>
> Thanks,
> Likitha
>
> From: Girish Shilamkar [mailto:gir...@clogeny.com]
> Sent: Monday, July 08, 2013 5:28 PM
> To: Likitha Shetty
> Cc: dev@cloudstack.apache.org
> Subject: Query about moving vms across d
Hello,
In order to create shared network StartIp/endIp/gateway/netmask are required. I
am looking at an old test case which creates shared network without IP range
and deploys vm in it. It of course fails as IP range is required while creating
shared network.
Has anything related to shared netw
/component/test_network_offering.py a3e87f6
Diff: https://reviews.apache.org/r/12497/diff/
Testing
---
Thanks,
Girish Shilamkar
1 - 100 of 230 matches
Mail list logo
