date:20200924

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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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] blueorangutan commented on pull request #3724: Storage-based Snapshots for KVM VMs

2020-09-24 Thread 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

2020-09-24 Thread 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

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

2020-09-24 Thread 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

2020-09-24 Thread 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

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

2020-09-24 Thread 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:: 
-

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

2020-09-24 Thread 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)

2020-09-24 Thread 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)

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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)

2020-09-24 Thread 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)

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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'

2020-09-24 Thread 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)

2020-09-24 Thread 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

2020-09-24 Thread 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

2020-09-24 Thread 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

[GitHub] [cloudstack] rhtyd closed issue #4170: Centos7 RPM packages are x86_64 should be noarch

2020-09-24 Thread GitBox



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


   



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 #4170: Centos7 RPM packages are x86_64 should be noarch

2020-09-24 Thread GitBox



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


   Closing this, official packages are x86_64 only



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 #4198: Kubernetes container service: coreos is EOL

2020-09-24 Thread GitBox



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


   @Pearl1594 may update her findings, we compared FlatCar vs Alpine vs Debian 
and it may make sense to reuse a Debian based template that's common for 
systemvm, and CKS host.



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 #4228: Dont add host back after agent service restart

2020-09-24 Thread GitBox



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


   @ravening @weizhouapache can you share your manual test results if any?



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-kubernetes-provider] branch master updated (b79d04b -> a58c22c)

2020-09-24 Thread 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-kubernetes-provider.git.


