[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support

[GitBox]

blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698753680


   @nvazquez a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support

[GitBox]

nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698753498


   @blueorangutan package



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support

[GitBox]

blueorangutan removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698752221


   @nvazquez a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] nvazquez removed a comment on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support

[GitBox]

nvazquez removed a comment on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698751791


   @blueorangutan package



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support

[GitBox]

blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698752221


   @nvazquez a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support

[GitBox]

nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698751791


   @blueorangutan package



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] resmo edited a comment on issue #3934: Zones are returned in a random order

[GitBox]

resmo edited a comment on issue #3934:
URL: https://github.com/apache/cloudstack/issues/3934#issuecomment-698743047


   I see this with in 4.13.1 
   
   # vmsnapshot test revert snapshot failed
   
   note the zone that changes from basic to advance from first test to second 
test.
   
   
https://github.com/ngine-io/ansible-collection-cloudstack/pull/40/checks?check_run_id=1162936780#step:6:3610
   
   ```
   TASK [cs_vmsnapshot : test revert snapshot] 

   changed: [testhost] => {"account": "admin", "api_http_method": "get", 
"api_key": 
"DL3Be8XtxsrKJOMRIIPR_2LUdknE5n8V33u41NWaetVcnKAQTeQhkDDVKAJCIyHV1I4fY3Phw8B49BpSnfLKtg",
 "api_region": "cloudstack", "api_timeout": 60, "api_url": 
"http://cloudstack-sim:/client/api";, "api_verify_ssl_cert": null, 
"changed": true, "created": "2020-09-24T22:12:33+", "current": true, 
"display_name": "ansible-test-fv-az176-89231118_snapshot", "domain": "ROOT", 
"id": "230bab25-dcd6-4641-97c5-79396730eef6", "name": 
"i-2-12-QA_VS_20200924221233", "state": "Ready", "tags": [], "type": 
"DiskAndMemory", "zone": "Sandbox-simulator-basic"}
   
   TASK [cs_vmsnapshot : verify test revert snapshot] 
*
   ok: [testhost] => {
   "changed": false,
   "msg": "All assertions passed"
   }
   
   TASK [cs_vmsnapshot : test remove snapshot in check mode] 
**
   fatal: [testhost]: FAILED! => {"api_http_method": "get", "api_key": 
"DL3Be8XtxsrKJOMRIIPR_2LUdknE5n8V33u41NWaetVcnKAQTeQhkDDVKAJCIyHV1I4fY3Phw8B49BpSnfLKtg",
 "api_region": "cloudstack", "api_timeout": 60, "api_url": 
"http://cloudstack-sim:/client/api";, "api_verify_ssl_cert": null, 
"changed": false, "msg": "Virtual machine 
'ansible-test-fv-az176-89231118-vm-snapshot' not found", "zone": 
"Sandbox-simulator-advanced"}
   ```
   
   but seems not to happen always, another test run passes.
   ```
TASK [cs_vmsnapshot : test remove snapshot in check mode] 
**
   changed: [testhost] => {"account": "admin", "api_http_method": "get", 
"api_key": 
"DL3Be8XtxsrKJOMRIIPR_2LUdknE5n8V33u41NWaetVcnKAQTeQhkDDVKAJCIyHV1I4fY3Phw8B49BpSnfLKtg",
 "api_region": "cloudstack", "api_timeout": 60, "api_url": 
"http://cloudstack-sim:/client/api";, "api_verify_ssl_cert": null, 
"changed": true, "created": "2020-09-24T22:10:49+", "current": true, 
"display_name": "ansible-test-fv-az76-30345916_snapshot", "domain": "ROOT", 
"id": "4013a799-6d7c-442c-8906-1a6162e3086e", "name": 
"i-2-14-QA_VS_20200924221049", "state": "Ready", "tags": [], "type": 
"DiskAndMemory", "zone": "Sandbox-simulator-basic"}
   ```
   
   this is the code for the zone selection: 
https://github.com/ngine-io/ansible-collection-cloudstack/blob/master/plugins/module_utils/cloudstack.py#L467
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] resmo edited a comment on issue #3934: Zones are returned in a random order

[GitBox]

resmo edited a comment on issue #3934:
URL: https://github.com/apache/cloudstack/issues/3934#issuecomment-698743047


   I see this with in 4.13.1 
   
   # vmsnapshot test revert snapshot failed
   
   note the zone that changes from basic to advance from first test to second 
test.
   
   
https://github.com/ngine-io/ansible-collection-cloudstack/pull/40/checks?check_run_id=1162936780#step:6:3610
   
   ```
   TASK [cs_vmsnapshot : test revert snapshot] 

   changed: [testhost] => {"account": "admin", "api_http_method": "get", 
"api_key": 
"DL3Be8XtxsrKJOMRIIPR_2LUdknE5n8V33u41NWaetVcnKAQTeQhkDDVKAJCIyHV1I4fY3Phw8B49BpSnfLKtg",
 "api_region": "cloudstack", "api_timeout": 60, "api_url": 
"http://cloudstack-sim:/client/api";, "api_verify_ssl_cert": null, 
"changed": true, "created": "2020-09-24T22:12:33+", "current": true, 
"display_name": "ansible-test-fv-az176-89231118_snapshot", "domain": "ROOT", 
"id": "230bab25-dcd6-4641-97c5-79396730eef6", "name": 
"i-2-12-QA_VS_20200924221233", "state": "Ready", "tags": [], "type": 
"DiskAndMemory", "zone": "Sandbox-simulator-basic"}
   
   TASK [cs_vmsnapshot : verify test revert snapshot] 
*
   ok: [testhost] => {
   "changed": false,
   "msg": "All assertions passed"
   }
   
   TASK [cs_vmsnapshot : test remove snapshot in check mode] 
**
   fatal: [testhost]: FAILED! => {"api_http_method": "get", "api_key": 
"DL3Be8XtxsrKJOMRIIPR_2LUdknE5n8V33u41NWaetVcnKAQTeQhkDDVKAJCIyHV1I4fY3Phw8B49BpSnfLKtg",
 "api_region": "cloudstack", "api_timeout": 60, "api_url": 
"http://cloudstack-sim:/client/api";, "api_verify_ssl_cert": null, 
"changed": false, "msg": "Virtual machine 
'ansible-test-fv-az176-89231118-vm-snapshot' not found", "zone": 
"Sandbox-simulator-advanced"}
   ```
   
   but seems not to happen always, another test run passes.
   ```
TASK [cs_vmsnapshot : test remove snapshot in check mode] 
**
   changed: [testhost] => {"account": "admin", "api_http_method": "get", 
"api_key": 
"DL3Be8XtxsrKJOMRIIPR_2LUdknE5n8V33u41NWaetVcnKAQTeQhkDDVKAJCIyHV1I4fY3Phw8B49BpSnfLKtg",
 "api_region": "cloudstack", "api_timeout": 60, "api_url": 
"http://cloudstack-sim:/client/api";, "api_verify_ssl_cert": null, 
"changed": true, "created": "2020-09-24T22:10:49+", "current": true, 
"display_name": "ansible-test-fv-az76-30345916_snapshot", "domain": "ROOT", 
"id": "4013a799-6d7c-442c-8906-1a6162e3086e", "name": 
"i-2-14-QA_VS_20200924221049", "state": "Ready", "tags": [], "type": 
"DiskAndMemory", "zone": "Sandbox-simulator-basic"}
   ```
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] dpassante opened a new issue #3934: Zones are returned in a random order

