[GitHub] cloudstack pull request: CLOUDSTACK-7986 [F5 LB] Failed to execute...

2014-11-28 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/47 CLOUDSTACK-7986 [F5 LB] Failed to execute IPAssocCommand due to com.clou... added 3 new method to strip partition information from VirtualServer, LBPool, VLAN api response. With BigIP

[GitHub] cloudstack pull request: CLOUDSTACK-7986 [F5 LB] Failed to execute...

2014-12-08 Thread sudhansu7
Github user sudhansu7 closed the pull request at: https://github.com/apache/cloudstack/pull/47 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] cloudstack pull request: CLOUDSTACK-7986 [F5 LB] Failed to execute...

2014-12-08 Thread sudhansu7
Github user sudhansu7 commented on the pull request: https://github.com/apache/cloudstack/pull/47#issuecomment-66239667 Thanks Rajani. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] cloudstack pull request: BUG-ID: CLOUDSTACK-8483 - Private templat...

2015-05-30 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/334 BUG-ID: CLOUDSTACK-8483 - Private template not visible in project added new 'projectId' parameter in createTemplate command and based current user, account and projectid decide the owner

[GitHub] cloudstack pull request: BUG-ID: CLOUDSTACK-8484 - Hosts without t...

2015-05-30 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/335 BUG-ID: CLOUDSTACK-8484 - Hosts without tag are not listed while list… …ing the hosts for migration for instance with tag While preparing the suitable hosts we are accidentally

[GitHub] cloudstack pull request: BUG-ID: CLOUDSTACK-8484 - Hosts without t...

2015-07-02 Thread sudhansu7
Github user sudhansu7 commented on the pull request: https://github.com/apache/cloudstack/pull/335#issuecomment-118007318 @remibergsma @runseb This issue exists in 4.5 , hence fixed it in 4.5. this can me merged to master as well. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: BUG-ID: CLOUDSTACK-8483 - Private templat...

2015-07-02 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/334#discussion_r33769718 --- Diff: api/src/org/apache/cloudstack/api/command/user/template/CreateTemplateCmd.java --- @@ -215,33 +221,39 @@ public long getEntityOwnerId

[GitHub] cloudstack pull request: BUG-ID: CLOUDSTACK-8483 - Private templat...

2015-07-02 Thread sudhansu7
Github user sudhansu7 commented on the pull request: https://github.com/apache/cloudstack/pull/334#issuecomment-118006807 @karuturi When a user switches to project view the ownership changes to project account. So any resource created inside project view should be mapped

[GitHub] cloudstack pull request: BUG-ID: CLOUDSTACK-8484 - Hosts without t...

2015-07-03 Thread sudhansu7
Github user sudhansu7 commented on the pull request: https://github.com/apache/cloudstack/pull/335#issuecomment-118331438 @remibergsma @karuturi I have added marvin test for this issue. Test result from local setup: Sudhansus-MAC:cloudstack sudhansu$ nosetests

[GitHub] cloudstack pull request: CLOUDSTACK-8919: Slow UI response while l...

2015-10-29 Thread sudhansu7
Github user sudhansu7 commented on the pull request: https://github.com/apache/cloudstack/pull/897#issuecomment-152273822 @remibergsma @runseb I have below performance analysis, which I did some time back. I trying to find out the dbdump to do a real test. But below analysis

[GitHub] cloudstack pull request: CLOUDSTACK-8815 : Issues with cloudstack-...

2015-09-10 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/799 CLOUDSTACK-8815 : Issues with cloudstack-management init script When a management server is halted or rebooted, the cloudstack-management init script does not successfully kill

[GitHub] cloudstack pull request: CLOUDSTACK-8917 : Instance tab takes long...

2015-09-28 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/894 CLOUDSTACK-8917 : Instance tab takes long time to load with 12K Vms modified sql that is used for retrieving vm count . In load test environment listVirtualmachine takes 8-11 sec

[GitHub] cloudstack pull request: CLOUDSTACK-8815 : Issues with cloudstack-...

2015-09-28 Thread sudhansu7
Github user sudhansu7 commented on the pull request: https://github.com/apache/cloudstack/pull/799#issuecomment-143724455 @remibergsma rebased against current master and also squashed 4 commits to one. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: CLOUDSTACK-8815 : Issues with cloudstack-...