from b79d04b  Improved Ingress Controller example to be clearer and more 
complete (#6)
 add a58c22c  Bumped go to 1.13 and fixed go-cloudstack module version + 
k8s module checksum (#8)

No new revisions were added by this update.

Summary of changes:
 Dockerfile |   2 +-
 cloudstack.go  |   2 +-
 cloudstack_instances.go|   2 +-
 cloudstack_loadbalancer.go |   5 +-
 go.mod |   4 +-
 go.sum |  26 +-
 .../xanzy/go-cloudstack/{ => v2}/LICENSE   |   0
 .../{ => v2}/cloudstack/APIDiscoveryService.go |   0
 .../{ => v2}/cloudstack/AccountService.go  |   0
 .../{ => v2}/cloudstack/AddressService.go  |   0
 .../{ => v2}/cloudstack/AffinityGroupService.go|   0
 .../{ => v2}/cloudstack/AlertService.go|   0
 .../{ => v2}/cloudstack/AsyncjobService.go |   0
 .../{ => v2}/cloudstack/AuthenticationService.go   |   0
 .../{ => v2}/cloudstack/AutoScaleService.go|   0
 .../{ => v2}/cloudstack/BaremetalService.go|   0
 .../{ => v2}/cloudstack/BigSwitchBCFService.go |   0
 .../{ => v2}/cloudstack/BrocadeVCSService.go   |   0
 .../{ => v2}/cloudstack/CertificateService.go  |   0
 .../{ => v2}/cloudstack/CloudIdentifierService.go  |   0
 .../{ => v2}/cloudstack/ClusterService.go  |   0
 .../{ => v2}/cloudstack/ConfigurationService.go|   0
 .../{ => v2}/cloudstack/CustomService.go   |   0
 .../{ => v2}/cloudstack/DiskOfferingService.go |   0
 .../{ => v2}/cloudstack/DomainService.go   |   0
 .../{ => v2}/cloudstack/EventService.go|   0
 .../{ => v2}/cloudstack/ExtFirewallService.go  |   0
 .../{ => v2}/cloudstack/ExtLoadBalancerService.go  |   0
 .../{ => v2}/cloudstack/ExternalDeviceService.go   |   0
 .../{ => v2}/cloudstack/FirewallService.go |   0
 .../{ => v2}/cloudstack/GuestOSService.go  |   0
 .../{ => v2}/cloudstack/HostService.go |   0
 .../{ => v2}/cloudstack/HypervisorService.go   |   0
 .../{ => v2}/cloudstack/ISOService.go  |   0
 .../{ => v2}/cloudstack/ImageStoreService.go   |   0
 .../{ => v2}/cloudstack/InternalLBService.go   |   0
 .../{ => v2}/cloudstack/LDAPService.go |   0
 .../{ => v2}/cloudstack/LimitService.go|   0
 .../{ => v2}/cloudstack/LoadBalancerService.go |   0
 .../{ => v2}/cloudstack/NATService.go  |   0
 .../{ => v2}/cloudstack/NetworkACLService.go   |   0
 .../{ => v2}/cloudstack/NetworkDeviceService.go|   0
 .../{ => v2}/cloudstack/NetworkOfferingService.go  |   0
 .../{ => v2}/cloudstack/NetworkService.go  |   0
 .../{ => v2}/cloudstack/NicService.go  |   0
 .../{ => v2}/cloudstack/NiciraNVPService.go|   0
 .../{ => v2}/cloudstack/NuageVSPService.go |   0
 .../cloudstack/OutofbandManagementService.go   |   0
 .../{ => v2}/cloudstack/OvsElementService.go   |   0
 .../{ => v2}/cloudstack/PodService.go  |   0
 .../{ => v2}/cloudstack/PoolService.go |   0
 .../{ => v2}/cloudstack/PortableIPService.go   |   0
 .../{ => v2}/cloudstack/ProjectService.go  |   0
 .../{ => v2}/cloudstack/QuotaService.go|   0
 .../{ => v2}/cloudstack/RegionService.go   |   0
 .../{ => v2}/cloudstack/ResourcemetadataService.go |   0
 .../{ => v2}/cloudstack/ResourcetagsService.go |   0
 .../{ => v2}/cloudstack/RoleService.go |   0
 .../{ => v2}/cloudstack/RouterService.go   |   0
 .../{ => v2}/cloudstack/SSHService.go  |   0
 .../{ => v2}/cloudstack/SecurityGroupService.go|   0
 .../{ => v2}/cloudstack/ServiceOfferingService.go  |   0
 .../{ => v2}/cloudstack/SnapshotService.go |   0
 .../{ => v2}/cloudstack/StoragePoolService.go  |   0
 .../{ => v2}/cloudstack/StratosphereSSPService.go  |   0
 .../{ => v2}/cloudstack/SwiftService.go|   0
 .../{ => v2}/cloudstack/SystemCapacityService.go   |   0
 .../{ => v2}/cloudstack/SystemVMService.go |   0
 .../{ => v2}/cloudstack/TemplateService.go |   0
 .../{ => v2}/cloudstack/UCSService.go  |   0
 .../{ => v2}/cloudstack/UsageService.go|   0
 .../{ => v2}/cloudstack/UserService.go |   4 +
 .../{ => v2}/cloudstack/VLANService.go |   0
 .../{ => v2}/cloudstack/VMGroupService.go  |   0
 .../{ => v2}/cloudstack/VPCService.go  |   0
 .../{ => v2}/cloudstack/VPNService.go  |   0
 .../{ => v2}/cloudstack/VirtualMachineService.go   |   0
 .../{ => v2}/cloudstack/VolumeService.go   |

[cloudstack-kubernetes-provider] branch master updated: Support loadBalancerSourceRanges (#9)

2020-09-24 Thread 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-kubernetes-provider.git


The following commit(s) were added to refs/heads/master by this push:
 new 2f17d26  Support loadBalancerSourceRanges (#9)
2f17d26 is described below

commit 2f17d26e671f0b598dff5c71f1b1c9fbd27dbe76
Author: Gregor Riepl 
AuthorDate: Thu Sep 24 08:52:52 2020 +0200

Support loadBalancerSourceRanges (#9)

* Add support for loadBalancerSourceRanges

* Added basic existing firewall rule comparison and creation boilerplate

* Implemented LB firewall rule creation+update and deletion

* WIP proper rule apply/cleanup

* Split protocol type into enum and fix variable names

* Split CIDR list

* Fixed a minor error handling logic error

* Modified update logic to always refresh firewall rules

* Make log less verbose

* Better error logging for firewall rules

* Fixed firewall rule protocol names

* Log firewall rule deletion steps

* Need listall=true for list commands...

* Log public IP ID

* Send project with list firewall rules call

* Drop debug logging corrections, they're unrelated

* Improve error handling on firewall rule creation

* Fixed string format error

* Better debugging for rules

* Delete matching fw rules first to prevent conflict
---
 cloudstack_loadbalancer.go | 342 +++--
 protocol.go| 105 ++
 2 files changed, 377 insertions(+), 70 deletions(-)

diff --git a/cloudstack_loadbalancer.go b/cloudstack_loadbalancer.go
index 91d4eff..2b30483 100644
--- a/cloudstack_loadbalancer.go
+++ b/cloudstack_loadbalancer.go
@@ -23,6 +23,7 @@ import (
"context"
"fmt"
"strconv"
+   "strings"
 
"github.com/xanzy/go-cloudstack/v2/cloudstack"
"k8s.io/klog"
@@ -31,13 +32,9 @@ import (
cloudprovider "k8s.io/cloud-provider"
 )
 
-// ServiceAnnotationLoadBalancerProxyProtocol is the annotation used on the
-// service to enable the proxy protocol on a CloudStack load balancer.
-// The value of this annotation is ignored, even if it is seemingly boolean.
-// Simple presence of the annotation will enable it.
-// Note that this protocol only applies to TCP service ports and
-// CloudStack 4.6 is required for it to work.
-const ServiceAnnotationLoadBalancerProxyProtocol = 
"service.beta.kubernetes.io/cloudstack-load-balancer-proxy-protocol"
+// defaultAllowedCIDR is the network range that is allowed on the firewall
+// by default when no explicit CIDR list is given on a LoadBalancer.
+const defaultAllowedCIDR = "0.0.0.0/0"
 
 type loadBalancer struct {
*cloudstack.CloudStackClient
@@ -129,51 +126,70 @@ func (cs *CSCloud) EnsureLoadBalancer(ctx 
context.Context, clusterName string, s
 
for _, port := range service.Spec.Ports {
// Construct the protocol name first, we need it a few times
-   protocol, err := constructProtocolName(port, 
service.Annotations)
-   if err != nil {
-   return nil, err
+   protocol := ProtocolFromServicePort(port, service.Annotations)
+   if protocol == LoadBalancerProtocolInvalid {
+   return nil, fmt.Errorf("unsupported load balancer 
protocol: %v", port.Protocol)
}
 
// All ports have their own load balancer rule, so add the port 
to lbName to keep the names unique.
lbRuleName := fmt.Sprintf("%s-%s-%d", lb.name, protocol, 
port.Port)
 
// If the load balancer rule exists and is up-to-date, we move 
on to the next rule.
-   exists, needsUpdate, err := 
lb.checkLoadBalancerRule(lbRuleName, port, protocol)
+   lbRule, needsUpdate, err := 
lb.checkLoadBalancerRule(lbRuleName, port, protocol)
if err != nil {
return nil, err
}
-   if exists && !needsUpdate {
-   klog.V(4).Infof("Load balancer rule %v is up-to-date", 
lbRuleName)
-   // Delete the rule from the map, to prevent it being 
deleted.
-   delete(lb.rules, lbRuleName)
-   continue
+
+   if lbRule != nil {
+   if needsUpdate {
+   klog.V(4).Infof("Updating load balancer rule: 
%v", lbRuleName)
+   if err := lb.updateLoadBalancerRule(lbRuleName, 
protocol); err != nil {
+   return nil, err
+   }
+   // Delete the rule from the map, to prevent it 
being deleted.
+   delete(lb.rules, lbRuleName)
+

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

2020-09-24 Thread GitBox



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


   @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 #4150: [HEALTH][4.14] Health Check Run

2020-09-24 Thread GitBox



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


   @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] rhtyd commented on issue #4299: Can't unmanage VMware instance

2020-09-24 Thread GitBox



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


   @harikrishna-patnala @nvazquez @shwstppr ping - update?



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 #4310: SystemVMs Publics IPs offered to accounts despite global settings

2020-09-24 Thread GitBox



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







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] branch 4.14 updated (ca1e02f -> 1efe6e2)

2020-09-24 Thread rohit

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

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


from ca1e02f  vmware: search unmanaged instances using hypervisor name 
(#4328)
 add d6152b3  server: Broadcast URI not set to vxlan, but vlan (Fix #3040) 
(#4190)
 new 1efe6e2  Merge remote-tracking branch 'origin/4.13' into 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 ++
 .../engine/orchestration/NetworkOrchestrator.java  | 23 ++-
 .../orchestration/NetworkOrchestratorTest.java | 74 ++
 3 files changed, 131 insertions(+), 15 deletions(-)

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

2020-09-24 Thread rohit

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

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

commit 1efe6e2df06a9fb6ff12b86b966724e6f224ed45
Merge: ca1e02f d6152b3
Author: Rohit Yadav 
AuthorDate: Thu Sep 24 12:13:23 2020 +0530

Merge remote-tracking branch 'origin/4.13' into 4.14

Signed-off-by: Rohit Yadav 

 api/src/main/java/com/cloud/network/Networks.java  | 49 ++
 .../engine/orchestration/NetworkOrchestrator.java  | 23 ++-
 .../orchestration/NetworkOrchestratorTest.java | 74 ++
 3 files changed, 131 insertions(+), 15 deletions(-)

diff --cc 
engine/orchestration/src/main/java/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
index ab2d8f3,c49da14..804775a
--- 
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
@@@ -2311,9 -2279,7 +2311,9 @@@ public class NetworkOrchestrator extend
  }
  
  if (vlanSpecified) {
- URI uri = BroadcastDomainType.fromString(vlanId);
+ URI uri = encodeVlanIdIntoBroadcastUri(vlanId, pNtwk);
 +// Aux: generate secondary URI for secondary VLAN ID (if 
provided) for performing checks
 +URI secondaryUri = isNotBlank(isolatedPvlan) ? 
BroadcastDomainType.fromString(isolatedPvlan) : null;
  //don't allow to specify vlan tag used by physical network for 
dynamic vlan allocation
  if (!(bypassVlanOverlapCheck && ntwkOff.getGuestType() == 
GuestType.Shared) && _dcDao.findVnet(zoneId, pNtwk.getId(), 
BroadcastDomainType.getValue(uri)).size() > 0) {
  throw new InvalidParameterValueException("The VLAN tag " + 
vlanId + " is already being used for dynamic vlan allocation for the guest 
network in zone "

[GitHub] [cloudstack] rhtyd edited a comment on issue #4067: Usage Server running issue

2020-09-24 Thread GitBox



rhtyd edited a comment on issue #4067:
URL: https://github.com/apache/cloudstack/issues/4067#issuecomment-698146935


   Similar issue of cloudstack-usage not working on raspberrypi4 
https://github.com/apache/cloudstack/issues/4342 was reported by @dony71



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 #4067: Usage Server running issue

2020-09-24 Thread GitBox



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


   Same issue https://github.com/apache/cloudstack/issues/4342 was reported by 
@dony71



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 #4342: cloudstack-usage kept restarting

2020-09-24 Thread GitBox



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


   



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 #4342: cloudstack-usage kept restarting

2020-09-24 Thread GitBox



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


   @dony71 this is known issue, cloudstack-usage does not work on raspberypi4.
   Closing this as this dupes https://github.com/apache/cloudstack/issues/4067



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 #30

2020-09-24 Thread Apache Jenkins Server

See 


Changes:

[github] vmware: while plugging in nics get existing sorted nic devices (#4336)


--
[...truncated 22.36 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.343 s]
[INFO] Apache CloudStack .. SUCCESS [  3.637 s]
[INFO] Apache CloudStack Framework - Managed Context .. FAILURE [  3.777 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 Component SKIPPED
[INFO] Apache CloudStack Server ... SKIPPED
[INFO] Apache CloudStack Framework - Quota  SKIPPED
[INFO]

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

2020-09-24 Thread GitBox



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


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



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

2020-09-24 Thread GitBox



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


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



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

88 matches

Mail list logo