[GitBox]

dpassante opened a new issue #3934:
URL: https://github.com/apache/cloudstack/issues/3934


   
   
   # ISSUE TYPE
   
* Bug Report
   
   # COMPONENT NAME
   
   ~~~
   API
   ~~~
   
   # CLOUDSTACK VERSION
   
   
   ~~~
   4.13.0.0
   ~~~
   
   # CONFIGURATION
   
   N/A
   
   # OS / ENVIRONMENT
   
   N/A
   
   # SUMMARY
   
   `listZones` returns the zones in a "random" order when it has not been 
explicitly defined with the new sort key feature.
   
   This impacts the correct functioning of the Ansible Cloudstack modules which 
use the first zone if no zone parameter is given.
   
   # STEPS TO REPRODUCE
   
   
   
   Keep zones unsorted:
   ~~~
   mysql> select id, name, sort_key from data_center;
   +++--+
   | id | name   | sort_key |
   +++--+
   |  1 | Sandbox-simulator-basic|0 |
   |  2 | Sandbox-simulator-advanced |0 |
   +++--+
   2 rows in set (0.00 sec)
   ~~~
   
   And use a playbook in which the `zone` param is not given.
   Several calls to `listZones` also do the trick.
   
   
   
   # EXPECTED RESULTS
   
   A list of zones sorted by SQL ID when no sort key are configured to avoid 
breaking some existing Ansible deployments.
   
   Ensure that a sort key is automatically incremented for new zones and 
existing zones in the case of a Cloudstack upgrade could also be a nice feature.
   
   # ACTUAL RESULTS
   
   
   
   ~~~
   $ for in in `seq 1 1 10`; do cs listZones |grep -m1 name; done
 "name": "Sandbox-simulator-basic",
 "name": "Sandbox-simulator-basic",
 "name": "Sandbox-simulator-basic",
 "name": "Sandbox-simulator-basic",
 "name": "Sandbox-simulator-basic",
 "name": "Sandbox-simulator-basic",
 "name": "Sandbox-simulator-advanced",
 "name": "Sandbox-simulator-advanced",
 "name": "Sandbox-simulator-advanced",
 "name": "Sandbox-simulator-advanced",
   ~~~
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] resmo commented on issue #3934: Zones are returned in a random order

[GitBox]

resmo commented on issue #3934:
URL: https://github.com/apache/cloudstack/issues/3934#issuecomment-698743047


   I see this with in 4.13.1 
   
   # vmsnapshot test revert snapshot failed
   
   note the zone that chates from basic to advance from first test to second 
test.
   ```
   TASK [cs_vmsnapshot : test revert snapshot] 

   changed: [testhost] => {"account": "admin", "api_http_method": "get", 
"api_key": 
"DL3Be8XtxsrKJOMRIIPR_2LUdknE5n8V33u41NWaetVcnKAQTeQhkDDVKAJCIyHV1I4fY3Phw8B49BpSnfLKtg",
 "api_region": "cloudstack", "api_timeout": 60, "api_url": 
"http://cloudstack-sim:/client/api";, "api_verify_ssl_cert": null, 
"changed": true, "created": "2020-09-24T22:12:33+", "current": true, 
"display_name": "ansible-test-fv-az176-89231118_snapshot", "domain": "ROOT", 
"id": "230bab25-dcd6-4641-97c5-79396730eef6", "name": 
"i-2-12-QA_VS_20200924221233", "state": "Ready", "tags": [], "type": 
"DiskAndMemory", "zone": "Sandbox-simulator-basic"}
   
   TASK [cs_vmsnapshot : verify test revert snapshot] 
*
   ok: [testhost] => {
   "changed": false,
   "msg": "All assertions passed"
   }
   
   TASK [cs_vmsnapshot : test remove snapshot in check mode] 
**
   fatal: [testhost]: FAILED! => {"api_http_method": "get", "api_key": 
"DL3Be8XtxsrKJOMRIIPR_2LUdknE5n8V33u41NWaetVcnKAQTeQhkDDVKAJCIyHV1I4fY3Phw8B49BpSnfLKtg",
 "api_region": "cloudstack", "api_timeout": 60, "api_url": 
"http://cloudstack-sim:/client/api";, "api_verify_ssl_cert": null, 
"changed": false, "msg": "Virtual machine 
'ansible-test-fv-az176-89231118-vm-snapshot' not found", "zone": 
"Sandbox-simulator-advanced"}
   ```
   
   but seems not to happen always, another test run passes.
   ```
TASK [cs_vmsnapshot : test remove snapshot in check mode] 
**
   changed: [testhost] => {"account": "admin", "api_http_method": "get", 
"api_key": 
"DL3Be8XtxsrKJOMRIIPR_2LUdknE5n8V33u41NWaetVcnKAQTeQhkDDVKAJCIyHV1I4fY3Phw8B49BpSnfLKtg",
 "api_region": "cloudstack", "api_timeout": 60, "api_url": 
"http://cloudstack-sim:/client/api";, "api_verify_ssl_cert": null, 
"changed": true, "created": "2020-09-24T22:10:49+", "current": true, 
"display_name": "ansible-test-fv-az76-30345916_snapshot", "domain": "ROOT", 
"id": "4013a799-6d7c-442c-8906-1a6162e3086e", "name": 
"i-2-14-QA_VS_20200924221049", "state": "Ready", "tags": [], "type": 
"DiskAndMemory", "zone": "Sandbox-simulator-basic"}
   ```
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support

[GitBox]

blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698725666


   Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2084



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support

[GitBox]

blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698717222


   @nvazquez a Jenkins job has been kicked to build packages. I'll keep you 
posted as I make progress.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] nvazquez commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support

[GitBox]

nvazquez commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698716996


   @blueorangutan package



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] tungntvn23 commented on issue #4314: Failed to attach binaries ISO for VM when create kubernetes cluster

[GitBox]

tungntvn23 commented on issue #4314:
URL: https://github.com/apache/cloudstack/issues/4314#issuecomment-698710171


   @shwstppr thank you, but issue happen randomly. Sometime it's OK but 
sometime not OK.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support

[GitBox]

blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698700643


   @rhtyd a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7 mgmt + 
vmware67, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] rhtyd commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support

[GitBox]

rhtyd commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698700405


   @blueorangutan test matrix 



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4194: enable update tags on disk offerings

