Cannot edit confluence wiki page

[Shanika Ediriweera]

Hi,

I am interested in CLOUDSTACK-9804
 project for gsoc
2017.

But I am unable to edit the page
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Google+Summer+of+Code+2017
.

Please could someone guide me how to edit the page and add my name in the
"Candidates interested" column.

Thank you.

Best Regards,

Shanika Ediriweera
Undergraduate
Dept. of Computer Science & Eng.
University of Moratuwa

[GitHub] cloudstack issue #1726: CLOUDSTACK-9560 Root volume of deleted VM left unrem...

[cloudmonger]

Github user cloudmonger commented on the issue:

https://github.com/apache/cloudstack/pull/1726
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 456
 Hypervisor xenserver
 NetworkType Advanced
 Passed=104
 Failed=1
 Skipped=7

_Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0


**Failed tests:**
* test_routers_network_ops.py

 * test_03_RVR_Network_check_router_state Failing since 2 runs


**Skipped tests:**
test_01_test_vm_volume_snapshot
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_11_ss_nfs_version_on_ssvm
test_nested_virtualization_vmware
test_3d_gpu_support
test_deploy_vgpu_enabled_vm

**Passed test suits:**
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_routers.py
test_reset_vm_on_reboot.py
test_deploy_vms_with_varied_deploymentplanners.py
test_network.py
test_router_dns.py
test_non_contigiousvlan.py
test_login.py
test_deploy_vm_iso.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_volumes.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_vm_life_cycle.py
test_disk_offerings.py


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...

[nathanejohnson]

Github user nathanejohnson commented on the issue:

https://github.com/apache/cloudstack/pull/1955
  
I just re-depoyed and re-ran the test:

Test that libvirt properly created domain with scsi controller ... === 
TestName: test_01_verify_libvirt | Status : SUCCESS ===
ok
Verify that libvirt settings are as expected after a VM stop / start 
... === TestName: test_02_verify_libvirt_after_restart | Status : SUCCESS ===
ok
Verify that libvirt settings are expected after a disk add ... === 
TestName: test_03_verify_libvirt_attach_disk | Status : SUCCESS ===
ok
Verify that guest sees scsi controller and disks ... === TestName: 
test_04_verify_guest_lspci | Status : SUCCESS ===
ok
Update os type to Ubuntu, change vm details rootdiskController ... === 
TestName: test_05_change_vm_ostype_restart | Status : SUCCESS ===
ok
Verify that guest sees scsi controller and disks after switching ostype 
and rdc ... === TestName: test_06_verify_guest_lspci_again | Status : SUCCESS 
===
ok

--
Ran 6 tests in 785.914s

OK

@wido , I believe all of your concerns should be addressed.  trim is 
enabled, it's using an enum and not a string, and spurious log messages have 
been eliminated.

I have squashed all of the commits into one.  I'm at a point where I'm 
happy with the code.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #2001: CLOUDSTACK-9830 Fix DST bug in QuotaAlertMana...

[nathanejohnson]

GitHub user nathanejohnson opened a pull request:

https://github.com/apache/cloudstack/pull/2001

CLOUDSTACK-9830 Fix DST bug in QuotaAlertManagerTest

Replace the use of joda with Calendar, set explicitly to UTC when
doing date manipulation.

This fixes an issue where when running this test on a machine set to a 
local timezone that is about to switch to DST, the difference returns 0 versus 
1.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/myENA/cloudstack CLOUDSTACK-9830

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cloudstack/pull/2001.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #2001


commit 58877ecac2db7c1601c354d219777d394a316abf
Author: Nathan Johnson 
Date:   2017-03-12T01:17:22Z

CLOUDSTACK-9830 Fix DST bug in QuotaAlertManagerTest

Replace the use of joda with Calendar, set explicitly to UTC when
doing date manipulation.




---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: old wishes

[Rafael Weingärtner]

I can reopen and add you as an Assignee if you wish to work on this one.
What is your Jira ID?

On Sat, Mar 11, 2017 at 7:42 PM, Jan-Arve Nygård 
wrote:

> Hi,
>
> I can't reopen CLOUDSTACK-4603
>  but i might be
> missing rights? If it's not possible to reopen the issue, i can start a
> discussion about this feature request before i decide to recreate it or
> not.
>
> -thinktwo
>
> 2017-02-08 10:25 GMT+01:00 Daan Hoogland :
>
> > H devs,
> >
> > I am busy closing all wishes that haven’t been  discussed for more then 2
> > years. Please feel free to reopen any that you feel are still relevant.
> > Going through them one by one turned out to be to time consuming.
> >
> > Verzonden vanuit Mail
> > voor KV-guests
> >
> >
> > daan.hoogl...@shapeblue.com
> > www.shapeblue.com
> > 53 Chandos Place, Covent Garden, Utrecht Utrecht 3531 VENetherlands
> > @shapeblue
> >
> >
> >
> >
>



-- 
Rafael Weingärtner

Re: old wishes

[Jan-Arve Nygård]

Hi,

I can't reopen CLOUDSTACK-4603
 but i might be
missing rights? If it's not possible to reopen the issue, i can start a
discussion about this feature request before i decide to recreate it or not.

-thinktwo

2017-02-08 10:25 GMT+01:00 Daan Hoogland :

> H devs,
>
> I am busy closing all wishes that haven’t been  discussed for more then 2
> years. Please feel free to reopen any that you feel are still relevant.
> Going through them one by one turned out to be to time consuming.
>
> Verzonden vanuit Mail
> voor KV-guests
>
>
> daan.hoogl...@shapeblue.com
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, Utrecht Utrecht 3531 VENetherlands
> @shapeblue
>
>
>
>

[GitHub] cloudstack issue #1730: VMTemplateZone needs some love

[cloudmonger]

Github user cloudmonger commented on the issue:

https://github.com/apache/cloudstack/pull/1730
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 455
 Hypervisor xenserver
 NetworkType Advanced
 Passed=69
 Failed=19
 Skipped=7

_Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0


**Failed tests:**
* test_portable_publicip.py

 * ContextSuite context=TestPortablePublicIPAcquire>:setup Failed

* test_vm_snapshots.py

 * test_change_service_offering_for_vm_with_snapshots Failed

 * ContextSuite context=TestVmSnapshot>:setup Failing since 10 runs

* test_routers.py

 * ContextSuite context=TestRouterServices>:setup Failing since 10 runs

* test_non_contigiousvlan.py

 * test_extendPhysicalNetworkVlan Failed

* test_affinity_groups.py

 * test_DeployVmAntiAffinityGroup Failed

* test_network_acl.py

 * test_network_acl Failed

* test_pvlan.py

 * test_create_pvlan_network Failed

* test_volumes.py

 * ContextSuite context=TestCreateVolume>:setup Failing since 12 runs

 * ContextSuite context=TestVolumes>:setup Failing since 11 runs

* test_ssvm.py

 * test_05_stop_ssvm Failed

 * test_06_stop_cpvm Failed

 * test_09_destroy_ssvm Failed

 * test_10_destroy_cpvm Failed

* test_routers_network_ops.py

 * test_01_isolate_network_FW_PF_default_routes_egress_true Failed

 * test_02_isolate_network_FW_PF_default_routes_egress_false Failed

 * test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failing since 2 
runs

 * test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failing since 
2 runs

 * test_03_RVR_Network_check_router_state Failed


**Skipped tests:**
test_01_test_vm_volume_snapshot
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_11_ss_nfs_version_on_ssvm
test_nested_virtualization_vmware
test_3d_gpu_support
test_deploy_vgpu_enabled_vm

**Passed test suits:**
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_reset_vm_on_reboot.py
test_deploy_vms_with_varied_deploymentplanners.py
test_network.py
test_router_dns.py
test_login.py
test_deploy_vm_iso.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_vm_life_cycle.py
test_disk_offerings.py


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...

[wido]

Github user wido commented on the issue:

https://github.com/apache/cloudstack/pull/1955
  
@nathanejohnson: Good! I would like to see that being enums.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: Return VPC ID of VM

[Rene Moser]

Now I see my fallacy. Thanks again for clarification. Never mind.
René