2015-09-21 Thread sudhansu7
Github user sudhansu7 commented on the pull request: https://github.com/apache/cloudstack/pull/799#issuecomment-141945369 @bhaisaab For CentOS 7 we have we cloud-management.service and cloud-management.config files. I don't see any other .rc file for cloud-management service

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-04-24 Thread sudhansu7
Github user sudhansu7 commented on the pull request: https://github.com/apache/cloudstack/pull/1516#issuecomment-213954430 @DaanHoogland Thanks for the review. I have made necessary changes. I think in t=your test setup op_host_capacity does not have capacity details, for which

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-04-24 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r60844257 --- Diff: engine/schema/src/com/cloud/capacity/dao/CapacityDaoImpl.java --- @@ -962,35 +962,58 @@ public boolean removeBy(Short capacityType, Long

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-04-22 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/1516 CLOUDSTACK-9366: Capacity of one zone-wide primary storage ignored Disable and Remove Host operation disables the primary storage capacity. Steps to replicate: Base Condition

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-19 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63894357 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-19 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63893791 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-19 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63895181 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-19 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63893826 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-13 Thread sudhansu7
Github user sudhansu7 commented on the pull request: https://github.com/apache/cloudstack/pull/1516#issuecomment-218960848 @rhtyd Jenkin build is failing for cloud-util tests. Below is the snippet. Could you please check

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-18 Thread sudhansu7
Github user sudhansu7 commented on the pull request: https://github.com/apache/cloudstack/pull/1516#issuecomment-219995275 @swill Thanks for the review. I have updated the test. Now its green. finally. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-18 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63715139 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-18 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63731204 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-05-05 Thread sudhansu7
Github user sudhansu7 commented on the pull request: https://github.com/apache/cloudstack/pull/1516#issuecomment-217105794 @rhtyd @swill The branch has been re-based against master and also squashed multiple commits to a single. --- If your project is set up for it, you can

[GitHub] cloudstack issue #1797: CLOUDSTACK-9630: Cannot use listNics API as advertis...

2017-01-24 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/1797 @pdion891 The first listNic example is incorrect code. Which has been fixed in this PR. You can use listVirtualMachine command to see the nic response. As per code

[GitHub] cloudstack issue #1246: CLOUDSTACK-9165 unable to use reserved IP range in a...

2017-03-02 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/1246 LGTM for code change. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack issue #892: CLOUDSTACK-8910: The reserved_capacity field increase...

2017-03-01 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/892 LGTM for code changes. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] cloudstack pull request #1795: CLOUDSTACK-9625:Unable to scale VM from any o...

2017-02-26 Thread sudhansu7
Github user sudhansu7 closed the pull request at: https://github.com/apache/cloudstack/pull/1795 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so

[GitHub] cloudstack issue #1795: CLOUDSTACK-9625:Unable to scale VM from any offering...

2017-02-26 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/1795 Closing this PR as this is already fixed in master. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] cloudstack pull request #1763: CLOUDSTACK-9594: API "list templates template...

2016-11-11 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/1763 CLOUDSTACK-9594: API "list templates templatefilter=all" reveals all API "list templates templatefilter=all" reveals all templates. Using a "list templates tem

[GitHub] cloudstack pull request #1795: CLOUDSTACK-9625:Unable to scale VM from any o...

2016-11-28 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/1795 CLOUDSTACK-9625:Unable to scale VM from any offering to a dynamic offering 1.create a custom service offering. 2.stop the running vm 3.scale the vm offering from small to custom

[GitHub] cloudstack pull request #1796: CLOUDSTACK-9626: Instance fails to start afte...

2016-11-28 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/1796 CLOUDSTACK-9626: Instance fails to start after unsuccesful compute offering upgrade. You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] cloudstack pull request #1797: CLOUDSTACK-9630: Cannot use listNics API as a...

2016-11-29 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/1797 CLOUDSTACK-9630: Cannot use listNics API as advertised listNics API for a VM, "type" was not returned within API response. EXPECTE

[GitHub] cloudstack issue #1797: CLOUDSTACK-9630: Cannot use listNics API as advertis...

2016-12-09 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/1797 @jburwell I have modified the base branch to 4.9 . Also added a marvin test to check nic details. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack issue #1763: CLOUDSTACK-9594: API "list templates templatefilter=...

2016-12-06 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/1763 @abhinandanprateek Below is test scenario and result. I have also reverted the changes introduced in CLOUDSTACK-9376. Test Scenario: Domain: SUDDOMAIN Domain Admin

[GitHub] cloudstack issue #1811: CLOUDSTACK-9649: In the management server log there ...

2016-12-07 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/1811 @borisstoyanov This appears in general in Basic Zone with VMware Host. The exception is raised in 'RouterStatusMonitor-' thread. --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request #1805: CLOUDSTACK-9637: Template create from snapsho...

2016-12-01 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/1805 CLOUDSTACK-9637: Template create from snapshot does not populate vm_t… **ISSUE** Template create from snapshot does not populate vm_template_details

[GitHub] cloudstack issue #1763: CLOUDSTACK-9594: API "list templates templatefilter=...

2016-12-01 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/1763 @rhtyd @jburwell I think the fix for CLOUDSTACK-9376 will not address below scenario. I will test this and confirm. 1. templatefilter=all or isofilter=all is applicable

[GitHub] cloudstack pull request #1810: CLOUDSTACK-9647: NIC adapter type becomes e10...

