Re: Cannot migrate VMware VM with root disk to host in different cluster (CloudStack 4.10)

2017-03-22 Thread Sergey Levitskiy

Can you trace which API call being used and what parameters were specified? 
migrateVirtualMachineWithVolumeAttempts vs migrateVirtualMachine



Re: Cannot migrate VMware VM with root disk to host in different cluster (CloudStack 4.10)

2017-03-22 Thread Tutkowski, Mike
Slight correction:

That image on Imgur is of the VirtualMachineRelocateSpec that’s passed to the 
VirtualMachineMO.changeDatastore method to attempt to migrate the VM and its 
root disk.

From: "Tutkowski, Mike" 
Date: Wednesday, March 22, 2017 at 11:10 PM
To: "dev@cloudstack.apache.org" 
Subject: Cannot migrate VMware VM with root disk to host in different cluster 
(CloudStack 4.10)

Hi,

Within CloudStack 4.10, I have two VMware 5.5 clusters (both in the same VMware 
datacenter).

Each cluster has a shared iSCSI-based datastore.

I can initiate the process of migrating a VM and its root disk from a host in 
one cluster to a host in another cluster. However, it fails with the following 
error message:

Required property datastore is missing from data object of type 
VirtualMachineRelocateSpecDiskLocator

while parsing serialized DataObject of type vim.vm.RelocateSpec.DiskLocator
at line 1, column 327

while parsing property "disk" of static type 
ArrayOfVirtualMachineRelocateSpecDiskLocator

while parsing serialized DataObject of type vim.vm.RelocateSpec
at line 1, column 187

while parsing call information for method RelocateVM_Task
at line 1, column 110

while parsing SOAP body
at line 1, column 102

while parsing SOAP envelope
at line 1, column 38

while parsing HTTP request for method relocate
on object of type vim.VirtualMachine
at line 1, column 0

When I re-ran the test in the debugger and looked at the 
VirtualMachineRelocateSpecDiskLocator instance mentioned in the error message, 
I saw the following:

http://imgur.com/a/vtKcq

It looks like the code is not specifying the source datastore.

Can anyone else reproduce this?

Thanks!
Mike


Re: Cannot migrate VMware VM with root disk to host in different cluster (CloudStack 4.10)

2017-03-22 Thread Tutkowski, Mike
Also, by the way, datastore-66 (referenced in the image) is the target 
datastore.

From: "Tutkowski, Mike" 
Date: Wednesday, March 22, 2017 at 11:13 PM
To: "dev@cloudstack.apache.org" 
Subject: Re: Cannot migrate VMware VM with root disk to host in different 
cluster (CloudStack 4.10)

Slight correction:

That image on Imgur is of the VirtualMachineRelocateSpec that’s passed to the 
VirtualMachineMO.changeDatastore method to attempt to migrate the VM and its 
root disk.

From: "Tutkowski, Mike" 
Date: Wednesday, March 22, 2017 at 11:10 PM
To: "dev@cloudstack.apache.org" 
Subject: Cannot migrate VMware VM with root disk to host in different cluster 
(CloudStack 4.10)

Hi,

Within CloudStack 4.10, I have two VMware 5.5 clusters (both in the same VMware 
datacenter).

Each cluster has a shared iSCSI-based datastore.

I can initiate the process of migrating a VM and its root disk from a host in 
one cluster to a host in another cluster. However, it fails with the following 
error message:

Required property datastore is missing from data object of type 
VirtualMachineRelocateSpecDiskLocator

while parsing serialized DataObject of type vim.vm.RelocateSpec.DiskLocator
at line 1, column 327

while parsing property "disk" of static type 
ArrayOfVirtualMachineRelocateSpecDiskLocator

while parsing serialized DataObject of type vim.vm.RelocateSpec
at line 1, column 187

while parsing call information for method RelocateVM_Task
at line 1, column 110

while parsing SOAP body
at line 1, column 102

while parsing SOAP envelope
at line 1, column 38