[GitBox]

blueorangutan commented on pull request #4194:
URL: https://github.com/apache/cloudstack/pull/4194#issuecomment-698675719


   Trillian test result (tid-2845)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 64649 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4194-t2845-kvm-centos7.zip
   Intermittent failure detected: 
/marvin/tests/smoke/test_kubernetes_clusters.py
   Intermittent failure detected: 
/marvin/tests/smoke/test_routers_network_ops.py
   Intermittent failure detected: /marvin/tests/smoke/test_volumes.py
   Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
   Intermittent failure detected: /marvin/tests/smoke/test_vpc_vpn.py
   Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
   Smoke tests completed. 81 look OK, 4 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_02_deploy_kubernetes_ha_cluster | `Error` | 3626.05 | 
test_kubernetes_clusters.py
   test_04_deploy_and_upgrade_kubernetes_cluster | `Error` | 0.05 | 
test_kubernetes_clusters.py
   test_05_deploy_and_upgrade_kubernetes_ha_cluster | `Error` | 0.05 | 
test_kubernetes_clusters.py
   test_06_deploy_and_invalid_upgrade_kubernetes_cluster | `Error` | 0.04 | 
test_kubernetes_clusters.py
   test_07_deploy_and_scale_kubernetes_cluster | `Error` | 0.05 | 
test_kubernetes_clusters.py
   ContextSuite context=TestKubernetesCluster>:teardown | `Error` | 37.76 | 
test_kubernetes_clusters.py
   ContextSuite context=TestCreateVolume>:setup | `Error` | 0.00 | 