On 03/11/2017 10:06 AM, Remi Bergsma wrote:
> Hi René,
> 
> From UI, you navigate to any VM, click Nics, then use “Add Network to VM” 
> button. 
> 
> You’re basically sending the addNicToVirtualMachine API call.
> 
> Regards, Remi
> 
> 
> On 11/03/2017, 09:57, "Rene Moser"  wrote:
> 
> Hi Remi
> 
> Perfect! Thanks for the clarification. I still wonder how to do it but
> that is another story :)
> 
> René
> 
> On 03/11/2017 08:40 AM, Remi Bergsma wrote:
> > Hi René,
> > 
> > I just posted some screenshots on the PR that show a VM can be part of 
> more than one VPC (mail wouldn’t allow screenshots) so please have a look at 
> Github.
> > 
> > Regards, Remi
> > 
> > On 10/03/2017, 17:52, "Rene Moser"  wrote:
> > 
> > Hi
> > 
> > I created https://github.com/apache/cloudstack/pull/1999 to return 
> the
> > vpcid the VM belongs to in listVirtualMachines-
> > 
> > I need this for the cloudstack ansible modules to fully determine 
> VMs
> > uniquely as listVirtualMachines returns all VMs (non-vpc and vpc) 
> and
> > they can be named identically.
> > 
> > (I have currently implemented a workaround to go though 
> listnetworks for
> > every VM, but this is a very expensive operation, and that is why I
> > wanted to solve it by returning the VPC id on the VM level.)
> > 
> > I had a discussion with ustcweizhou which says that a VM can be 
> part of
> > multiple VPCs and therefore it would not make sense to add vpcid to 
> the
> > response. I disagree, a VM can not be in different VPCs.
> > 
> > Can anyone join the discussion? Is it possible a VM can be in 
> different
> > VPCs?
> > 
> > Thanks for the clarification
> > 
> > René
> > 
> > 
> 
> 