while parsing HTTP request for method relocate
on object of type vim.VirtualMachine
at line 1, column 0

When I re-ran the test in the debugger and looked at the 
VirtualMachineRelocateSpecDiskLocator instance mentioned in the error message, 
I saw the following:

http://imgur.com/a/vtKcq

It looks like the code is not specifying the source datastore.

Can anyone else reproduce this?

Thanks!
Mike


Cannot migrate VMware VM with root disk to host in different cluster (CloudStack 4.10)

2017-03-22 Thread Tutkowski, Mike
Hi,

Within CloudStack 4.10, I have two VMware 5.5 clusters (both in the same VMware 
datacenter).

Each cluster has a shared iSCSI-based datastore.

I can initiate the process of migrating a VM and its root disk from a host in 
one cluster to a host in another cluster. However, it fails with the following 
error message:

Required property datastore is missing from data object of type 
VirtualMachineRelocateSpecDiskLocator

while parsing serialized DataObject of type vim.vm.RelocateSpec.DiskLocator
at line 1, column 327

while parsing property "disk" of static type 
ArrayOfVirtualMachineRelocateSpecDiskLocator

while parsing serialized DataObject of type vim.vm.RelocateSpec
at line 1, column 187

while parsing call information for method RelocateVM_Task
at line 1, column 110

while parsing SOAP body
at line 1, column 102

while parsing SOAP envelope
at line 1, column 38

while parsing HTTP request for method relocate
on object of type vim.VirtualMachine
at line 1, column 0

When I re-ran the test in the debugger and looked at the 
VirtualMachineRelocateSpecDiskLocator instance mentioned in the error message, 
I saw the following:

http://imgur.com/a/vtKcq

It looks like the code is not specifying the source datastore.

Can anyone else reproduce this?

Thanks!
Mike


[GitHub] cloudstack issue #1994: CLOUDSTACK-9827: Storage tags stored in multiple pla...

2017-03-22 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue:

https://github.com/apache/cloudstack/pull/1994
  
OK, this LGTM (as long as Travis shows green before we merge). Thanks!


---
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: Controlling Boot order of VMs

2017-03-22 Thread Sergey Levitskiy
If you use VMware hypervisor you can try passing boot order arguments as  VM 
advanced settings e.g.
Name:   bios.bootOrder  Value:  
ethernet1,ethernet2,hdd,cdrom,floppy
Name:   bios.hddOrder   Value:  scsi2:2,scsi0:1,ide1:0

Most of advanced settings will be delivered in a pass-through mode over to VMX 
file which in turn will activate them on the next VM start. But there is no 
guarantee since some of the option require special handling during VM 
reconfiguration in this case code change will be needed similar to PR1605/PR1310

Thanks,
Sergey


On 3/22/17, 2:59 PM, "Syed Ahmed"  wrote:

Hi Guys,

I was wondering if it is possible to set the boot order of VMs that
get created. We have a use case where we want to boot the VM from the
network. Does something like this exist? If no would people be
interested in having this functionality.

Thanks,
-Syed




Controlling Boot order of VMs

2017-03-22 Thread Syed Ahmed
Hi Guys,

I was wondering if it is possible to set the boot order of VMs that
get created. We have a use case where we want to boot the VM from the
network. Does something like this exist? If no would people be
interested in having this functionality.

Thanks,
-Syed


[GitHub] cloudstack issue #1994: CLOUDSTACK-9827: Storage tags stored in multiple pla...

2017-03-22 Thread mike-tutkowski
Github user mike-tutkowski commented on the issue:

https://github.com/apache/cloudstack/pull/1994
  
I should have a chance to look through this later tonight. In the 
meanwhile, perhaps we can re-push the code to kick off Travis again (since its 
most recent run has a failure). Thanks!


---
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 #2011: CLOUDSTACK-9811: fix duplicated nics on VR caused by...

2017-03-22 Thread ustcweizhou
Github user ustcweizhou commented on the issue:

https://github.com/apache/cloudstack/pull/2011
  
@DaanHoogland @karuturi @borisstoyanov 

Here is a short explanation why the issue happened:
Normally getBroadcastUriFromBridge method in KVM plugin will get the URI 
from a bridge name. for example, breth1-100 -> vlan://100, brem1-200 -> 
vlan://200.
However, for the bridge like brp55p1-100, it will return vlan://1 which is 
obviously wrong.

The method getBroadcastUriFromBridge  is used in 
prepareNetworkElementCommand for command IpAssocCommand  in VR. Each time when 
a new ip is associated, it will check the existing nics by getting the URIs 
from bridge name of nics. If the URIs do not contain the uri of te new ip, then 
a new nic will be plugged and new ip will be associated to the new nic.

The issue happened when cloudstack try to set more information of  eth2 
(public nic) of a VR, as for the error described in part 1 , the URI of eth2 
return wrong result (vlan://1) which is different from uri of ip (it is 
vlan://7 from StartCommand), so eth3 is added.





---
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-docs-admin pull request #42: French translation of Virtual Machin...

2017-03-22 Thread lmrv
GitHub user lmrv opened a pull request:

https://github.com/apache/cloudstack-docs-admin/pull/42

French translation of Virtual Machine and Isolation with vlan

We are now about 60% of string translated

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

$ git pull https://github.com/lmrv/cloudstack-docs-admin master

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

https://github.com/apache/cloudstack-docs-admin/pull/42.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 #42


commit 2cdc7dbbdcd63a3b4f23cc9704ede11da8254562
Author: Antoine Le Morvan 
Date:   2017-03-22T21:36:19Z

French translation of Virtual Machine and Isolation with vlan
We are now about 60% of string translated




---
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 #2011: CLOUDSTACK-9811: fix duplicated nics on VR caused by...

2017-03-22 Thread borisstoyanov
Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/2011
  
thank you guys, tests are looking good as well. 
I think we have enough LGTM to merge this now
cc: @karuturi 
tag:mergeready


---
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 #2011: CLOUDSTACK-9811: fix duplicated nics on VR caused by...

2017-03-22 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/2011
  
Trillian test result (tid-962)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 28290 seconds
Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr2011-t962-kvm-centos7.zip
Intermitten failure detected: /marvin/tests/smoke/test_privategw_acl.py
Intermitten failure detected: /marvin/tests/smoke/test_snapshots.py
Test completed. 46 look ok, 2 have error(s)


Test | Result | Time (s) | Test File
--- | --- | --- | ---
test_04_rvpc_privategw_static_routes | `Failure` | 359.28 | 
test_privategw_acl.py
test_02_list_snapshots_with_removed_data_store | `Error` | 0.03 | 
test_snapshots.py
test_01_vpc_site2site_vpn | Success | 164.78 | test_vpc_vpn.py
test_01_vpc_remote_access_vpn | Success | 66.08 | test_vpc_vpn.py
test_01_redundant_vpc_site2site_vpn | Success | 255.18 | test_vpc_vpn.py
test_02_VPC_default_routes | Success | 265.84 | test_vpc_router_nics.py
test_01_VPC_nics_after_destroy | Success | 527.08 | test_vpc_router_nics.py
test_05_rvpc_multi_tiers | Success | 504.56 | test_vpc_redundant.py
test_04_rvpc_network_garbage_collector_nics | Success | 1403.26 | 
test_vpc_redundant.py
test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
Success | 557.21 | test_vpc_redundant.py
test_02_redundant_VPC_default_routes | Success | 752.65 | 
test_vpc_redundant.py
test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | Success | 1300.22 | 
test_vpc_redundant.py
test_09_delete_detached_volume | Success | 156.34 | test_volumes.py
test_08_resize_volume | Success | 156.52 | test_volumes.py
test_07_resize_fail | Success | 161.15 | test_volumes.py
test_06_download_detached_volume | Success | 156.03 | test_volumes.py
test_05_detach_volume | Success | 155.78 | test_volumes.py
test_04_delete_attached_volume | Success | 150.94 | test_volumes.py
test_03_download_attached_volume | Success | 156.59 | test_volumes.py
test_02_attach_volume | Success | 89.67 | test_volumes.py
test_01_create_volume | Success | 711.12 | test_volumes.py
test_deploy_vm_multiple | Success | 277.07 | test_vm_life_cycle.py
test_deploy_vm | Success | 0.02 | test_vm_life_cycle.py
test_advZoneVirtualRouter | Success | 0.02 | test_vm_life_cycle.py
test_10_attachAndDetach_iso | Success | 26.52 | test_vm_life_cycle.py
test_09_expunge_vm | Success | 125.20 | test_vm_life_cycle.py
test_08_migrate_vm | Success | 40.67 | test_vm_life_cycle.py
test_07_restore_vm | Success | 0.07 | test_vm_life_cycle.py
test_06_destroy_vm | Success | 125.63 | test_vm_life_cycle.py
test_03_reboot_vm | Success | 125.67 | test_vm_life_cycle.py
test_02_start_vm | Success | 10.12 | test_vm_life_cycle.py
test_01_stop_vm | Success | 40.25 | test_vm_life_cycle.py
test_CreateTemplateWithDuplicateName | Success | 50.40 | test_templates.py
test_08_list_system_templates | Success | 0.02 | test_templates.py
test_07_list_public_templates | Success | 0.02 | test_templates.py
test_05_template_permissions | Success | 0.04 | test_templates.py
test_04_extract_template | Success | 5.12 | test_templates.py
test_03_delete_template | Success | 5.08 | test_templates.py
test_02_edit_template | Success | 90.17 | test_templates.py
test_01_create_template | Success | 50.36 | test_templates.py
test_10_destroy_cpvm | Success | 161.58 | test_ssvm.py
test_09_destroy_ssvm | Success | 163.45 | test_ssvm.py
test_08_reboot_cpvm | Success | 131.54 | test_ssvm.py
test_07_reboot_ssvm | Success | 163.49 | test_ssvm.py
test_06_stop_cpvm | Success | 131.59 | test_ssvm.py
test_05_stop_ssvm | Success | 138.58 | test_ssvm.py
test_04_cpvm_internals | Success | 1.26 | test_ssvm.py
test_03_ssvm_internals | Success | 3.83 | test_ssvm.py
test_02_list_cpvm_vm | Success | 0.09 | test_ssvm.py
test_01_list_sec_storage_vm | Success | 0.10 | test_ssvm.py
test_01_snapshot_root_disk | Success | 10.95 | test_snapshots.py
test_04_change_offering_small | Success | 239.49 | test_service_offerings.py
test_03_delete_service_offering | Success | 0.03 | test_service_offerings.py
test_02_edit_service_offering | Success | 0.04 | test_service_offerings.py
test_01_create_service_offering | Success | 0.08 | test_service_offerings.py
test_02_sys_template_ready | Success | 0.09 | test_secondary_storage.py
test_01_sys_vm_start | Success | 0.13 | test_secondary_storage.py
test_09_reboot_router | Success | 40.25 | test_routers.py
test_08_start_router | Success | 30.21 | test_routers.py
test_07_stop_router | Success | 10.12 | test_routers.py
test_06_router_advanced | Success | 0.04 | test_routers.py
test_05_router_basic | Success | 0.03 | test_routers.py

[GitHub] cloudstack issue #1582: CLOUDSTACK-9408 for the move away from download.clou...

2017-03-22 Thread DaanHoogland
Github user DaanHoogland commented on the issue:

https://github.com/apache/cloudstack/pull/1582
  
@karuturi I hade a brief peek, #1982 does not have templates for lxc or 
ovm3, so that at least is one problem.
I think it needs to be reverted and recreated, probably on top of this one. 
Or else this one would again need to be rebased.


---
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 #2011: CLOUDSTACK-9811: fix duplicated nics on VR caused by...

2017-03-22 Thread DaanHoogland
Github user DaanHoogland commented on the issue:

https://github.com/apache/cloudstack/pull/2011
  
@ustcweizhou this enables looking for a nic by a name like a1b2c3 
(\D+\d+\D*\d*\D*\d*) it will also allow all shorter as long as they start with 
 which makes sense. But to satisfy @karuturi's question in 
2003, could you *briefly* describe how this prevents extra nics from being 
created? otherwise LGTM


---
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 #2003: CLOUDSTACK-9811: fixed an issue if the dev is not in...

2017-03-22 Thread borisstoyanov
Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/2003
  
@karuturi we have 3 LGTMs, I think we're good to merge this.
tag:mergeready


---
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 #2003: CLOUDSTACK-9811: fixed an issue if the dev is not in...

2017-03-22 Thread swill
Github user swill commented on the issue:

https://github.com/apache/cloudstack/pull/2003
  
I think #2011 fixes the observed issue, but this PR improves the IP 
ordering fix originally added to the strongswan pr.  This implementation 
ensures (again) that duplicate public ips will not be saved to the data bag 
while ensuring the source nature ip is primary on its nic. 

In short, I think both PRs are worth merging. 


---
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 #1918: Management Server UI (VM statistics page) CPU Utiliz...

2017-03-22 Thread jayakarteek
Github user jayakarteek commented on the issue:

https://github.com/apache/cloudstack/pull/1918
  
@rafaelweingartner , @nvazquez 

Review the 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 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.
---


[Request for help] CCC Miami

2017-03-22 Thread Giles Sirett
Hi everybody

Will, Wido, Simon, myself and Paul have been working really hard to make sure 
that CCC Miami is a great success
However - we now need you help

http://us.cloudstackcollab.org/

We've got a great looking set of talks, many of them from people on these lists 
- so thanks for participating guys
Theres free cloudstack training for any newbies and also an all day hackathon - 
a chance to get together and work on some of the long term priorities for 
cloudstack

. We now need to drive attendance to the event and get the word outside of 
these lists


  1.  If YOU are not already registered, do so
  2.  Please tweet/post that you are going to the event
  3.  If you're speaking at the event - please tweet about it- tell the world 
about your session
  4.  Please speak to colleagues, partners & customers and encourage them to 
come along - I've even created a sample email below that you can send to people
  5.  Look through the schedule and pick a couple of talks that really interest 
you. Then tweet about those talks and mention the speaker by handle

This event has been wholly community organised. There is no marketing machine 
to kick in and help drive attendees, please can everybody here just spend a few 
minutes to help out with this


Kind regards
Giles

=sample email to send===
Dear

I are writing to let you the CloudStack Collaboration Conference 
[http://us.cloudstackcollab.org]will be held  in Miami, May 16-18, and to 
invite you to get involved.


The conference is aimed at developers, operators and users to discuss and 
evolve the Apache Cloudstack  project, its functionality and real world 
operability.

The conference has over 30 great expert talks on Cloudstack, free cloudstack 
training and a hackathon session for collaboration. It presents an opportunity 
to learn what's going on at the cutting-edge of the project and also to network 
with peers in organizations  similar to yours.

Hosted along side the ApacheCon event, participants have a wonderful 
opportunity to learn and collaborate on CloudStack, while also discovering new 
projects.

Miami is a wonderful travel destination with something for everyone. Consider 
extending your stay a day or two to enjoy the beach and sun.
*  The earlier you register, the more you save, so get your tickets early.
For registration information , see here http://us.cloudstackcollab.org/#attend 
[http://us.cloudstackcollab.org/#attend]

*
Sponsors
CloudStack Collab brings together the open source community, users and 
developers of Apache Cloudstack. Sponsoring the conference presents a great 
opportunity to
* showcase your thought leadership among a vibrant open source community
* recruit top technical talent
* create brand preference for your products, services and/or technologies
* connect with top influencers driving today's technology purchasing decisions

The Linux Foundation is managing the event logistics, so they will be handling 
the sponsorship details.

For more information see http://us.cloudstackcollab.org/#sponsors 
[http://us.cloudstackcollab.org/#sponsors]

Hope to see you all at Cloudstack Collaboration Conference Miami !




giles.sir...@shapeblue.com 
www.shapeblue.com
53 Chandos Place, Covent Garden, London  WC2N 4HSUK
@shapeblue
  
 



[GitHub] cloudstack issue #2003: CLOUDSTACK-9811: fixed an issue if the dev is not in...

2017-03-22 Thread ustcweizhou
Github user ustcweizhou commented on the issue:

https://github.com/apache/cloudstack/pull/2003
  
@karuturi to be honest, I think PR#2011 fixes the issue, not this 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 issue #2003: CLOUDSTACK-9811: fixed an issue if the dev is not in...

2017-03-22 Thread borisstoyanov
Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/2003
  
@rhtyd I think this was introduced with the StrongSwan implementation which 
was 4.10 I think. 


---
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 #1850: CLOUDSTACK-9694: Unable to limit the Public IPs in V...

2017-03-22 Thread niteshsarda
Github user niteshsarda commented on the issue:

https://github.com/apache/cloudstack/pull/1850
  
I have tested this and LGTM for test.


---
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 #2003: CLOUDSTACK-9811: fixed an issue if the dev is not in...

2017-03-22 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/2003
  
LGTM, does this affect 4.9 as well @swill ? /cc @borisstoyanov 


---
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 #2011: CLOUDSTACK-9811: fix duplicated nics on VR caused by...

2017-03-22 Thread rhtyd
Github user rhtyd commented on the issue:

https://github.com/apache/cloudstack/pull/2011
  
LGTM. Can we check travis failures?


---
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 #837: CLOUDSTACK-8855 Improve Error Message for Host...

2017-03-22 Thread bvbharatk
Github user bvbharatk commented on a diff in the pull request:

https://github.com/apache/cloudstack/pull/837#discussion_r107355223
  
--- Diff: server/src/com/cloud/alert/AlertManagerImpl.java ---
@@ -767,7 +767,9 @@ public void sendAlert(AlertType alertType, long 
dataCenterId, Long podId, Long c
 // set up a new alert
 AlertVO newAlert = new AlertVO();
 newAlert.setType(alertType.getType());
-newAlert.setSubject(subject);
+//do not have a seperate column for content.
+//appending the message to the subject for now.
+newAlert.setSubject(subject+content);
--- End diff --

@rafaelweingartner 
I agree with you but "what can be done today" depends on the individuals 
priorities and bandwidth. Just so that the required improvement dose not get 
lost in comments i have created a ticket in Jira to track this, CLOUDSTACK-9846.


---
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 #1994: CLOUDSTACK-9827: Storage tags stored in multiple pla...

2017-03-22 Thread karuturi
Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1994
  
@mike-tutkowski can you do a final review please?


---
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 #1582: CLOUDSTACK-9408 for the move away from download.clou...

2017-03-22 Thread karuturi
Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1582
  
@DaanHoogland ok. This is not mergeable yet. I will see what are the 
conflicts with #1982. It  might require new changes.
@borisstoyanov do you see the same template values as I posted. Just want 
to confirm if its my environment issue. As per the above template data, if you 
run KVM tests, it should fail as it doesnt have the system vm template url for 
kvm.


---
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 #2011: CLOUDSTACK-9811: fix duplicated nics on VR caused by...

2017-03-22 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/2011
  
@borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests


---
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 #2011: CLOUDSTACK-9811: fix duplicated nics on VR caused by...

2017-03-22 Thread borisstoyanov
Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/2011
  
Looks like tests were aborted, will restart now
@blueorangutan test


---
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 #2003: CLOUDSTACK-9811: fixed an issue if the dev is not in...

2017-03-22 Thread karuturi
Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/2003
  
@ustcweizhou can you also review this fix? Is pr #2011 still required?


---
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 #2014: WIP

2017-03-22 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/2014
  
Packaging result: ✔centos6 ✔centos7 ✔debian. JID-598


---
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 #1582: CLOUDSTACK-9408 for the move away from download.clou...

2017-03-22 Thread DaanHoogland
Github user DaanHoogland commented on the issue:

https://github.com/apache/cloudstack/pull/1582
  
@karuturi the intent of this PR was never to change any of the templates, 
only the locations. I see some changes in there that I think are the result of 
the many rebasing this PR has undergone. I think you need to check against the 
change that updated the templates. Maybe reapplying that code can work.


---
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 #1582: CLOUDSTACK-9408 for the move away from download.clou...

2017-03-22 Thread borisstoyanov
Github user borisstoyanov commented on the issue:

https://github.com/apache/cloudstack/pull/1582
  
@karuturi the results are posted already.


---
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 #2014: WIP

2017-03-22 Thread blueorangutan
Github user blueorangutan commented on the issue:

https://github.com/apache/cloudstack/pull/2014
  
@abhinandanprateek a Jenkins job has been kicked to build packages. I'll 
keep you posted as I make progress.


---
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 #2014: WIP

2017-03-22 Thread abhinandanprateek
Github user abhinandanprateek commented on the issue:

https://github.com/apache/cloudstack/pull/2014
  
@blueorangutan package


---
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 #2014: IR24:WIP

2017-03-22 Thread abhinandanprateek
GitHub user abhinandanprateek opened a pull request:

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

IR24:WIP



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

$ git pull https://github.com/shapeblue/cloudstack ir24

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

https://github.com/apache/cloudstack/pull/2014.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 #2014


commit 03c050360bf2e8f904222ade2bfbe48627ed7e54
Author: Abhinandan Prateek 
Date:   2017-03-22T06:54:19Z

IR24:WIP




---
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 #1582: CLOUDSTACK-9408 for the move away from download.clou...

2017-03-22 Thread karuturi
Github user karuturi commented on the issue:

https://github.com/apache/cloudstack/pull/1582
  
@borisstoyanov can you post the results?
@DaanHoogland After deploying from this PR, I am seeing the below values in 
vm_template table. I think LXC, KVM and Xen values below aren't right.

$ > select id,unique_name,name,type,url,updated,created from vm_template 
where type='SYSTEM';

| id | unique_name | name  | type   | url   

   | updated | created 
|-|-||-|-|
| 1 | routing-1   | SystemVM Template (XenServer) | SYSTEM | 
http://jenkins.buildacloud.org/job/build-systemvm64-master/lastSuccessfulBuild/artifact/tools/appliance/dist/systemvm64template-master-4.6.0-xen.vhd.bz2
 | NULL| 2017-03-21 07:11:27
|  3 | routing-3   | SystemVM Template (KVM)   | SYSTEM |   

   | NULL| 
2017-03-21 07:11:27 |
|  8 | routing-8   | SystemVM Template (vSphere)   | SYSTEM | 
https://download.cloudstack.org/systemvm/4.10/systemvm64template-master-4.10.0-vmware.ova
| NULL| 
2017-03-21 07:11:27 |
|  9 | routing-9   | SystemVM Template (HyperV)| SYSTEM | 
https://download.cloudstack.org/systemvm/4.10/systemvm64template-master-4.10.0-hyperv.vhd.zip
| NULL| 
2017-03-21 07:11:27 |
| 10 | routing-10  | SystemVM Template (LXC)   | SYSTEM | 
http://download.cloud.com/templates/acton/acton-systemvm-02062012.qcow2.bz2 
 | NULL 
   | 2017-03-21 07:11:48 |
| 12 | routing-12  | SystemVM Template (Ovm3)  | SYSTEM | 
http://download.cloudstack.org/systemvm/4.6/systemvm64template-4.6.0-ovm.raw.bz2
 | NULL 
   | 2017-03-21 07:12:26 |



---
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.
---