test_volumes.py
   ContextSuite context=TestVolumes>:setup | `Error` | 0.00 | test_volumes.py
   test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Error` | 938.19 | 
test_vpc_redundant.py
   test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
`Error` | 348.27 | test_vpc_redundant.py
   test_04_rvpc_network_garbage_collector_nics | `Error` | 3912.53 | 
test_vpc_redundant.py
   test_hostha_kvm_host_fencing | `Error` | 101.18 | test_hostha_kvm.py
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4340: Changing test_pvlan vlan id to prevent conflict with smoke tests env config

[GitBox]

blueorangutan commented on pull request #4340:
URL: https://github.com/apache/cloudstack/pull/4340#issuecomment-698651477


   Trillian test result (tid-2850)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 56124 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4340-t2850-kvm-centos7.zip
   Intermittent failure detected: 
/marvin/tests/smoke/test_kubernetes_clusters.py
   Intermittent failure detected: 
/marvin/tests/smoke/test_outofbandmanagement.py
   Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
   Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
   Smoke tests completed. 83 look OK, 2 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 270.60 | 
test_vpc_redundant.py
   test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | 
`Failure` | 337.41 | test_vpc_redundant.py
   test_hostha_kvm_host_fencing | `Error` | 170.49 | test_hostha_kvm.py
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4338: server: check guest os preference of last host when start a vm

[GitBox]

blueorangutan commented on pull request #4338:
URL: https://github.com/apache/cloudstack/pull/4338#issuecomment-698583879


   Trillian test result (tid-2846)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 46460 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4338-t2846-kvm-centos7.zip
   Intermittent failure detected: 
/marvin/tests/smoke/test_kubernetes_supported_versions.py
   Intermittent failure detected: /marvin/tests/smoke/test_privategw_acl.py
   Smoke tests completed. 82 look OK, 1 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_02_vpc_privategw_static_routes | `Failure` | 257.66 | 
test_privategw_acl.py
   test_03_vpc_privategw_restart_vpc_cleanup | `Failure` | 341.11 | 
test_privategw_acl.py
   test_04_rvpc_privategw_static_routes | `Failure` | 482.25 | 
test_privategw_acl.py
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4269: cks: assorted fixes, test refactoring

[GitBox]

blueorangutan commented on pull request #4269:
URL: https://github.com/apache/cloudstack/pull/4269#issuecomment-698566087


   Trillian test result (tid-2848)
   Environment: vmware-67u3 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 43270 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4269-t2848-vmware-67u3.zip
   Intermittent failure detected: /marvin/tests/smoke/test_vpc_router_nics.py
   Smoke tests completed. 79 look OK, 0 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4150: [HEALTH][4.14] Health Check Run

[GitBox]

blueorangutan commented on pull request #4150:
URL: https://github.com/apache/cloudstack/pull/4150#issuecomment-698541178


   Trillian test result (tid-2849)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 38486 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4150-t2849-kvm-centos7.zip
   Intermittent failure detected: 
/marvin/tests/smoke/test_affinity_groups_projects.py
   Intermittent failure detected: /marvin/tests/smoke/test_affinity_groups.py
   Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
   Smoke tests completed. 80 look OK, 3 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_DeployVmAntiAffinityGroup_in_project | `Error` | 67.62 | 
test_affinity_groups_projects.py
   test_DeployVmAntiAffinityGroup | `Error` | 38.07 | test_affinity_groups.py
   test_hostha_kvm_host_degraded | `Failure` | 670.81 | test_hostha_kvm.py
   test_hostha_kvm_host_fencing | `Failure` | 646.48 | test_hostha_kvm.py
   test_hostha_kvm_host_recovering | `Failure` | 645.41 | test_hostha_kvm.py
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] olivierlemasle commented on pull request #4327: Re-enable IP address usage hiding

[GitBox]

olivierlemasle commented on pull request #4327:
URL: https://github.com/apache/cloudstack/pull/4327#issuecomment-698437757


   Kindly ping @rhtyd 



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] slavkap opened a new pull request #4345: Binding listening socket to all address for remote debug

[GitBox]

slavkap opened a new pull request #4345:
URL: https://github.com/apache/cloudstack/pull/4345


   ## Description
   
   Since Java 9 the notation 'address=port' only applies to localhost.
   For remote debug you have to explicitly specify that you want to listen
   to all IP addresses (e.g. address=*8000)
   
   ## Types of changes
   
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   - [ ] New feature (non-breaking change which adds functionality)
   - [ ] Bug fix (non-breaking change which fixes an issue)
   - [ x] Enhancement (improves an existing feature and functionality)
   - [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
   
   ## How Has This Been Tested?
   tested on master and 4.14
   
   
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] DaanHoogland commented on pull request #4144: Fix Usage failed to get pid

[GitBox]

DaanHoogland commented on pull request #4144:
URL: https://github.com/apache/cloudstack/pull/4144#issuecomment-698341080


   though the ha error is a bit offsetting I see not how any of these errors 
can be related, @rhtyd agree?



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] andrijapanicsb commented on issue #4299: Can't unmanage VMware instance

[GitBox]

andrijapanicsb commented on issue #4299:
URL: https://github.com/apache/cloudstack/issues/4299#issuecomment-698331570


   I don't understand how I am supposed to know that those are PR specific, 
considering that the failure has zero things to do with the PR I was testing...



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4144: Fix Usage failed to get pid

[GitBox]

blueorangutan commented on pull request #4144:
URL: https://github.com/apache/cloudstack/pull/4144#issuecomment-698317533


   Trillian test result (tid-2844)
   Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
   Total time taken: 57660 seconds
   Marvin logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4144-t2844-kvm-centos7.zip
   Intermittent failure detected: 
/marvin/tests/smoke/test_kubernetes_clusters.py
   Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
   Smoke tests completed. 83 look OK, 2 have error(s)
   Only failed tests results shown below:
   
   
   Test | Result | Time (s) | Test File
   --- | --- | --- | ---
   test_01_deploy_kubernetes_cluster | `Error` | 3618.75 | 
test_kubernetes_clusters.py
   test_02_deploy_kubernetes_ha_cluster | `Error` | 0.04 | 
test_kubernetes_clusters.py
   test_04_deploy_and_upgrade_kubernetes_cluster | `Failure` | 1984.43 | 
test_kubernetes_clusters.py
   test_05_deploy_and_upgrade_kubernetes_ha_cluster | `Error` | 0.04 | 
test_kubernetes_clusters.py
   test_07_deploy_and_scale_kubernetes_cluster | `Failure` | 906.89 | 
test_kubernetes_clusters.py
   ContextSuite context=TestKubernetesCluster>:teardown | `Error` | 941.71 | 
test_kubernetes_clusters.py
   test_hostha_kvm_host_fencing | `Error` | 175.18 | test_hostha_kvm.py
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] rhtyd commented on issue #4299: Can't unmanage VMware instance

[GitBox]

rhtyd commented on issue #4299:
URL: https://github.com/apache/cloudstack/issues/4299#issuecomment-698294224


   Sure thanks @nvazquez; @andrijapanicsb pl avoid raising bugs that may be PR 
specific as issues on master



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] rhtyd commented on issue #4343: java.lang.ArithmeticException: / by zero - when scaling VR

[GitBox]

rhtyd commented on issue #4343:
URL: https://github.com/apache/cloudstack/issues/4343#issuecomment-698293852


   ping @shwstppr - do you remember if that's fixed?



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] harikrishna-patnala opened a new issue #4344: Changing disk controller type is not working for root disk or data disk

[GitBox]

harikrishna-patnala opened a new issue #4344:
URL: https://github.com/apache/cloudstack/issues/4344


   
   
   # ISSUE TYPE
   
* Bug Report
   
   # COMPONENT NAME
   
   ~~~
   VMware
   ~~~
   
   # CLOUDSTACK VERSION
   
   
   ~~~
   Cloudstack 4.14.0.0, master 
   ~~~
   
   # CONFIGURATION
   
   
   
   # OS / ENVIRONMENT
   
   
   
   # SUMMARY
   
   Changed the controller from IDE to SCSI from VM settings tab on stopped VM 
and then started the VM. 
   Now cloudstack shows scsi but vmware shows IDE.
   CloudStack deliberately fetching the controller details from already 
existing disks and sets the same irrespective of the VM settings or template 
settings.
   
   
![image](https://user-images.githubusercontent.com/3348673/94136813-197cc680-fe83-11ea-9ced-09c9f47682cf.png)
   
![image](https://user-images.githubusercontent.com/3348673/94136833-239ec500-fe83-11ea-8179-3e984f8b2f4e.png)
   
   
   # STEPS TO REPRODUCE
   
   1. Create VM (on VMware)
   2. Stop VM
   3. Go to VM settings
   4. Change root.disk.controller from IDE to SCSI
   5. Start the VM
   
   ~~~
   
   ~~~
   
   
   
   # EXPECTED RESULTS
   
   
   ~~~
   Controller at vCenter should change to SCSI on the corresponding hard disk 
of VM
   ~~~
   
   # ACTUAL RESULTS
   
   
   
   ~~~
   Controller at vCenter still shows IDE on the corresponding hard disk of VM
   ~~~
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] borisstoyanov opened a new issue #4343: java.lang.ArithmeticException: / by zero - when scaling VR

[GitBox]

borisstoyanov opened a new issue #4343:
URL: https://github.com/apache/cloudstack/issues/4343


   
   
   # ISSUE TYPE
   
* Bug Report
   
   
   # COMPONENT NAME
   
   ~~~
   VR Scaling
   ~~~
   
   # CLOUDSTACK VERSION
   
   
   ~~~
   4.11.3
   ~~~
   
   # CONFIGURATION
   
   with VMWARE 6.5
   
   # OS / ENVIRONMENT
   
   
   
   # SUMMARY
   
   
   
   # STEPS TO REPRODUCE
   
   
   
   ~~~
   Version used: 4.11.3
   
   Steps: 
   1. Set the new system offering with 2 cores instead of 1 and fill in the 
same details as the default one. 
   2. Scale the VR to the new Offering
   
   2020-09-23 13:26:51,689 ERROR [c.c.h.v.r.VmwareResource] 
(DirectAgent-132:ctx-85f479ac 10.10.2.140, job-306/job-310, cmd: 
ScaleVmCommand) (logid:3407676c) Unexpected exception:
   java.lang.ArithmeticException: / by zero
   ~~~
   
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support

[GitBox]

blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698278040


   Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2083



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] nvazquez commented on issue #4299: Can't unmanage VMware instance

[GitBox]

nvazquez commented on issue #4299:
URL: https://github.com/apache/cloudstack/issues/4299#issuecomment-698275548


   @rhtyd we've moved to PR #4307, I'll check on this PR



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support

[GitBox]

blueorangutan commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698263651


   @harikrishna-patnala a Jenkins job has been kicked to build packages. I'll 
keep you posted as I make progress.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] harikrishna-patnala commented on pull request #4307: [VMware] vSphere advanced capabilities and Full OVF properties support

[GitBox]

harikrishna-patnala commented on pull request #4307:
URL: https://github.com/apache/cloudstack/pull/4307#issuecomment-698263502


   @blueorangutan package



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] ravening commented on pull request #4338: server: check guest os preference of last host when start a vm

[GitBox]

ravening commented on pull request #4338:
URL: https://github.com/apache/cloudstack/pull/4338#issuecomment-698247764


   LGTM



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #3730: Update SystemVM scripts from python2 to python3

[GitBox]

blueorangutan commented on pull request #3730:
URL: https://github.com/apache/cloudstack/pull/3730#issuecomment-698243694


   Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2082



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #3730: Update SystemVM scripts from python2 to python3

[GitBox]

blueorangutan commented on pull request #3730:
URL: https://github.com/apache/cloudstack/pull/3730#issuecomment-698227196


   @GabrielBrascher a Jenkins job has been kicked to build packages. I'll keep 
you posted as I make progress.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] GabrielBrascher commented on pull request #3730: Update SystemVM scripts from python2 to python3

[GitBox]

GabrielBrascher commented on pull request #3730:
URL: https://github.com/apache/cloudstack/pull/3730#issuecomment-698226935







This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4177: Prevent deploying IPv6 network if Zone has no IPv6 DNS configured

[GitBox]

blueorangutan commented on pull request #4177:
URL: https://github.com/apache/cloudstack/pull/4177#issuecomment-698225527


   Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2081



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4177: Prevent deploying IPv6 network if Zone has no IPv6 DNS configured

[GitBox]

blueorangutan commented on pull request #4177:
URL: https://github.com/apache/cloudstack/pull/4177#issuecomment-698208235


   @GabrielBrascher a Jenkins job has been kicked to build packages. I'll keep 
you posted as I make progress.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] GabrielBrascher commented on pull request #4177: Prevent deploying IPv6 network if Zone has no IPv6 DNS configured

[GitBox]

GabrielBrascher commented on pull request #4177:
URL: https://github.com/apache/cloudstack/pull/4177#issuecomment-698207801


   @blueorangutan package



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] GabrielBrascher commented on pull request #4177: Prevent deploying IPv6 network if Zone has no IPv6 DNS configured

[GitBox]

GabrielBrascher commented on pull request #4177:
URL: https://github.com/apache/cloudstack/pull/4177#issuecomment-698207517


   Fixed conflict with the master.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #3724: Storage-based Snapshots for KVM VMs

[GitBox]

blueorangutan commented on pull request #3724:
URL: https://github.com/apache/cloudstack/pull/3724#issuecomment-698205448


   Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2080



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] rhtyd commented on pull request #4131: [HEALTH] Master/4.15 Health Check please don't merge this

[GitBox]

rhtyd commented on pull request #4131:
URL: https://github.com/apache/cloudstack/pull/4131#issuecomment-698190713


   @blueorangutan test matrix



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4131: [HEALTH] Master/4.15 Health Check please don't merge this

[GitBox]

blueorangutan commented on pull request #4131:
URL: https://github.com/apache/cloudstack/pull/4131#issuecomment-698190884


   @rhtyd a Trillian-Jenkins matrix job (centos7 mgmt + xs71, centos7 mgmt + 
vmware67, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] rhtyd closed pull request #3724: Storage-based Snapshots for KVM VMs

[GitBox]

rhtyd closed pull request #3724:
URL: https://github.com/apache/cloudstack/pull/3724


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] slavkap opened a new pull request #3724: Storage-based Snapshots for KVM VMs

[GitBox]

slavkap opened a new pull request #3724:
URL: https://github.com/apache/cloudstack/pull/3724


   ## Description
   
   Cloudstack with KVM as a hypervisor provides live VM snapshots only with 
volumes which image format is QCOW. 
   This is a limitation for storage providers with disks in RAW format.
   
   link to ML: 
http://mail-archives.apache.org/mod_mbox/cloudstack-dev/201911.mbox/%3ccaa6fghf7eay-a3xgn5zswkvvp7zrcunzv5nfaqkaf+et3zh...@mail.gmail.com%3e#archives
   
   
   With this feature
   - CloudStack users will be able to do a live consistent snapshots of all 
disks attached to the VM 
   - Is using respective storage provider plugin implementation to take the 
snapshot
   - For NFS/Local storages we are using qemu command - drive-backup. We 
recommend to use
this functionality only if you have VMs with mixed volumes (in RAW and 
QCOW format). 
   - You can create VM snapshot without it's memory
   - Creates in the UI and DB VM snapshot and volume's snapshot related to the 
VM snapshot 
   
   ## Types of changes
   
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   - [X] New feature (non-breaking change which adds functionality)
   - [ ] Bug fix (non-breaking change which fixes an issue)
   - [ ] Enhancement (improves an existing feature and functionality)
   - [ ] Cleanup (Code refactoring and cleanup, that may add test cases)
   
   ## Documentation
   
   https://github.com/apache/cloudstack-documentation/pull/153
   
   ## Prerequisites
   
   - Install qemu-guest-agent on guest machine (used to freeze/thaw the virtual 
machine)
   - In global settings "kvm.vmstoragesnapshot.enabled" has to be set to true
   
   - If volumes' image format is QCOW you need - qemu version 1.6+ for 
drive-backup command. If you’re with the latest stable Ubuntu it will be 
included, but for CentOS7 you need to install qemu-kvm-ev.
   - When using drive-backup command the option - 
"snapshot.backup.to.secondary" should be enabled.
   
   ## How Has This Been Tested?
   
   Environment:
   
   - CloudStack - management and agent version - 4.14.0.0-SNAPSHOT
   - OS for management and hypervisor - CentOS7
   - Hypervisor - KVM
   - Primary storages - NFS, StorPool, Local storage and Ceph. 
   
   Take snapshot:
   - Take VM snapshot without memory on running virtual machine
   - We have tested this feature with these storage providers - NFS, StorPool, 
Local storage and Ceph. 
   - There is one smoke test, which creates VM, installs qemu-guest-agent, 
takes, reverts and deletes VM snapshot. 
   
   Revert snapshot
   - Stop the virtual machine and revert the VM snapshot
   
   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #3724: Storage-based Snapshots for KVM VMs

[GitBox]

blueorangutan commented on pull request #3724:
URL: https://github.com/apache/cloudstack/pull/3724#issuecomment-69818


   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] rhtyd commented on pull request #3724: Storage-based Snapshots for KVM VMs

[GitBox]

rhtyd commented on pull request #3724:
URL: https://github.com/apache/cloudstack/pull/3724#issuecomment-698187570


   @blueorangutan package



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[cloudstack-documentation] branch master updated: secondary storage management

[rohit]

This is an automated email from the ASF dual-hosted git repository.

rohit pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git


The following commit(s) were added to refs/heads/master by this push:
 new b72ba71  secondary storage management
 new 33e1122  Merge pull request #147 from shapeblue/sec_stg_mgmt
b72ba71 is described below

commit b72ba712777a6394faec72d8490edcf8d1287e2e
Author: Pearl Dsilva 
AuthorDate: Wed Aug 12 17:39:17 2020 +0530

secondary storage management
---
 source/adminguide/storage.rst | 68 ---
 1 file changed, 45 insertions(+), 23 deletions(-)

diff --git a/source/adminguide/storage.rst b/source/adminguide/storage.rst
index 987dd5f..9757d45 100644
--- a/source/adminguide/storage.rst
+++ b/source/adminguide/storage.rst
@@ -29,7 +29,7 @@ Primary Storage
 ---
 
 This section gives technical details about CloudStack
-primary storage. For more information about the concepts behind primary 
storage 
+primary storage. For more information about the concepts behind primary storage
 see :ref:`about-primary-storage` . For information about how to install and 
configure
 primary storage through the CloudStack UI, see the in the Installation Guide.
 
@@ -183,6 +183,28 @@ secondary storage. For information about how to install 
and configure
 secondary storage through the CloudStack UI, see the Advanced
 Installation Guide. about-secondary-storage>`_
 