[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...

[nathanejohnson]

Github user nathanejohnson commented on the issue:

https://github.com/apache/cloudstack/pull/1955
  
@wido Actually I was just about to push a commit that changed the string to 
enum, but builds are failing locally because of this test:

assertTrue(QuotaAlertManagerImpl.getDifferenceDays(now, new 
DateTime(now).plusDays(1).toDate()) == 1L);

Daylight savings time strikes again


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1955: CLOUDSTACK-8239 Add VirtIO SCSI support for KVM host...

[wido]

Github user wido commented on the issue:

https://github.com/apache/cloudstack/pull/1955
  
Do we want to pass the unmap not as a string? Shouldn't that be a enum?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

[priyankparihar]

Github user priyankparihar commented on the issue:

https://github.com/apache/cloudstack/pull/1813
  
@borisstoyanov 
> If the global settings are not available and user goes and add a custom 
value in that field, he'll get a non-meaningful message.


What about primary storage pool level setting ?


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

[priyankparihar]

Github user priyankparihar commented on the issue:

https://github.com/apache/cloudstack/pull/1813
  
@sadhugit 
>if you run the test on vmware ,you need to use iscsi  protocol.

***scsi** (modified)


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #1813: CLOUDSTACK-9604: Root disk resize support for...

[priyankparihar]

Github user priyankparihar commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/1813#discussion_r105532534
  
--- Diff: test/integration/component/test_rootvolume_resize.py ---
@@ -0,0 +1,1140 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+
+""" P1 tests for testing resize of  root volume functionality
+
+Test Plan: https://cwiki.apache.org/confluence/display/CLOUDSTACK/
+Root+Resize+Support
+
+Issue Link: https://issues.apache.org/jira/browse/CLOUDSTACK-9829
+"""
+# Import Local Modules
+from nose.plugins.attrib import attr
+from marvin.cloudstackTestCase import cloudstackTestCase, unittest
+from marvin.lib.base import (Account,
+ ServiceOffering,
+ VirtualMachine,
+ Resources,
+ Domain,
+ Volume,
+ Snapshot,
+ Template,
+ VmSnapshot,
+ Host,
+ Configurations,
+ StoragePool)
+from marvin.lib.common import (get_domain,
+   get_zone,
+   get_template,
+   matchResourceCount,
+   list_snapshots,
+   list_hosts,
+   list_configurations,
+   list_storage_pools)
+from marvin.lib.utils import (cleanup_resources,
+  validateList)
+from marvin.codes import (PASS,
+  FAIL,
+  FAILED,
+  RESOURCE_PRIMARY_STORAGE,
+  INVALID_INPUT)
+from marvin.lib.utils import checkVolumeSize
+import time
+from marvin.sshClient import SshClient
+
+
+class TestResizeVolume(cloudstackTestCase):
+@classmethod
+def setUpClass(cls):
+cls.testClient = super(TestResizeVolume, cls).getClsTestClient()
+cls.api_client = cls.testClient.getApiClient()
+cls.hypervisor = (cls.testClient.getHypervisorInfo()).lower()
+cls.storageID = None
+# Fill services from the external config file
+cls.services = cls.testClient.getParsedTestDataConfig()
+# Get Zone, Domain and templates
+cls.domain = get_domain(cls.api_client)
+cls.zone = get_zone(
+cls.api_client,
+cls.testClient.getZoneForTests())
+cls.services["mode"] = cls.zone.networktype
+cls._cleanup = []
+cls.unsupportedStorageType = False
+cls.unsupportedHypervisorType = False
+cls.updateclone = False
+if cls.hypervisor not in ['xenserver',"kvm","vmware"]:
+cls.unsupportedHypervisorType=True
+return
+cls.template = get_template(
+cls.api_client,
+cls.zone.id
+)
+cls.services["virtual_machine"]["zoneid"] = cls.zone.id
+cls.services["virtual_machine"]["template"] = cls.template.id
+cls.services["volume"]["zoneid"] = cls.zone.id
+try:
+cls.parent_domain = Domain.create(cls.api_client,
+  services=cls.services[
+  "domain"],
+  parentdomainid=cls.domain.id)
+cls.parentd_admin = Account.create(cls.api_client,
+   cls.services["account"],
+   admin=True,
+   
domainid=cls.parent_domain.id)
+cls._cleanup.append(cls.parentd_admin)
+

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

[sadhugit]

Github user sadhugit commented on the issue:

https://github.com/apache/cloudstack/pull/1813
  
Yes ,ide won't support  for vmware that is the reason test case failed with
unsupported storage error.
.
if you run the test on vmware ,you need to use iscsi  protocol.


regards
sadhu


On Fri, Mar 10, 2017 at 5:19 PM, Mowgli  wrote:

> @borisstoyanov  ,
> Please change your root disk controller to ISCSI. It could be specified in
> global configuration. For IDE it is not supported.
>
> @sadhugit  do you want to add something ?
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> ,
> or mute the thread
> 

> .
>



---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack issue #1722: CLOUDSTACK-9558 Cleanup the snapshots on the primary...

[cloudmonger]

Github user cloudmonger commented on the issue:

https://github.com/apache/cloudstack/pull/1722
  
### ACS CI BVT Run
 **Sumarry:**
 Build Number 453
 Hypervisor xenserver
 NetworkType Advanced
 Passed=103
 Failed=2
 Skipped=7

_Link to logs Folder (search by build_no):_ 
https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0


**Failed tests:**
* test_routers_network_ops.py

 * test_01_RVR_Network_FW_PF_SSH_default_routes_egress_true Failed

 * test_02_RVR_Network_FW_PF_SSH_default_routes_egress_false Failed


**Skipped tests:**
test_01_test_vm_volume_snapshot
test_vm_nic_adapter_vmxnet3
test_static_role_account_acls
test_11_ss_nfs_version_on_ssvm
test_nested_virtualization_vmware
test_3d_gpu_support
test_deploy_vgpu_enabled_vm

**Passed test suits:**
test_deploy_vm_with_userdata.py
test_affinity_groups_projects.py
test_portable_publicip.py
test_over_provisioning.py
test_global_settings.py
test_scale_vm.py
test_service_offerings.py
test_routers_iptables_default_policy.py
test_loadbalance.py
test_routers.py
test_reset_vm_on_reboot.py
test_deploy_vms_with_varied_deploymentplanners.py
test_network.py
test_router_dns.py
test_non_contigiousvlan.py
test_login.py
test_deploy_vm_iso.py
test_list_ids_parameter.py
test_public_ip_range.py
test_multipleips_per_nic.py
test_regions.py
test_affinity_groups.py
test_network_acl.py
test_pvlan.py
test_volumes.py
test_nic.py
test_deploy_vm_root_resize.py
test_resource_detail.py
test_secondary_storage.py
test_vm_life_cycle.py
test_disk_offerings.py


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: Return VPC ID of VM

[Remi Bergsma]

Hi René,

From UI, you navigate to any VM, click Nics, then use “Add Network to VM” 
button. 

You’re basically sending the addNicToVirtualMachine API call.

Regards, Remi


On 11/03/2017, 09:57, "Rene Moser"  wrote:

Hi Remi

Perfect! Thanks for the clarification. I still wonder how to do it but
that is another story :)

