[cloudstack] 01/01: [Merge 4.11] CLOUDSTACK-10341: Reduce systemvmtemplate size (#2506)

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 ba61b512b2b1006d7164e039a3cb57db9ba64fab Merge: 95d10c2 986ecfa Author: Rohit Yadav

[cloudstack] branch master updated (95d10c2 -> ba61b51)

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 95d10c2 [Merge 4.11] CLOUDSTACK-10227: Fix mkdir path for centos rpms (#2513) add 986ecfa

[GitHub] rhtyd commented on issue #2514: Problem with NAT configuration and VMs not accessing each other via public IPs

rhtyd commented on issue #2514: Problem with NAT configuration and VMs not accessing each other via public IPs URL: https://github.com/apache/cloudstack/pull/2514#issuecomment-376795363 @rafaelweingartner since it's a useful bugfix, can you change the base branch and rebase the PR against

[GitHub] rhtyd commented on issue #2506: CLOUDSTACK-10341: Reduce systemvmtemplate size, install nftables

rhtyd commented on issue #2506: CLOUDSTACK-10341: Reduce systemvmtemplate size, install nftables URL: https://github.com/apache/cloudstack/pull/2506#issuecomment-376792059 Test LGTM, failures are due to environment issues or not related to this PR. Merging this based on results and two

[cloudstack] branch 4.11 updated: CLOUDSTACK-10341: Reduce systemvmtemplate size, install nftables (#2506)

This is an automated email from the ASF dual-hosted git repository. rohit pushed a commit to branch 4.11 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.11 by this push: new 986ecfa CLOUDSTACK-10341: Reduce

[GitHub] rhtyd closed pull request #2506: CLOUDSTACK-10341: Reduce systemvmtemplate size, install nftables

rhtyd closed pull request #2506: CLOUDSTACK-10341: Reduce systemvmtemplate size, install nftables URL: https://github.com/apache/cloudstack/pull/2506 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] blueorangutan commented on issue #2510: CLOUDSTACK-10334: Fix inadequate information for handling catch clauses

blueorangutan commented on issue #2510: CLOUDSTACK-10334: Fix inadequate information for handling catch clauses URL: https://github.com/apache/cloudstack/pull/2510#issuecomment-376804849 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make

[cloudstack] branch master updated: CLOUDSTACK-10334: Fix inadequate information for handling catch clauses (#2510)

This is an automated email from the ASF dual-hosted git repository. rafael pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/master by this push: new c1c587f CLOUDSTACK-10334: Fix inadequate

[GitHub] rafaelweingartner closed pull request #2510: CLOUDSTACK-10334: Fix inadequate information for handling catch clauses

rafaelweingartner closed pull request #2510: CLOUDSTACK-10334: Fix inadequate information for handling catch clauses URL: https://github.com/apache/cloudstack/pull/2510 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] rafaelweingartner commented on issue #2510: CLOUDSTACK-10334: Fix inadequate information for handling catch clauses

rafaelweingartner commented on issue #2510: CLOUDSTACK-10334: Fix inadequate information for handling catch clauses URL: https://github.com/apache/cloudstack/pull/2510#issuecomment-376831626 @DaanHoogland I am ok too. I will merge then 

[GitHub] rafaelweingartner commented on issue #2510: CLOUDSTACK-10334: Fix inadequate information for handling catch clauses

rafaelweingartner commented on issue #2510: CLOUDSTACK-10334: Fix inadequate information for handling catch clauses URL: https://github.com/apache/cloudstack/pull/2510#issuecomment-376831626 @rhtyd I am ok too. I will merge then 

[GitHub] rafaelweingartner commented on issue #2514: Problem with NAT configuration and VMs not accessing each other via public IPs

rafaelweingartner commented on issue #2514: Problem with NAT configuration and VMs not accessing each other via public IPs URL: https://github.com/apache/cloudstack/pull/2514#issuecomment-376837801 To tell you the truth I do not understand how you solve this in the routing table. I was

[GitHub] rafaelweingartner commented on issue #2514: Problem with NAT configuration and VMs not accessing each other via public IPs

rafaelweingartner commented on issue #2514: Problem with NAT configuration and VMs not accessing each other via public IPs URL: https://github.com/apache/cloudstack/pull/2514#issuecomment-376837801 To tell you the truth, I do not understand how you solve this in the routing table. I saw

[GitHub] rafaelweingartner commented on issue #2514: Problem with NAT configuration and VMs not accessing each other via public IPs

rafaelweingartner commented on issue #2514: Problem with NAT configuration and VMs not accessing each other via public IPs URL: https://github.com/apache/cloudstack/pull/2514#issuecomment-376837801 To tell you the truth, I do not understand how you solve this in the routing table. I was

[GitHub] ernjvr commented on issue #2449: CLOUDSTACK-10278 idempotent column addition

ernjvr commented on issue #2449: CLOUDSTACK-10278 idempotent column addition URL: https://github.com/apache/cloudstack/pull/2449#issuecomment-376842088 Upgrade Test Result: I installed a version 4.6 environment with some of the DB changes that were already in 4.11, created a few networks:

[GitHub] blueorangutan commented on issue #2512: Only use the host if its Resource State is Enabled.

blueorangutan commented on issue #2512: Only use the host if its Resource State is Enabled. URL: https://github.com/apache/cloudstack/pull/2512#issuecomment-376849406 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] DaanHoogland commented on issue #2512: Only use the host if its Resource State is Enabled.

DaanHoogland commented on issue #2512: Only use the host if its Resource State is Enabled. URL: https://github.com/apache/cloudstack/pull/2512#issuecomment-376849160 @blueorangutan package This is an automated message from

[GitHub] DaanHoogland commented on issue #2510: CLOUDSTACK-10334: Fix inadequate information for handling catch clauses

DaanHoogland commented on issue #2510: CLOUDSTACK-10334: Fix inadequate information for handling catch clauses URL: https://github.com/apache/cloudstack/pull/2510#issuecomment-376804536 not sure @rafaelweingartner , should we? @blueorangutan package

[GitHub] blueorangutan commented on issue #2510: CLOUDSTACK-10334: Fix inadequate information for handling catch clauses

blueorangutan commented on issue #2510: CLOUDSTACK-10334: Fix inadequate information for handling catch clauses URL: https://github.com/apache/cloudstack/pull/2510#issuecomment-376812696 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1846

[GitHub] rhtyd commented on issue #2436: Enable restart of redundant VPCs implementing Rolling Restart

rhtyd commented on issue #2436: Enable restart of redundant VPCs implementing Rolling Restart URL: https://github.com/apache/cloudstack/pull/2436#issuecomment-376819867 I'll move the changes to https://github.com/apache/cloudstack/pull/2508. Closing this.

[GitHub] rhtyd closed pull request #2436: Enable restart of redundant VPCs implementing Rolling Restart

rhtyd closed pull request #2436: Enable restart of redundant VPCs implementing Rolling Restart URL: https://github.com/apache/cloudstack/pull/2436 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] rhtyd commented on issue #2412: CLOUDSTACK-9677: Adding storage policy support for swift as secondary…

rhtyd commented on issue #2412: CLOUDSTACK-9677: Adding storage policy support for swift as secondary… URL: https://github.com/apache/cloudstack/pull/2412#issuecomment-376848209 Ping @khos2ow can you advise on testing of this PR against swift? This is ready to merge based on 2+lgtms and

[GitHub] rhtyd commented on issue #2510: CLOUDSTACK-10334: Fix inadequate information for handling catch clauses

rhtyd commented on issue #2510: CLOUDSTACK-10334: Fix inadequate information for handling catch clauses URL: https://github.com/apache/cloudstack/pull/2510#issuecomment-376806301 @DaanHoogland @rafaelweingartner this is simply a logging change, Travis is green. I'm okay merging based on

[GitHub] blueorangutan commented on issue #2512: Only use the host if its Resource State is Enabled.

blueorangutan commented on issue #2512: Only use the host if its Resource State is Enabled. URL: https://github.com/apache/cloudstack/pull/2512#issuecomment-376856138 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1847

[GitHub] ustcweizhou commented on issue #2514: Problem with NAT configuration and VMs not accessing each other via public IPs

ustcweizhou commented on issue #2514: Problem with NAT configuration and VMs not accessing each other via public IPs URL: https://github.com/apache/cloudstack/pull/2514#issuecomment-376835719 @rafaelweingartner We faced the same issue before. it is fixed by adding some rules in ip

[GitHub] rafaelweingartner commented on issue #2514: Problem with NAT configuration and VMs not accessing each other via public IPs

rafaelweingartner commented on issue #2514: Problem with NAT configuration and VMs not accessing each other via public IPs URL: https://github.com/apache/cloudstack/pull/2514#issuecomment-376837801 To tell you the truth, I do not understand how you solve this in the routing table. I saw

[GitHub] rhtyd commented on issue #2449: CLOUDSTACK-10278 idempotent column addition

rhtyd commented on issue #2449: CLOUDSTACK-10278 idempotent column addition URL: https://github.com/apache/cloudstack/pull/2449#issuecomment-376805263 Will re-kick test. @blueorangutan test This is an automated message

[GitHub] nitin-maharana commented on a change in pull request #2512: Only use the host if its Resource State is Enabled.

nitin-maharana commented on a change in pull request #2512: Only use the host if its Resource State is Enabled. URL: https://github.com/apache/cloudstack/pull/2512#discussion_r177693341 ## File path:

[GitHub] ernjvr commented on issue #2449: CLOUDSTACK-10278 idempotent column addition

ernjvr commented on issue #2449: CLOUDSTACK-10278 idempotent column addition URL: https://github.com/apache/cloudstack/pull/2449#issuecomment-376842088 Upgrade Test Result: I installed a version 4.6 environment, created a few networks: isolated, shared, vpc and created vms on each. I

[GitHub] DaanHoogland closed pull request #2449: CLOUDSTACK-10278 idempotent column addition

DaanHoogland closed pull request #2449: CLOUDSTACK-10278 idempotent column addition URL: https://github.com/apache/cloudstack/pull/2449 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is

[cloudstack] branch 4.11 updated: CLOUDSTACK-10278 idempotent column addition (#2449)

This is an automated email from the ASF dual-hosted git repository. dahn pushed a commit to branch 4.11 in repository https://gitbox.apache.org/repos/asf/cloudstack.git The following commit(s) were added to refs/heads/4.11 by this push: new 7647e10 CLOUDSTACK-10278 idempotent column

[GitHub] DaanHoogland commented on issue #2449: CLOUDSTACK-10278 idempotent column addition

DaanHoogland commented on issue #2449: CLOUDSTACK-10278 idempotent column addition URL: https://github.com/apache/cloudstack/pull/2449#issuecomment-376851510 merging based on LGTMs and test reports This is an automated

[GitHub] borisstoyanov commented on issue #2505: CLOUDSTACK-10333: Secure Live VM Migration for KVM

borisstoyanov commented on issue #2505: CLOUDSTACK-10333: Secure Live VM Migration for KVM URL: https://github.com/apache/cloudstack/pull/2505#issuecomment-376871848 Maybe this ^^ needs further investigation since I've provisioned the certificates on both the hosts and still getting the

[GitHub] blueorangutan commented on issue #2512: Only use the host if its Resource State is Enabled.

blueorangutan commented on issue #2512: Only use the host if its Resource State is Enabled. URL: https://github.com/apache/cloudstack/pull/2512#issuecomment-376860368 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] borisstoyanov commented on issue #2505: CLOUDSTACK-10333: Secure Live VM Migration for KVM

borisstoyanov commented on issue #2505: CLOUDSTACK-10333: Secure Live VM Migration for KVM URL: https://github.com/apache/cloudstack/pull/2505#issuecomment-376870609 @rhtyd I'm trying to migrate a VM between unsecured hosts and got the following exception: ``` 2018-03-28

[GitHub] blueorangutan commented on issue #2505: CLOUDSTACK-10333: Secure Live VM Migration for KVM

blueorangutan commented on issue #2505: CLOUDSTACK-10333: Secure Live VM Migration for KVM URL: https://github.com/apache/cloudstack/pull/2505#issuecomment-376870877 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

[GitHub] ernjvr commented on issue #2490: Create database upgrade from 4.11.0.0 to 4.11.1.0 & VMWare version to OS mappings

ernjvr commented on issue #2490: Create database upgrade from 4.11.0.0 to 4.11.1.0 & VMWare version to OS mappings URL: https://github.com/apache/cloudstack/pull/2490#issuecomment-376858228 Upgrade Test Result: Created an environment to install a 4.11 version using this PR code. After the

[GitHub] blueorangutan commented on issue #2506: CLOUDSTACK-10341: Reduce systemvmtemplate size, install nftables

blueorangutan commented on issue #2506: CLOUDSTACK-10341: Reduce systemvmtemplate size, install nftables URL: https://github.com/apache/cloudstack/pull/2506#issuecomment-376859587 Trillian test result (tid-2428) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6

[GitHub] DaanHoogland commented on issue #2490: Create database upgrade from 4.11.0.0 to 4.11.1.0 & VMWare version to OS mappings

DaanHoogland commented on issue #2490: Create database upgrade from 4.11.0.0 to 4.11.1.0 & VMWare version to OS mappings URL: https://github.com/apache/cloudstack/pull/2490#issuecomment-376860152 @nitin-maharana @nvazquez @rhtyd can we merge this?

[GitHub] DaanHoogland commented on issue #2512: Only use the host if its Resource State is Enabled.

DaanHoogland commented on issue #2512: Only use the host if its Resource State is Enabled. URL: https://github.com/apache/cloudstack/pull/2512#issuecomment-376860281 @blueorangutan test This is an automated message from the

[GitHub] borisstoyanov commented on issue #2505: CLOUDSTACK-10333: Secure Live VM Migration for KVM

borisstoyanov commented on issue #2505: CLOUDSTACK-10333: Secure Live VM Migration for KVM URL: https://github.com/apache/cloudstack/pull/2505#issuecomment-376870609 @rhtyd I'm trying to migrate a VM between unsecured hosts and got the following exception: ``` 2018-03-28

[GitHub] blueorangutan commented on issue #2505: CLOUDSTACK-10333: Secure Live VM Migration for KVM

blueorangutan commented on issue #2505: CLOUDSTACK-10333: Secure Live VM Migration for KVM URL: https://github.com/apache/cloudstack/pull/2505#issuecomment-376878832 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-1848 This

[GitHub] blueorangutan commented on issue #2462: [CLOUDSTACK-10301] Allow updating the network ACL list name and Description

blueorangutan commented on issue #2462: [CLOUDSTACK-10301] Allow updating the network ACL list name and Description URL: https://github.com/apache/cloudstack/pull/2462#issuecomment-376969129 @borisstoyanov a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run

[GitHub] nvazquez commented on issue #2490: Create database upgrade from 4.11.0.0 to 4.11.1.0 & VMWare version to OS mappings

nvazquez commented on issue #2490: Create database upgrade from 4.11.0.0 to 4.11.1.0 & VMWare version to OS mappings URL: https://github.com/apache/cloudstack/pull/2490#issuecomment-376977574 Sorry for the delay @ernjvr @rhtyd @DaanHoogland. I've examined CPU cap failure and found out

[GitHub] borisstoyanov commented on issue #2462: [CLOUDSTACK-10301] Allow updating the network ACL list name and Description

borisstoyanov commented on issue #2462: [CLOUDSTACK-10301] Allow updating the network ACL list name and Description URL: https://github.com/apache/cloudstack/pull/2462#issuecomment-376969001 sure @rafaelweingartner @blueorangutan test

[GitHub] borisstoyanov commented on issue #2492: Fix the name of the column used to hold IPv4 range in 'vlan' table.

borisstoyanov commented on issue #2492: Fix the name of the column used to hold IPv4 range in 'vlan' table. URL: https://github.com/apache/cloudstack/pull/2492#issuecomment-376968757 These errors aren't reported frequently.. but I don't think they're related... don't see any reason to..

[GitHub] blueorangutan commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop)

blueorangutan commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop) URL: https://github.com/apache/cloudstack/pull/2511#issuecomment-377005467 Trillian test result (tid-2430) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt

[GitHub] blueorangutan commented on issue #2401: [CLOUDSTACK-10226] CloudStack is not importing Local storage properly

blueorangutan commented on issue #2401: [CLOUDSTACK-10226] CloudStack is not importing Local storage properly URL: https://github.com/apache/cloudstack/pull/2401#issuecomment-377066408 Trillian test result (tid-2436) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7

[GitHub] rafaelweingartner commented on issue #2401: [CLOUDSTACK-10226] CloudStack is not importing Local storage properly

rafaelweingartner commented on issue #2401: [CLOUDSTACK-10226] CloudStack is not importing Local storage properly URL: https://github.com/apache/cloudstack/pull/2401#issuecomment-377066722 @DaanHoogland I checked this last log result. Can I consider that the tests is successful then?

[GitHub] blueorangutan commented on issue #2505: CLOUDSTACK-10333: Secure Live VM Migration for KVM

blueorangutan commented on issue #2505: CLOUDSTACK-10333: Secure Live VM Migration for KVM URL: https://github.com/apache/cloudstack/pull/2505#issuecomment-377036238 Trillian test result (tid-2427) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time

[GitHub] rafaelweingartner commented on issue #2401: [CLOUDSTACK-10226] CloudStack is not importing Local storage properly

rafaelweingartner commented on issue #2401: [CLOUDSTACK-10226] CloudStack is not importing Local storage properly URL: https://github.com/apache/cloudstack/pull/2401#issuecomment-376947269 I checked the logs. The problems did not seem to be related to the PR. However, it is always good

[GitHub] blueorangutan commented on issue #2449: CLOUDSTACK-10278 idempotent column addition

blueorangutan commented on issue #2449: CLOUDSTACK-10278 idempotent column addition URL: https://github.com/apache/cloudstack/pull/2449#issuecomment-376805440 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[GitHub] DaanHoogland commented on issue #2401: [CLOUDSTACK-10226] CloudStack is not importing Local storage properly

DaanHoogland commented on issue #2401: [CLOUDSTACK-10226] CloudStack is not importing Local storage properly URL: https://github.com/apache/cloudstack/pull/2401#issuecomment-376946409 I think it is @rafaelweingartner let me rekick anyway, you can download the logs and examine them.

[GitHub] rafaelweingartner commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop)

rafaelweingartner commented on issue #2511: [CLOUDSTACK-10344] bug when moving ACL rules (change order with drag and drop) URL: https://github.com/apache/cloudstack/pull/2511#issuecomment-376940534 @nitin-maharana thanks for the review. Do you approve the PR?

[GitHub] rafaelweingartner commented on issue #2496: [CLOUDSTACK-10332] Users are not able to change/edit the protocol of an ACL rule

rafaelweingartner commented on issue #2496: [CLOUDSTACK-10332] Users are not able to change/edit the protocol of an ACL rule URL: https://github.com/apache/cloudstack/pull/2496#issuecomment-376940834 @borisstoyanov thanks! So, are you ok with this PR?

[GitHub] rafaelweingartner commented on issue #2492: Fix the name of the column used to hold IPv4 range in 'vlan' table.

rafaelweingartner commented on issue #2492: Fix the name of the column used to hold IPv4 range in 'vlan' table. URL: https://github.com/apache/cloudstack/pull/2492#issuecomment-376941144 @borisstoyanov are these errors intermittent ones?

[GitHub] blueorangutan commented on issue #2449: CLOUDSTACK-10278 idempotent column addition

blueorangutan commented on issue #2449: CLOUDSTACK-10278 idempotent column addition URL: https://github.com/apache/cloudstack/pull/2449#issuecomment-376941687 Trillian test result (tid-2429) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken:

[GitHub] rafaelweingartner commented on issue #2449: CLOUDSTACK-10278 idempotent column addition

rafaelweingartner commented on issue #2449: CLOUDSTACK-10278 idempotent column addition URL: https://github.com/apache/cloudstack/pull/2449#issuecomment-376942892 @DaanHoogland are you going to introduce the changes here in master as well?

[GitHub] rafaelweingartner commented on issue #2462: [CLOUDSTACK-10301] Allow updating the network ACL list name and Description

rafaelweingartner commented on issue #2462: [CLOUDSTACK-10301] Allow updating the network ACL list name and Description URL: https://github.com/apache/cloudstack/pull/2462#issuecomment-376944372 @borisstoyanov can you run tests here? This is a simple, but useful feature in production.

[GitHub] rafaelweingartner commented on issue #2486: [CLOUDSTACK-10323] Allow changing disk offering during volume migration

rafaelweingartner commented on issue #2486: [CLOUDSTACK-10323] Allow changing disk offering during volume migration URL: https://github.com/apache/cloudstack/pull/2486#issuecomment-376943966 @borisstoyanov I ammended the code as you suggested.

[GitHub] rafaelweingartner commented on issue #2486: [CLOUDSTACK-10323] Allow changing disk offering during volume migration

rafaelweingartner commented on issue #2486: [CLOUDSTACK-10323] Allow changing disk offering during volume migration URL: https://github.com/apache/cloudstack/pull/2486#issuecomment-376943966 @borisstoyanov I amended the code as you suggested.

[GitHub] rafaelweingartner commented on issue #2401: [CLOUDSTACK-10226] CloudStack is not importing Local storage properly

rafaelweingartner commented on issue #2401: [CLOUDSTACK-10226] CloudStack is not importing Local storage properly URL: https://github.com/apache/cloudstack/pull/2401#issuecomment-376944918 @DaanHoogland it seems that the tests you executed here were successful (despite the errors

[GitHub] blueorangutan commented on issue #2401: [CLOUDSTACK-10226] CloudStack is not importing Local storage properly

blueorangutan commented on issue #2401: [CLOUDSTACK-10226] CloudStack is not importing Local storage properly URL: https://github.com/apache/cloudstack/pull/2401#issuecomment-376946598 @DaanHoogland a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke

[GitHub] rafaelweingartner commented on issue #2414: [CLOUDSTACK-10241] Duplicated file SRs being created in XenServer pools

rafaelweingartner commented on issue #2414: [CLOUDSTACK-10241] Duplicated file SRs being created in XenServer pools URL: https://github.com/apache/cloudstack/pull/2414#issuecomment-376931360 @rhtyd I will wait more 24 hours before merging this one to give you some extra time in cause you