+Migration of data between secondary storages is now supported. One may choose
+to completely migrate the data or migrate data such that the stores
+are balanced by choosing the appropriate Migration Policy. In order to 
facilitate
+distributing the migration load, SSVMs are spawned up if a file transfer takes
+more than a defined threshold. Following are the Global setting values to one 
may
+want to look at before proceeding with the migration task:
+
+
+   
+--++
+   | Configuration Parameters | Description

|
+   
+==++
+   | image.store.imbalance.threshold  | The storage imbalance threshold that 
is compared with the standard deviation percentage for a storage utilization 
metric. The value is a percentage in decimal format. |
+   
+--++
+   | secstorage.max.migrate.sessions  | The max number of concurrent copy 
command execution sessions that an SSVM can handle  
 |
+   
+--++
+   | max.ssvm.count   | Number of additional SSVMs to handle 
migration of data objects concurrently  
  |
+   
+--++
+   | max.data.migration.wait.time | Maximum wait time for a data migration 
task before spawning a new SSVM 
|
+   
+--++
+
+Secondary storages can also be set to read-only in order to cordon it off
+from being used for storing any further templates, volumes and snapshots.
 
 Working With Volumes
 
@@ -208,7 +230,7 @@ from a volume as well; this is the standard procedure for 
private
 template creation. Volumes are hypervisor-specific: a volume from one
 hypervisor type may not be used on a guest of another hypervisor type.
 