René

On 03/11/2017 08:40 AM, Remi Bergsma wrote:
> Hi René,
> 
> I just posted some screenshots on the PR that show a VM can be part of 
more than one VPC (mail wouldn’t allow screenshots) so please have a look at 
Github.
> 
> Regards, Remi
> 
> On 10/03/2017, 17:52, "Rene Moser"  wrote:
> 
> Hi
> 
> I created https://github.com/apache/cloudstack/pull/1999 to return the
> vpcid the VM belongs to in listVirtualMachines-
> 
> I need this for the cloudstack ansible modules to fully determine VMs
> uniquely as listVirtualMachines returns all VMs (non-vpc and vpc) and
> they can be named identically.
> 
> (I have currently implemented a workaround to go though listnetworks 
for
> every VM, but this is a very expensive operation, and that is why I
> wanted to solve it by returning the VPC id on the VM level.)
> 
> I had a discussion with ustcweizhou which says that a VM can be part 
of
> multiple VPCs and therefore it would not make sense to add vpcid to 
the
> response. I disagree, a VM can not be in different VPCs.
> 
> Can anyone join the discussion? Is it possible a VM can be in 
different
> VPCs?
> 
> Thanks for the clarification
> 
> René
> 
> 

[GitHub] cloudstack issue #1999: CLOUDSTACK-9600: listVirtualMachines: add VPC ID to ...

[resmo]

Github user resmo commented on the issue:

https://github.com/apache/cloudstack/pull/1999
  
Ok, then. 

Thanks @ustcweizhou  and @remibergsma for the clarification, changing the 
code to add vpcid on the nic's network response.

Closing this PR in favor of a new and fresh one.


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] cloudstack pull request #1999: CLOUDSTACK-9600: listVirtualMachines: add VPC...

[resmo]

Github user resmo closed the pull request at:

https://github.com/apache/cloudstack/pull/1999


---
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 is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Re: Return VPC ID of VM

[Rene Moser]

Hi Remi

Perfect! Thanks for the clarification. I still wonder how to do it but
that is another story :)

René

On 03/11/2017 08:40 AM, Remi Bergsma wrote:
> Hi René,
> 
> I just posted some screenshots on the PR that show a VM can be part of more 
> than one VPC (mail wouldn’t allow screenshots) so please have a look at 
> Github.
> 
> Regards, Remi
> 
> On 10/03/2017, 17:52, "Rene Moser"  wrote:
> 
> Hi
> 
> I created https://github.com/apache/cloudstack/pull/1999 to return the
> vpcid the VM belongs to in listVirtualMachines-
> 
> I need this for the cloudstack ansible modules to fully determine VMs
> uniquely as listVirtualMachines returns all VMs (non-vpc and vpc) and
> they can be named identically.
> 
> (I have currently implemented a workaround to go though listnetworks for
> every VM, but this is a very expensive operation, and that is why I
> wanted to solve it by returning the VPC id on the VM level.)
> 
> I had a discussion with ustcweizhou which says that a VM can be part of
> multiple VPCs and therefore it would not make sense to add vpcid to the
> response. I disagree, a VM can not be in different VPCs.
> 
> Can anyone join the discussion? Is it possible a VM can be in different
> VPCs?
> 
> Thanks for the clarification
> 
> René
> 
>