2016-12-02 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/1810 CLOUDSTACK-9647: NIC adapter type becomes e1000 , even after changing… **Issue:** NIC adapter type becomes e1000 , even after changing the global parameter

[GitHub] cloudstack issue #1796: CLOUDSTACK-9626: Instance fails to start after unsuc...

2016-12-02 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/1796 @rhtyd modified the base branch to 4.9. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] cloudstack pull request #1797: CLOUDSTACK-9630: Cannot use listNics API as a...

2016-11-29 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1797#discussion_r90009164 --- Diff: server/src/com/cloud/api/ApiResponseHelper.java --- @@ -3461,10 +3470,21 @@ public NicResponse createNicResponse(Nic result

[GitHub] cloudstack pull request #914: BUG-ID CLOUDSTACK-8939 VM Snapshot size with m...

2016-12-04 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/914#discussion_r90809764 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java --- @@ -3206,25 +3206,33 @@ public long

[GitHub] cloudstack pull request #1811: CLOUDSTACK-9649: In the management server log...

2016-12-02 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/1811 CLOUDSTACK-9649: In the management server log there is an error ISSUE In the management server log there is an error 2016-10-01 00:07:31,670 ERROR

[GitHub] cloudstack pull request #1844: CLOUDSTACK-9668 : disksizeallocated of Primar...

2016-12-21 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/1844 CLOUDSTACK-9668 : disksizeallocated of PrimaryStorage is different fr… …om the total size of a volume update capacity if current allocated is different from used bytes in DB

[GitHub] cloudstack pull request #1849: CLOUDSTACK-9690: Scale CentOS7 VM fails with ...

2016-12-21 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/1849 CLOUDSTACK-9690: Scale CentOS7 VM fails with error Scale CentOS7 VM fails with error "Cannot scale up the vm because of memory constraint violation" When creating VM fro

[GitHub] cloudstack pull request #1850: CLOUDSTACK-9694: Unable to limit the Public I...

2016-12-21 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/1850 CLOUDSTACK-9694: Unable to limit the Public IPs in VPC Unable to limit the Public IPs in VPC. In VPC network, while acquiring the IP addresses, in the resource_count table

[GitHub] cloudstack issue #1849: CLOUDSTACK-9690: Scale CentOS7 VM fails with error

2016-12-22 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/1849 @rhtyd Thanks for the Review. I have amended the code as per your suggestion. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] cloudstack pull request #1869: CLOUDSTACK-9701: When host is disabled/remove...

2016-12-26 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/1869 CLOUDSTACK-9701: When host is disabled/removed, capacity_type for loc… …al storage in op_host_capacity is still enabled Description = When a host is disabled

[GitHub] cloudstack pull request #2015: CLOUDSTACK-9843 : Performance improvement of ...

2017-03-23 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/2015 CLOUDSTACK-9843 : Performance improvement of SSHHelper A delay of 1 sec has been introduced in SSHHelper Class. This is a fail safe code. Removing this will improves the performance

[GitHub] cloudstack issue #2019: CLOUDSTACK-9851 travis CI build failure after merge ...

2017-03-27 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/2019 LGTM for code change. Most of the recent travis CI is failing for this. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] cloudstack issue #2002: CLOUDSTACK-9831: Previous pod_id still remains in th...

2017-03-30 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/2002 @serg38 Added changes for failure case. If migration fails the old pod id will be restored. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request #2012: CLOUDSTACK-9842: Unable to map root volume us...

2017-03-21 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/2012 CLOUDSTACK-9842: Unable to map root volume usage to VM If a VM is cold migrated the vm_instance_id and uuid of volume is nullified. So there is no link between volume and vm

[GitHub] cloudstack issue #1797: CLOUDSTACK-9630: Cannot use listNics API as advertis...

2017-03-17 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/1797 @pdion891 removed broadcasturi from api response. Also added marvin test . --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] cloudstack pull request #914: BUG-ID CLOUDSTACK-8939 VM Snapshot size with m...

2017-04-13 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/914#discussion_r111375928 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java --- @@ -3206,25 +3206,33 @@ public long

[GitHub] cloudstack pull request #2002: CLOUDSTACK-9831: Previous pod_id still remain...

2017-03-13 Thread sudhansu7
GitHub user sudhansu7 opened a pull request: https://github.com/apache/cloudstack/pull/2002 CLOUDSTACK-9831: Previous pod_id still remains in the vm_instance table Previous pod_id still remains in the vm_instance table after VM migration with migrateVirtualMachineWithVolume

[GitHub] cloudstack issue #1850: CLOUDSTACK-9694: Unable to limit the Public IPs in V...

2017-03-08 Thread sudhansu7
Github user sudhansu7 commented on the issue: https://github.com/apache/cloudstack/pull/1850 @koushik-das Added marvin test. Please review. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does