-.. note:: 
+.. note::
CloudStack supports attaching up to
 
- 13 data disks on XenServer hypervisor versions 6.0 and above,
@@ -386,9 +408,9 @@ volume from one storage pool to another.
 Detaching and Moving Volumes
 
 
-.. note:: 
-   T

[cloudstack-documentation] branch master updated (316861b -> 135edab)

[rohit]

This is an automated email from the ASF dual-hosted git repository.

rohit pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git.


from 316861b  Merge pull request #145 from shapeblue/proj-roleBased-users
 new 17e219c  Edit OVS Plugin
 new ab40420  Update source/plugins/ovs-plugin.rst
 new 135edab  Merge pull request #133 from shapeblue/edit-ovs-plugin

The 225 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 source/plugins/ovs-plugin.rst | 6 ++
 1 file changed, 6 insertions(+)

[cloudstack-documentation] branch master updated (caf374e -> 316861b)

[rohit]

This is an automated email from the ASF dual-hosted git repository.

rohit pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git.


from caf374e  Merge pull request #149 from soreana/master
 new 948217f  Modified project details - role based users in projects
 new b2a6e5e  Review comments addressed
 new c20b522  minor detail added regarding roles
 new 21a0334  comments addressed
 new 316861b  Merge pull request #145 from shapeblue/proj-roleBased-users

The 222 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 source/adminguide/projects.rst | 96 ++
 1 file changed, 50 insertions(+), 46 deletions(-)

[cloudstack-documentation] branch master updated (846c1c1 -> caf374e)

[rohit]

This is an automated email from the ASF dual-hosted git repository.

rohit pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git.


from 846c1c1  Merge pull request #151 from shapeblue/fix-cks-vmw-controllers
 new dc6cb43  Update accounts.rst
 new 10cee4f  Update wording which was wrong
 new bed1155  Fixed dhclient.eth0.leases path.
 new 63981e2  Update accounts.rst
 new 04699fc  Update accounts.rst
 new eac4948  Update user-data.rst
 new caf374e  Merge pull request #149 from soreana/master

The 217 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 source/adminguide/virtual_machines/user-data.rst | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

[cloudstack-documentation] branch master updated: cks: root disk controller and nic adapter for CoreOS VMware

[rohit]

This is an automated email from the ASF dual-hosted git repository.

rohit pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack-documentation.git


The following commit(s) were added to refs/heads/master by this push:
 new a0a1d31  cks: root disk controller and nic adapter for CoreOS VMware
 new 846c1c1  Merge pull request #151 from shapeblue/fix-cks-vmw-controllers
a0a1d31 is described below

commit a0a1d317779560033134d1d3fab08a57c9cfd6f3
Author: Abhishek Kumar 
AuthorDate: Thu Sep 10 01:24:22 2020 +0530

cks: root disk controller and nic adapter for CoreOS VMware

Signed-off-by: Abhishek Kumar 
---
 source/plugins/cloudstack-kubernetes-service.rst | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/source/plugins/cloudstack-kubernetes-service.rst 
b/source/plugins/cloudstack-kubernetes-service.rst
index ade34ac..a601329 100644
--- a/source/plugins/cloudstack-kubernetes-service.rst
+++ b/source/plugins/cloudstack-kubernetes-service.rst
@@ -146,6 +146,9 @@ The following Global Settings value must be set to the name 
of Template to be us
 
 Using a CoreOS template is required - you can find CoreOS templates for 
CloudStack here, http://dl.openvm.eu/cloudstack/coreos/x86_64/
 
+.. note::
+   For VMware, CoreOS template must be registered with root disk controller as 
**pvscsi** and NIC adapter type as **Vmxnet3**.
+
 The following Global Setting value must be set to the name of Network Offering 
to be used for creating a new network when no network has been selected while 
creating a Kubernetes cluster:
 
 .. parsed-literal::

[GitHub] [cloudstack] blueorangutan commented on pull request #4340: Changing test_pvlan vlan id to prevent conflict with smoke tests env config

[GitBox]

blueorangutan commented on pull request #4340:
URL: https://github.com/apache/cloudstack/pull/4340#issuecomment-698184750


   @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been 
kicked to run smoke tests



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] rhtyd commented on pull request #4340: Changing test_pvlan vlan id to prevent conflict with smoke tests env config

[GitBox]

rhtyd commented on pull request #4340:
URL: https://github.com/apache/cloudstack/pull/4340#issuecomment-698184293


   @blueorangutan test



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4150: [HEALTH][4.14] Health Check Run

[GitBox]

blueorangutan commented on pull request #4150:
URL: https://github.com/apache/cloudstack/pull/4150#issuecomment-698184266


   @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been 
kicked to run smoke tests



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Build failed in Jenkins: Cloudstack » cloudstack-marvin #31

[Apache Jenkins Server]

See 


Changes:

[github] agent: Compare indirect agent lb algorithm when cloudstack agent 
connects (#4335)

[github] vmware: search unmanaged instances using hypervisor name (#4328)

[github] server: Broadcast URI not set to vxlan, but vlan (Fix #3040) (#4190)


--
[...truncated 22.33 KB...]
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[JENKINS] Archiving disabled
[INFO] 
[INFO] Reactor Summary for Apache CloudStack 4.15.0.0-SNAPSHOT:
[INFO] 
[INFO] Apache CloudStack Developer Tools - Checkstyle Configuration SUCCESS [  
9.851 s]
[INFO] Apache CloudStack .. SUCCESS [  3.779 s]
[INFO] Apache CloudStack Framework - Managed Context .. FAILURE [  3.931 s]
[INFO] Apache CloudStack Framework  SKIPPED
[INFO] Apache CloudStack Framework - Certificate Authority  SKIPPED
[INFO] Apache CloudStack Utils  SKIPPED
[INFO] Apache CloudStack Framework - Database . SKIPPED
[INFO] Apache CloudStack Framework - Configuration  SKIPPED
[INFO] Apache CloudStack Framework - Direct Download to Primary Storage SKIPPED
[INFO] Apache CloudStack API .. SKIPPED
[INFO] Apache CloudStack Framework - REST . SKIPPED
[INFO] Apache CloudStack Framework - IPC .. SKIPPED
[INFO] Apache CloudStack Cloud Engine . SKIPPED
[INFO] Apache CloudStack Cloud Engine API . SKIPPED
[INFO] Apache CloudStack Cloud Engine Schema Component  SKIPPED
[INFO] Apache CloudStack Framework - Security . SKIPPED
[INFO] Apache CloudStack Core . SKIPPED
[INFO] Apache CloudStack Agents ... SKIPPED
[INFO] Apache CloudStack Framework - Clustering ... SKIPPED
[INFO] Apache CloudStack Framework - Event Notification ... SKIPPED
[INFO] Apache CloudStack Framework - Jobs . SKIPPED
[INFO] Apache CloudStack Cloud Engine Internal Components API SKIPPED
[INFO] Apache CloudStack Agent Management Servers Load Balancer SKIPPED
[INFO] Apache CloudStack Framework - Storage Config Drive C

[GitHub] [cloudstack] rhtyd commented on pull request #4150: [HEALTH][4.14] Health Check Run

[GitBox]

rhtyd commented on pull request #4150:
URL: https://github.com/apache/cloudstack/pull/4150#issuecomment-698183718


   @blueorangutan test



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] rvalle commented on issue #4310: SystemVMs Publics IPs offered to accounts despite global settings

[GitBox]

rvalle commented on issue #4310:
URL: https://github.com/apache/cloudstack/issues/4310#issuecomment-698182286


   @rhtyd the Public IPs marked as for System VMs are offered for user 
consumption (web UI) from the beginning.
   
   I mean, after a fresh installation when non of the public IPs have been 
consumed. 
   
   I am assuming that it has been implemented that System VMs will use 
(optionally strictly)  vlan_ip_ranges flagged for SystemVMs, but it has not 
been implemented that those IPs are not to be used other accounts. 



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4131: [HEALTH] Master/4.15 Health Check please don't merge this

[GitBox]

blueorangutan commented on pull request #4131:
URL: https://github.com/apache/cloudstack/pull/4131#issuecomment-698181884


   Packaging result: ✔centos7 ✔centos8 ✔debian. JID-2079



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] rvalle commented on issue #4295: Api Authentication Error

[GitBox]

rvalle commented on issue #4295:
URL: https://github.com/apache/cloudstack/issues/4295#issuecomment-698180058


   @DaanHoogland I will rebuild my environment again and retest. 
   
   Big mystery to me. 
   
   I can reproduce this bug each time. I am on Ubuntu 18.04, perhaps the issue 
is related to JVM patch level? 
   
   I will investigate further and see what I can find out.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4269: cks: assorted fixes, test refactoring

[GitBox]

blueorangutan commented on pull request #4269:
URL: https://github.com/apache/cloudstack/pull/4269#issuecomment-698174600


   @rhtyd a Trillian-Jenkins test job (centos7 mgmt + vmware-67u3) has been 
kicked to run smoke tests



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] rhtyd commented on pull request #4269: cks: assorted fixes, test refactoring

[GitBox]

rhtyd commented on pull request #4269:
URL: https://github.com/apache/cloudstack/pull/4269#issuecomment-698174255


   @blueorangutan test centos7 vmware-67u3



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] rhtyd closed issue #4011: Allow .img extension for qcow2 type of template

[GitBox]

rhtyd closed issue #4011:
URL: https://github.com/apache/cloudstack/issues/4011


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] rhtyd commented on issue #4011: Allow .img extension for qcow2 type of template

[GitBox]

rhtyd commented on issue #4011:
URL: https://github.com/apache/cloudstack/issues/4011#issuecomment-698169120


   Checked UriUtils.java, img is supported for qcow2 format



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4131: [HEALTH] Master/4.15 Health Check please don't merge this

[GitBox]

blueorangutan commented on pull request #4131:
URL: https://github.com/apache/cloudstack/pull/4131#issuecomment-698165398


   @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted 
as I make progress.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] rhtyd commented on pull request #4131: [HEALTH] Master/4.15 Health Check please don't merge this

[GitBox]

rhtyd commented on pull request #4131:
URL: https://github.com/apache/cloudstack/pull/4131#issuecomment-698165178


   @blueorangutan package



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4141: [New feature] Load balancer customization (haproxy-based)

[GitBox]

blueorangutan commented on pull request #4141:
URL: https://github.com/apache/cloudstack/pull/4141#issuecomment-698165421


   @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been 
kicked to run smoke tests



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] rhtyd commented on pull request #4141: [New feature] Load balancer customization (haproxy-based)

[GitBox]

rhtyd commented on pull request #4141:
URL: https://github.com/apache/cloudstack/pull/4141#issuecomment-698165034


   @blueorangutan test



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] DaanHoogland commented on pull request #4338: server: check guest os preference of last host when start a vm

[GitBox]

DaanHoogland commented on pull request #4338:
URL: https://github.com/apache/cloudstack/pull/4338#issuecomment-698165087


   @blueorangutan test



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4338: server: check guest os preference of last host when start a vm

[GitBox]

blueorangutan commented on pull request #4338:
URL: https://github.com/apache/cloudstack/pull/4338#issuecomment-698165411


   @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4150: [HEALTH][4.14] Health Check Run

[GitBox]

blueorangutan commented on pull request #4150:
URL: https://github.com/apache/cloudstack/pull/4150#issuecomment-698164929


   Packaging result: ✔centos7 ✖centos8 ✔debian. JID-2078



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] blueorangutan commented on pull request #4194: enable update tags on disk offerings

[GitBox]

blueorangutan commented on pull request #4194:
URL: https://github.com/apache/cloudstack/pull/4194#issuecomment-698161662


   @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has 
been kicked to run smoke tests



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] DaanHoogland commented on pull request #4194: enable update tags on disk offerings

[GitBox]

DaanHoogland commented on pull request #4194:
URL: https://github.com/apache/cloudstack/pull/4194#issuecomment-698161345


   @blueorangutan test



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[cloudstack] 01/01: Merge remote-tracking branch 'origin/4.14'

[rohit]

This is an automated email from the ASF dual-hosted git repository.

rohit pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack.git

commit 86fcb142383c4bb5742225eb8352a41734fe787f
Merge: 4f8b88b 1efe6e2
Author: Rohit Yadav 
AuthorDate: Thu Sep 24 12:20:29 2020 +0530

Merge remote-tracking branch 'origin/4.14'

Signed-off-by: Rohit Yadav 

 api/src/main/java/com/cloud/network/Networks.java  | 49 +++
 .../com/cloud/agent/manager/AgentManagerImpl.java  |  9 ++-
 .../engine/orchestration/NetworkOrchestrator.java  | 23 ++-
 .../orchestration/NetworkOrchestratorTest.java | 73 ++
 .../cloudstack/agent/lb/IndirectAgentLB.java   |  2 +-
 .../hypervisor/vmware/resource/VmwareResource.java |  4 +-
 .../agent/lb/IndirectAgentLBServiceImpl.java   |  5 +-
 .../com/cloud/hypervisor/vmware/mo/ClusterMO.java  |  7 ++-
 .../com/cloud/hypervisor/vmware/mo/HostMO.java | 41 +++-
 .../hypervisor/vmware/mo/VmwareHypervisorHost.java |  2 +-
 10 files changed, 187 insertions(+), 28 deletions(-)

diff --cc 
engine/orchestration/src/main/java/com/cloud/agent/manager/AgentManagerImpl.java
index 3f1b506,6bfc3de..f69f54c
--- 
a/engine/orchestration/src/main/java/com/cloud/agent/manager/AgentManagerImpl.java
+++ 
b/engine/orchestration/src/main/java/com/cloud/agent/manager/AgentManagerImpl.java
@@@ -1091,9 -1094,9 +1096,9 @@@ public class AgentManagerImpl extends M
  
  final HostVO host = 
_resourceMgr.createHostVOForConnectedAgent(startup);
  if (host != null) {
 -ready = new ReadyCommand(host.getDataCenterId(), 
host.getId());
 +ready = new ReadyCommand(host.getDataCenterId(), 
host.getId(), NumbersUtil.enableHumanReadableSizes);
  
- if 
(!indirectAgentLB.compareManagementServerList(host.getId(), 
host.getDataCenterId(), agentMSHostList)) {
+ if 
(!indirectAgentLB.compareManagementServerList(host.getId(), 
host.getDataCenterId(), agentMSHostList, lbAlgorithm)) {
  final List newMSList = 
indirectAgentLB.getManagementServerList(host.getId(), host.getDataCenterId(), 
null);
  ready.setMsHostList(newMSList);
  
ready.setLbAlgorithm(indirectAgentLB.getLBAlgorithmName());
diff --cc 
engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
index 892f771,804775a..e12dca0
--- 
a/engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
+++ 
b/engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
@@@ -2513,14 -2465,8 +2513,14 @@@ public class NetworkOrchestrator extend
  //Logical router's UUID provided as VLAN_ID
  userNetwork.setVlanIdAsUUID(vlanIdFinal); //Set 
transient field
  } else {
- uri = BroadcastDomainType.fromString(vlanIdFinal);
+ uri = encodeVlanIdIntoBroadcastUri(vlanIdFinal, 
pNtwk);
  }
 +
 +if 
(_networksDao.listByPhysicalNetworkPvlan(physicalNetworkId, 
uri.toString()).size() > 0) {
 +throw new InvalidParameterValueException("Network 
with vlan " + vlanIdFinal +
 +" already exists or overlaps with other 
network pvlans in zone " + zoneId);
 +}
 +
  userNetwork.setBroadcastUri(uri);
  if (!vlanIdFinal.equalsIgnoreCase(Vlan.UNTAGGED)) {
  
userNetwork.setBroadcastDomainType(BroadcastDomainType.Vlan);
diff --cc 
engine/orchestration/src/test/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestratorTest.java
index e68ac5c,ed5265c..ff0a6fa
--- 
a/engine/orchestration/src/test/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestratorTest.java
+++ 
b/engine/orchestration/src/test/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestratorTest.java
@@@ -464,23 -467,74 +467,93 @@@ public class NetworkOrchestratorTest ex
  }
  
  @Test
 +public void testDontReleaseNicWhenPreserveNicsSettingEnabled() {
 +VirtualMachineProfile vm = mock(VirtualMachineProfile.class);
 +NicVO nic = mock(NicVO.class);
 +NetworkVO network = mock(NetworkVO.class);
 +
 +when(vm.getType()).thenReturn(Type.User);
 +when(network.getGuruName()).thenReturn(guruName);
 +
when(testOrchastrator._networksDao.findById(nic.getNetworkId())).thenReturn(network);
 +
 +Long nicId = 1L;
 +when(nic.getId()).thenReturn(nicId);
 +
when(vm.getParameter(VirtualMachineProfile.Param.PreserveNics)).thenReturn(true);
 +
 +testOrchastrator.removeNic(vm, nic);
 +
 +verify(nic, never()).setState(Nic.State.Deallocating);
 +

[cloudstack] branch master updated (4f8b88b -> 86fcb14)

[rohit]

This is an automated email from the ASF dual-hosted git repository.

rohit pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/cloudstack.git.


from 4f8b88b  vmware: while plugging in nics get existing sorted nic 
devices (#4336)
 add 534dd47  agent: Compare indirect agent lb algorithm when cloudstack 
agent connects (#4335)
 add ca1e02f  vmware: search unmanaged instances using hypervisor name 
(#4328)
 add d6152b3  server: Broadcast URI not set to vxlan, but vlan (Fix #3040) 
(#4190)
 add 1efe6e2  Merge remote-tracking branch 'origin/4.13' into 4.14
 new 86fcb14  Merge remote-tracking branch 'origin/4.14'

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 api/src/main/java/com/cloud/network/Networks.java  | 49 +++
 .../com/cloud/agent/manager/AgentManagerImpl.java  |  9 ++-
 .../engine/orchestration/NetworkOrchestrator.java  | 23 ++-
 .../orchestration/NetworkOrchestratorTest.java | 73 ++
 .../cloudstack/agent/lb/IndirectAgentLB.java   |  2 +-
 .../hypervisor/vmware/resource/VmwareResource.java |  4 +-
 .../agent/lb/IndirectAgentLBServiceImpl.java   |  5 +-
 .../com/cloud/hypervisor/vmware/mo/ClusterMO.java  |  7 ++-
 .../com/cloud/hypervisor/vmware/mo/HostMO.java | 41 +++-
 .../hypervisor/vmware/mo/VmwareHypervisorHost.java |  2 +-
 10 files changed, 187 insertions(+), 28 deletions(-)

[GitHub] [cloudstack] rhtyd closed issue #3040: Broadcast URI not set to vxlan, but vlan

[GitBox]

rhtyd closed issue #3040:
URL: https://github.com/apache/cloudstack/issues/3040


   



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [cloudstack] ravening commented on pull request #4228: Dont add host back after agent service restart

[GitBox]

ravening commented on pull request #4228:
URL: https://github.com/apache/cloudstack/pull/4228#issuecomment-698157726


   @rhtyd Sure... I will share the results by tomorrow



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org