[GitHub] cloudstack issue #1980: CLOUDSTACK-9805: Display VR list in network details
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1980 Who (user type) has visibility tab of this tab? can it be hidden ? many service providers **_may_** not want users to see this. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1990: CLOUDSTACK-9674: Baremetal deployment improvements
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1990 Hi, The CloudStack documentation is very light on baremetal deployment. Please can you share some documentation that we use to test and therefore merge this enhancement and also we can then add it to the official documentation so users can consume it.. Thanks --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1878 @sureshanaparti I see in vCenter any deployment of an RVR with a duplicate MAC, failing with a vCenter error regarding a duplicate MAC. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1813 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #2026: CLOUDSTACK-9861: Expire VM snapshots after configure...
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/2026 @blueorangutan package --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack-docs-install pull request: Updates to the vSphere secti...
GitHub user PaulAngus opened a pull request: https://github.com/apache/cloudstack-docs-install/pull/20 Updates to the vSphere section of install docs updated to include vSphere 5.1 and 5.5 improved gramma and explainations allows for non cloudstack vms to be present in vSphere Clusters (it is possible and is required by some SDN solutions). You can merge this pull request into a Git repository by running: $ git pull https://github.com/PaulAngus/cloudstack-docs-install master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack-docs-install/pull/20.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #20 commit 0451ce5c21d442156f0628d3dd4619ac83776c42 Author: <> Date: 2015-03-26T14:01:15Z vmware-hypervisor commit d56fe837579215fa07022827cbf22cbf2a00348e Author: unknown Date: 2015-03-27T17:01:49Z minor spacing change commit 06d09e228a66c05e482a1ecbe9626d0950e0b774 Author: Paul Angus Date: 2015-03-27T21:19:30Z further updates and rewording to vSphere hypervisor section commit 7148a6f05a9601fc5938d2f5ceb2ae51e627f965 Author: Paul Angus Date: 2015-03-27T21:19:50Z Merge branch 'master' of https://github.com/PaulAngus/cloudstack-docs-install --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack-docs-install pull request: Updates to the vSphere secti...
Github user PaulAngus closed the pull request at: https://github.com/apache/cloudstack-docs-install/pull/20 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack-docs-install pull request: Storage Setup Documentation ...
GitHub user PaulAngus opened a pull request: https://github.com/apache/cloudstack-docs-install/pull/21 Storage Setup Documentation Improvement Storage section of install docs expanded and improved to give much more background information, You can merge this pull request into a Git repository by running: $ git pull https://github.com/PaulAngus/cloudstack-docs-install master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack-docs-install/pull/21.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #21 commit 0451ce5c21d442156f0628d3dd4619ac83776c42 Author: <> Date: 2015-03-26T14:01:15Z vmware-hypervisor commit d56fe837579215fa07022827cbf22cbf2a00348e Author: unknown Date: 2015-03-27T17:01:49Z minor spacing change commit 06d09e228a66c05e482a1ecbe9626d0950e0b774 Author: Paul Angus Date: 2015-03-27T21:19:30Z further updates and rewording to vSphere hypervisor section commit 7148a6f05a9601fc5938d2f5ceb2ae51e627f965 Author: Paul Angus Date: 2015-03-27T21:19:50Z Merge branch 'master' of https://github.com/PaulAngus/cloudstack-docs-install commit 0b1e377a1b894e2e25c2d6d48b821ba61358204f Author: Paul Angus Date: 2015-04-07T07:42:49Z large storage docs update pt1 commit ee0d5ee485cfef0fc3154991e36a98b121f776fc Author: unknown Date: 2015-04-08T16:54:17Z large update to storage doc pt2 + images included. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack-docs-install pull request: Storage Setup Documentation ...
Github user PaulAngus commented on the pull request: https://github.com/apache/cloudstack-docs-install/pull/21#issuecomment-90975913 I seem not to have cleaned out commits from previous PR in this one. 0451ce5, d56fe83, 06d09e2 and 7148a6f already comitted. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack-docs-install pull request: Storage Setup Documentation ...
Github user PaulAngus closed the pull request at: https://github.com/apache/cloudstack-docs-install/pull/21 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack-docs-install pull request: Storage section of install d...
GitHub user PaulAngus opened a pull request: https://github.com/apache/cloudstack-docs-install/pull/22 Storage section of install docs expanded and improved to give much more ... ...background information, images added You can merge this pull request into a Git repository by running: $ git pull https://github.com/PaulAngus/cloudstack-docs-install master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack-docs-install/pull/22.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #22 commit f1c9141eda30b9ac5b63a38894547201e2d84a0d Author: Paul Angus Date: 2015-04-09T14:57:38Z Storage section of install docs expanded and improved to give much more background information, images added --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request: [4.7] VMware boot order fix
Github user PaulAngus commented on the pull request: https://github.com/apache/cloudstack/pull/1395#issuecomment-180581817 I believe Abhi is going to be looking at Marvin soon. We could maybe consider using the vmware python libraries (https://github.com/vmware/pyvmomi) when using Marvin to test vSphere specific code. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request: Remove extraenous log dir and add catalin...
GitHub user PaulAngus opened a pull request: https://github.com/apache/cloudstack/pull/1087 Remove extraenous log dir and add catalina.out log rotation. remove unused /var/log/cloudstack-management directory add cloudstack-catalina to logrotate.d directory so that /var/log/cloudstack/management/catalina.out is rotated. You can merge this pull request into a Git repository by running: $ git pull https://github.com/PaulAngus/cloudstack master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1087.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1087 commit 19e5166b8ce35653d207a3823902d9c2178c35ca Author: Paul Angus Date: 2015-11-16T13:17:03Z remove unused /var/log/cloudstack-management directory add cloudstack-catalina to logrotate.d directory so that /var/log/cloudstack/management/catalina.out is rotated --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request: Remove extraenous log dir and add catalin...
Github user PaulAngus commented on the pull request: https://github.com/apache/cloudstack/pull/1087#issuecomment-157708006 Apologies. https://issues.apache.org/jira/browse/CLOUDSTACK-9073 created removal of /var/log/cloudstack-management and addition of catalina log rotation in /etc/logrotate.d/ has been running against 4.5.1 for 2 months with no issues. updated packaging was built against current master and (centos6 and 7) deployed with files/directories as expected. Left running overnight /var/log/cloudstack/management/catalina.log seen to rotate and compress previous file. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request: Remove extraenous log dir and add catalin...
Github user PaulAngus commented on the pull request: https://github.com/apache/cloudstack/pull/1087#issuecomment-161780789 So looking at the rpm spec options using +%config(noreplace) will cause a new version of cloudstack-catalina (cloudstack-catalina.rpmnew) to be created if cloudstack-catalina has been edited. However as the contents of /etc/logrotate.d/ are enumerated and run this would result in both the new and old behaviour both being applied - which seems the worst of all worlds. Therefore I suggest the orginal PR (without commit ec698f5377ad98ac2a34488a0102c45c2408d089) which will overwrite if the new rpm has an updated value in it [1]; is the lesser of two evils. [1] http://www-uxsup.csx.cam.ac.uk/~jw35/docs/rpm_config.html --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request: Remove extraneous log directory and add c...
Github user PaulAngus commented on the pull request: https://github.com/apache/cloudstack/pull/1537#issuecomment-218744867 Jenkins has run successfully @swill --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request: Remove extraneous log directory and add c...
GitHub user PaulAngus reopened a pull request: https://github.com/apache/cloudstack/pull/1537 Remove extraneous log directory and add catalina.out log rotation rebased and squashed replacement for PR: https://github.com/apache/cloudstack/pull/1087 Remove extraneous log directory (/var/log/cloudstack-management) and adds catalina.out log rotation You can merge this pull request into a Git repository by running: $ git pull https://github.com/PaulAngus/cloudstack master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1537.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1537 commit 11e3582fdf2ab94e70591f4603e84db59519f342 Author: Paul Angus Date: 2016-05-10T20:24:29Z Remove extraneous log directory and add catalina.out log rotation --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request: Remove extraneous log directory and add c...
Github user PaulAngus closed the pull request at: https://github.com/apache/cloudstack/pull/1537 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request: Remove extraneous log directory and add c...
GitHub user PaulAngus reopened a pull request: https://github.com/apache/cloudstack/pull/1537 Remove extraneous log directory and add catalina.out log rotation rebased and squashed replacement for PR: https://github.com/apache/cloudstack/pull/1087 Remove extraneous log directory (/var/log/cloudstack-management) and adds catalina.out log rotation You can merge this pull request into a Git repository by running: $ git pull https://github.com/PaulAngus/cloudstack master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1537.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1537 commit 11e3582fdf2ab94e70591f4603e84db59519f342 Author: Paul Angus Date: 2016-05-10T20:24:29Z Remove extraneous log directory and add catalina.out log rotation --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request: Remove extraneous log directory and add c...
Github user PaulAngus closed the pull request at: https://github.com/apache/cloudstack/pull/1537 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request: Remove extraneous log directory and add c...
GitHub user PaulAngus reopened a pull request: https://github.com/apache/cloudstack/pull/1537 Remove extraneous log directory and add catalina.out log rotation rebased and squashed replacement for PR: https://github.com/apache/cloudstack/pull/1087 Remove extraneous log directory (/var/log/cloudstack-management) and adds catalina.out log rotation You can merge this pull request into a Git repository by running: $ git pull https://github.com/PaulAngus/cloudstack master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1537.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1537 commit 11e3582fdf2ab94e70591f4603e84db59519f342 Author: Paul Angus Date: 2016-05-10T20:24:29Z Remove extraneous log directory and add catalina.out log rotation --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request: Remove extraneous log directory and add c...
Github user PaulAngus closed the pull request at: https://github.com/apache/cloudstack/pull/1537 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request: Remove extraneous log directory and add c...
Github user PaulAngus commented on the pull request: https://github.com/apache/cloudstack/pull/1537#issuecomment-219077622 Hurrah!! passed both this time @swill --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request: Remove extraneous log directory and add c...
Github user PaulAngus commented on the pull request: https://github.com/apache/cloudstack/pull/1537#issuecomment-219119839 Thanks Will. Kind regards, Paul Angus Regards Paul Angus VP Technology www.shapeblue.com From: Will Stevens [mailto:notificati...@github.com] Sent: 13 May 2016 16:38 To: apache/cloudstack Cc: Paul Angus ; State change Subject: Re: [apache/cloudstack] Remove extraneous log directory and add catalina.out log rotation (#1537) Thank you for your patience sir. ð I will get this merged... â You are receiving this because you modified the open/close state. Reply to this email directly or view it on GitHub<https://github.com/apache/cloudstack/pull/1537#issuecomment-219079650> --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request: Remove extraenous log dir and add catalin...
Github user PaulAngus closed the pull request at: https://github.com/apache/cloudstack/pull/1087 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1612: packaging: Marvin and integration-tests packages
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1612 @rohit is it worth putting into straight into the apachecloudstack repo (rather than apache/cloudstack). Kind regards, Paul Angus VP Technology paul.an...@shapeblue.com www.shapeblue.com From: Rohit Yadav [mailto:notificati...@github.com] Sent: 20 July 2016 07:28 To: apache/cloudstack Cc: Paul Angus ; Mention Subject: [apache/cloudstack] packaging: Marvin and integration-tests packages (#1612) This introduces two new cloudstack packages: marvin and integration-tests. The two packages will make it easier for CI systems to install Marvin for a specific cloudstack release/build and run integration tests that are specific for that version/build. /cc @swill<https://github.com/swill> @PaulAngus<https://github.com/PaulAngus> @jburwell<https://github.com/jburwell> @wido<https://github.com/wido> @bvbharat<https://github.com/bvbharat> You can view, comment on, or merge this pull request online at: https://github.com/apache/cloudstack/pull/1612 Commit Summary * packaging: Marvin and integration-tests packages File Changes * A debian/cloudstack-integration-tests.install<https://github.com/apache/cloudstack/pull/1612/files#diff-0> (18) * A debian/cloudstack-marvin.install<https://github.com/apache/cloudstack/pull/1612/files#diff-1> (18) * A debian/cloudstack-marvin.postinst<https://github.com/apache/cloudstack/pull/1612/files#diff-2> (22) * M debian/control<https://github.com/apache/cloudstack/pull/1612/files#diff-3> (10) * M debian/rules<https://github.com/apache/cloudstack/pull/1612/files#diff-4> (8) * M packaging/centos63/cloud.spec<https://github.com/apache/cloudstack/pull/1612/files#diff-5> (35) * M packaging/centos7/cloud.spec<https://github.com/apache/cloudstack/pull/1612/files#diff-6> (35) Patch Links: * https://github.com/apache/cloudstack/pull/1612.patch * https://github.com/apache/cloudstack/pull/1612.diff â You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<https://github.com/apache/cloudstack/pull/1612>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AEll7DVR5Q2zsmQAwaeoUZjLMzwggN41ks5qXcAKgaJpZM4JQdsP>. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1612: packaging: Marvin and integration-tests packages
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1612 Sorry yes â just wondering â rather than have to move it later as well⦠Kind regards, Paul Angus VP Technology paul.an...@shapeblue.com www.shapeblue.com From: Rohit Yadav [mailto:notificati...@github.com] Sent: 20 July 2016 09:22 To: apache/cloudstack Cc: Paul Angus ; Mention Subject: Re: [apache/cloudstack] packaging: Marvin and integration-tests packages (#1612) @PaulAngus<https://github.com/PaulAngus> sorry I did not get the question. Put what repo in that org? Based on the response received on the dev-ML thread on splitting marvin, this PR implements the alternative to package marvin and tests as separately install-able deb/rpm packages. â You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<https://github.com/apache/cloudstack/pull/1612#issuecomment-233880931>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AEll7MKmlPcMAc28Yf9HcW-uocAOM2wiks5qXdqLgaJpZM4JQdsP>. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1649: CLOUDSTACK-9463: Fix dynamic-roles migrate script fo...
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1649 LGTM tested against a client's customised commands.properties file. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1650: fix macchinina template specificied for vmwar...
GitHub user PaulAngus opened a pull request: https://github.com/apache/cloudstack/pull/1650 fix macchinina template specificied for vmware in Marvin tests fix macchinina template specificied for vmware - .vmdk.bz2 is not valid⦠for CloudStack. edit to use .ova You can merge this pull request into a Git repository by running: $ git pull https://github.com/shapeblue/cloudstack templatefix Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1650.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1650 commit 4d22367760ab1c605bbc31e9c749ebb544c4c9ca Author: Paul Angus Date: 2016-08-19T15:23:26Z fix macchinina template speficied for vmware - .vmdk.bz2 is not valid for CloudStack. edit to use .ova --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1651: Marvin Tests: fix expected return string for ...
GitHub user PaulAngus opened a pull request: https://github.com/apache/cloudstack/pull/1651 Marvin Tests: fix expected return string for successful pings. fix expected return string for successful pings. these pings are from the test instance inside the isolated networks. The original response was searched for was that which would be seen on a Mac. You can merge this pull request into a Git repository by running: $ git pull https://github.com/shapeblue/cloudstack pingfixes Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1651.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1651 commit 686b87d68051d4e3f0a9619d346a29d84d82d5ac Author: Paul Angus Date: 2016-08-19T15:25:57Z fix expected return string for successful pings. these pings are from the test instance inside the isolated networks. The original response was taken from a Mac. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1652: Marvin Tests: Fix VPC network offering select...
GitHub user PaulAngus opened a pull request: https://github.com/apache/cloudstack/pull/1652 Marvin Tests: Fix VPC network offering selection . It appears that original code did not expect there to be an alternate default VPC offering (Nuage) . VPC selection uses isdefault=True which selects the Nuage Network offering, stopping the test from being run. Fix was to look for name="Default VPC Offering" instead. You can merge this pull request into a Git repository by running: $ git pull https://github.com/shapeblue/cloudstack vpcselectionfix Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1652.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1652 commit f1b64a3b509ed59ac0061a4e11dbff34d7ce1ba0 Author: Paul Angus Date: 2016-08-19T15:29:22Z VPC fix network offering. orginal code did not expect there to be an alternate default (Nuage) there when the isdefault=True is used a Nuage Network offering is returned stopping the test from being run. Fix was to look for name="Default VPC Offering" --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1641: CLOUDSTACK-9459: the try's catch block was shortenin...
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1641 LGTM - tested against a client db which initially failed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1652: Marvin Tests: Fix VPC network offering selection .
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1652 Without full understanding of the Nuage plugin, fixing the test seemed the safest path, one hopes that the nuage offering was configured that way for a good reason. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1651: Marvin Tests: fix expected return string for success...
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1651 The best way to fix this would be a ping function that gets called. However, that's not something that i can personally do and this will work for (at least) Ubuntu, CentOS/RHEL and macchinina - making it infinitely better that it was. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1651: Marvin Tests: fix expected return string for success...
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1651 that works for me. I'll adjust the PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1652: Marvin Tests: Fix VPC network offering selection .
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1652 Thanks frank, would you make that change to the Nuage Network offering or shall I? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1651: Marvin Tests: fix expected return string for ...
Github user PaulAngus closed the pull request at: https://github.com/apache/cloudstack/pull/1651 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1651: Marvin Tests: fix expected return string for success...
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1651 My git foo abandoned me and left a mess â¹ . Iâll be sending a clean pr later today⦠Kind regards, Paul Angus VP Technology paul.an...@shapeblue.com www.shapeblue.com From: serg38 [mailto:notificati...@github.com] Sent: 21 September 2016 12:40 To: apache/cloudstack Cc: Paul Angus ; Mention Subject: Re: [apache/cloudstack] Marvin Tests: fix expected return string for successful pings. (#1651) @PaulAngus<https://github.com/PaulAngus> @jburwell<https://github.com/jburwell> @rhtyd<https://github.com/rhtyd> This PR was closed without merging. Are we still on track to include the fix to 4.9.1 ? I can ask @nvazquez<https://github.com/nvazquez> to pick this one up if needed. â You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<https://github.com/apache/cloudstack/pull/1651#issuecomment-248586339>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AEll7Pg-uQb7jTR2TV6XlXDo1d4JiQF2ks5qsRd3gaJpZM4Jomvr>. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1683: fix ping tests to properly recognise successf...
GitHub user PaulAngus opened a pull request: https://github.com/apache/cloudstack/pull/1683 fix ping tests to properly recognise successful ping test fix ping tests to properly recognise successful ping test You can merge this pull request into a Git repository by running: $ git pull https://github.com/shapeblue/cloudstack pingfixes Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1683.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1683 commit 2b240e8aa6e66e6078a8b95a57c3850d139d7d77 Author: Paul Angus Date: 2016-09-20T19:09:08Z fix ping tests to properly recognise successful ping test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1687: Marvintests vpcofferingfix
GitHub user PaulAngus opened a pull request:
https://github.com/apache/cloudstack/pull/1687
Marvintests vpcofferingfix
It appears that original code did not expect there to be an alternate
default VPC offering (Nuage) .
VPC selection uses isdefault=True which selects the Nuage Network offering,
stopping the test from being run.
Fix was to look for name="Default VPC Offering" instead.
Replaces PR #1652 (same content, differenet branch)
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/shapeblue/cloudstack
marvintests-vpcofferingfix
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/1687.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1687
commit 456680dbb2328e2ac613d9ea422351a06a412601
Author: Will Stevens
Date: 2016-05-11T04:08:44Z
Merge pull request #880 from SudharmaJain/cs-8901
CLOUDSTACK-8901: PrepareTemplate job thread hard-coded to max 8 threads The
thread pool was hardcoded to use 8 threads,
com.cloud.template.TemplateManagerImpl.configure(String, Map):
_preloadExecutor = Executors.newFixedThreadPool(8, new
NamedThreadFactory("Template-Preloader"));
Added the change to pick threadpool size from configuration.
* pr/880:
CLOUDSTACK-8901: PrepareTemplate job thread hard-coded to max 8 threads
Signed-off-by: Will Stevens
commit 95abb6efc350189cb2be0cfd655c1205ced42155
Author: Rohit Yadav
Date: 2016-04-21T10:13:04Z
CLOUDSTACK-9361: Centrally handle API validations
Validate API arguments based on annotations. Introduces:
- NotNullOrEmpty: for doing null and empty string checks
- PositiveNumber: number > 0 (natural number)
Signed-off-by: Rohit Yadav
commit 4347776ac6ef9ae86fb016862f4a6b2376f8319a
Author: Rohit Yadav
Date: 2016-03-15T08:16:06Z
CLOUDSTACK-8562: DB-Backed Dynamic Role Based API Access Checker
This feature allows root administrators to define new roles and associate
API
permissions to them.
A limited form of role-based access control for the CloudStack management
server
API is provided through a properties file, commands.properties, embedded in
the
WAR distribution. Therefore, customizing API permissions requires unpacking
the
distribution and modifying this file consistently on all servers. The old
system
also does not permit the specification of additional roles.
FS:
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Dynamic+Role+Based+API+Access+Checker+for+CloudStack
DB-Backed Dynamic Role Based API Access Checker for CloudStack brings
following
changes, features and use-cases:
- Moves the API access definitions from commands.properties to the mgmt
server DB
- Allows defining custom roles (such as a read-only ROOT admin) beyond the
current set of four (4) roles
- All roles will resolve to one of the four known roles types (Admin,
Resource
Admin, Domain Admin and User) which maintains this association by
requiring
all new defined roles to specify a role type.
- Allows changes to roles and API permissions per role at runtime including
additions or
removal of roles and/or modifications of permissions, without the need
of restarting management server(s)
Upgrade/installation notes:
- The feature will be enabled by default for new installations, existing
deployments will continue to use the older static role based api access
checker
with an option to enable this feature
- During fresh installation or upgrade, the upgrade paths will add four
default
roles based on the four default role types
- For ease of migration, at the time of upgrade commands.properties will be
used
to add existing set of permissions to the default roles. cloud.account
will have a new role_id column which will be populated based on default
roles
as well
Dynamic-roles migration tool: scripts/util/migrate-dynamicroles.py
- Allows admins to migrate to the dynamic role based checker at a future
date
- Performs a harder one-way migrate and update
- Migrates rules from existing commands.properties file into db and
deprecates it
- Enables an internal hidden switch to enable dynamic role based checker
feature
Signed-off-by: Rohit Yadav
commit afcbd270b2eb9086a76dda83c0f9ba2e034b0afd
Author: Rohit Yadav
Date: 2016-04-29T19:10:47Z
CLOUDSTACK-8562: Deprecate commands.properties
- Removes commands.properties file
- Fixes apidocs and marvin to be independent of commands.properties usage
- Removes bundling of commands.properties in deb/rpm packaging
- Removes file references across codebase
[GitHub] cloudstack pull request #1688: Marvin Tests: Fix VPC network offering select...
GitHub user PaulAngus opened a pull request: https://github.com/apache/cloudstack/pull/1688 Marvin Tests: Fix VPC network offering selection It appears that original code did not expect there to be an alternate default VPC offering (Nuage) . VPC selection uses isdefault=True which selects the Nuage Network offering, stopping the test from being run. Fix was to look for name="Default VPC Offering" instead. Replaces PR #1652 (same content, differenet branch) You can merge this pull request into a Git repository by running: $ git pull https://github.com/shapeblue/cloudstack 4.8-marvintest-vpcofferingfix Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1688.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1688 commit 3bafd509caa5368bd7adbd81c65682dda5685b2f Author: Paul Angus Date: 2016-09-23T19:40:36Z Marvin Tests: Fix VPC network offering selection --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1687: Marvintests vpcofferingfix
Github user PaulAngus closed the pull request at: https://github.com/apache/cloudstack/pull/1687 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1691: fix ping tests to properly recognise successf...
GitHub user PaulAngus opened a pull request: https://github.com/apache/cloudstack/pull/1691 fix ping tests to properly recognise successful ping test fix ping tests to properly recognise successful ping test (on 4.8 branch) You can merge this pull request into a Git repository by running: $ git pull https://github.com/shapeblue/cloudstack 4.8-fixpingtests Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1691.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1691 commit 95abb6efc350189cb2be0cfd655c1205ced42155 Author: Rohit Yadav Date: 2016-04-21T10:13:04Z CLOUDSTACK-9361: Centrally handle API validations Validate API arguments based on annotations. Introduces: - NotNullOrEmpty: for doing null and empty string checks - PositiveNumber: number > 0 (natural number) Signed-off-by: Rohit Yadav commit 4347776ac6ef9ae86fb016862f4a6b2376f8319a Author: Rohit Yadav Date: 2016-03-15T08:16:06Z CLOUDSTACK-8562: DB-Backed Dynamic Role Based API Access Checker This feature allows root administrators to define new roles and associate API permissions to them. A limited form of role-based access control for the CloudStack management server API is provided through a properties file, commands.properties, embedded in the WAR distribution. Therefore, customizing API permissions requires unpacking the distribution and modifying this file consistently on all servers. The old system also does not permit the specification of additional roles. FS: https://cwiki.apache.org/confluence/display/CLOUDSTACK/Dynamic+Role+Based+API+Access+Checker+for+CloudStack DB-Backed Dynamic Role Based API Access Checker for CloudStack brings following changes, features and use-cases: - Moves the API access definitions from commands.properties to the mgmt server DB - Allows defining custom roles (such as a read-only ROOT admin) beyond the current set of four (4) roles - All roles will resolve to one of the four known roles types (Admin, Resource Admin, Domain Admin and User) which maintains this association by requiring all new defined roles to specify a role type. - Allows changes to roles and API permissions per role at runtime including additions or removal of roles and/or modifications of permissions, without the need of restarting management server(s) Upgrade/installation notes: - The feature will be enabled by default for new installations, existing deployments will continue to use the older static role based api access checker with an option to enable this feature - During fresh installation or upgrade, the upgrade paths will add four default roles based on the four default role types - For ease of migration, at the time of upgrade commands.properties will be used to add existing set of permissions to the default roles. cloud.account will have a new role_id column which will be populated based on default roles as well Dynamic-roles migration tool: scripts/util/migrate-dynamicroles.py - Allows admins to migrate to the dynamic role based checker at a future date - Performs a harder one-way migrate and update - Migrates rules from existing commands.properties file into db and deprecates it - Enables an internal hidden switch to enable dynamic role based checker feature Signed-off-by: Rohit Yadav commit afcbd270b2eb9086a76dda83c0f9ba2e034b0afd Author: Rohit Yadav Date: 2016-04-29T19:10:47Z CLOUDSTACK-8562: Deprecate commands.properties - Removes commands.properties file - Fixes apidocs and marvin to be independent of commands.properties usage - Removes bundling of commands.properties in deb/rpm packaging - Removes file references across codebase Signed-off-by: Rohit Yadav commit 003b97bcffe640e38d089c98e8215de4c8a805e6 Author: Rohit Yadav Date: 2016-05-04T09:58:12Z maven: Fix jstl version usage This would fix regression from recent mvn version changes. Without this patch users get redirected to error.jsp as jstl-1.2 jar is not installed Signed-off-by: Rohit Yadav commit dc5b52937bf9c14634452ed47a851a689a3015c4 Author: Will Stevens Date: 2016-05-11T05:32:02Z Merge release branch 4.8 to master * 4.8: When no zone name is available display a default commit 71c9c90e0c81ea9e0f90ab71d3430d62cd8fa02e Author: Nick Livens Date: 2016-05-02T11:16:14Z CLOUDSTACK-9365 : updateVirtualMachine with userdata should not error when a VM is attached to multiple networks from which one or more doesn't support userdata commit 9df51faa8a2f0678e7927ac6ef23752b1eae8f71 Author: rahul singal Date: 2016-05-
[GitHub] cloudstack pull request #1691: fix ping tests to properly recognise successf...
Github user PaulAngus closed the pull request at: https://github.com/apache/cloudstack/pull/1691 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1692: fix ping tests to properly recognise successf...
GitHub user PaulAngus opened a pull request: https://github.com/apache/cloudstack/pull/1692 fix ping tests to properly recognise successful ping test fix ping tests to properly recognise successful ping test (on 4.8 branch) You can merge this pull request into a Git repository by running: $ git pull https://github.com/shapeblue/cloudstack 4.8-marvinpingtests Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1692.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1692 commit 3cc59c1165e411610c014b162fc0d1583d6a70d8 Author: Paul Angus Date: 2016-09-26T11:08:41Z fix ping tests to properly recognise successful ping test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1652: Marvin Tests: Fix VPC network offering select...
Github user PaulAngus closed the pull request at: https://github.com/apache/cloudstack/pull/1652 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1683: fix ping tests to properly recognise successf...
Github user PaulAngus closed the pull request at: https://github.com/apache/cloudstack/pull/1683 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1684 That isnt correct @bhaisaab. commands.properties can be different on every mgmt server. Cloudstack would not know which version to use. From: Rohit Yadav Sent: 29 Sep 2016 18:16 To: apache/cloudstack Cc: Paul Angus; Mention Subject: Re: [apache/cloudstack] CLOUDSTACK-9489: the new config vars that are added do not goto DB if⦠(#1684) @jburwell<https://github.com/jburwell> when a fresh mgmt server is deployed for upgrading an existing CloudStack environment, several files from /etc/cloudstack/management are copied to the new server, especially the db.properties file. The commands.properties is no different, in the past all such deployment (where in-place upgrades were avoided) copied all these config files in /etc/cloudstack/management from old server to new server. Official docs only talk about in-place upgrades, our packaging system (post-install/upgrade scripts) assume in-place upgrade as well; if someone's doing fresh mgmt server for upgrade, they know what they are doing and its out-of-scope of what the project can support. At the same time, we may add documentation for such kind of upgrades as well. The solution for this problem is documentation and release/upgrade notes. â You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<https://github.com/apache/cloudstack/pull/1684#issuecomment-250531992>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AEll7O8ks5aJSv3zEVnq9pNqLuZonMjBks5qu_JsgaJpZM4KEn6g>. Paul Angus VP Technology s: +44 203 603 0540 | m: +44 7711 418784 | d: +44 203 468 5163 e: paul.an...@shapeblue.com  | w: www.shapeblue.com  | t: @cloudyangus @shapeblue a: 53 Chandos Place, Covent Garden, London WC2N 4HSUK Shape Blue Ltd is a company incorporated in England & Wales. ShapeBlue Services India LLP is a company incorporated in India and is operated under license from Shape Blue Ltd. Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is operated under license from Shape Blue Ltd. ShapeBlue SA Pty Ltd is a company registered by The Republic of South Africa and is traded under license from Shape Blue Ltd. ShapeBlue is a registered trademark.This email and any attachments to it may be confidential and are intended solely for the use of the individual to whom it is addressed. Any views or opinions expressed are solely those of the author and do not necessarily represent those of Shape Blue Ltd or related companies. If you are not the intended recipient of this email, you must neither take any action based upon its contents, nor copy or show it to anyone. Please contact the sender if you believe you have received this email in error. Find out more about ShapeBlue and our range of CloudStack related services: IaaS Cloud Design & Build|CSForge - rapid IaaS deployment framework CloudStack Consulting|CloudStack Software Engineering CloudStack Infrastructure Support|CloudStack Bootcamp Training Courses --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1692: Fix Smoke Test Failures
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1692 @jburwell you need both the url fix (as the original file type was not valid) AND the macchininia template to have the e1000 driver in it. for the tests to work on vSphere. Nux said that he'd have a look at it. but i haven't heard anything since. To be honest, a strong dependency on a homebrew OS is a little concerning to me, although I understand the potential speed improvements. TinyCore might be more sustainable for us as a community. http://www.tinycorelinux.net/welcome.html --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1692: Fix Smoke Test Failures
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1692 VMWare LGTM Kind regards, Paul Angus VP Technology paul.an...@shapeblue.com www.shapeblue.com From: NuxRo [mailto:notificati...@github.com] Sent: 03 October 2016 14:11 To: apache/cloudstack Cc: Paul Angus ; Mention Subject: Re: [apache/cloudstack] Fix Smoke Test Failures (#1692) Can you guys let me know if this runs OK on all your HVs? I've rebuilt the kernel http://dl.openvm.eu/cloudstack/macchinina/x86_64/testing/ â You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<https://github.com/apache/cloudstack/pull/1692#issuecomment-251101576>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AEll7KlDg3CMfNOpQUgIVQv6pN70ochNks5qwP7GgaJpZM4KGZx8>. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request: Remove extraneous log directory and add c...
GitHub user PaulAngus opened a pull request: https://github.com/apache/cloudstack/pull/1537 Remove extraneous log directory and add catalina.out log rotation rebased and squashed replacement for PR: https://github.com/apache/cloudstack/pull/1087 Remove extraneous log directory (/var/log/cloudstack-management) and adds catalina.out log rotation You can merge this pull request into a Git repository by running: $ git pull https://github.com/PaulAngus/cloudstack master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cloudstack/pull/1537.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1537 commit a90dff507b155ba9977a4aa1e986eeea4f80d717 Author: Paul Angus Date: 2016-05-10T20:24:29Z Remove extraneous log directory and add catalina.out log rotation --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #:
Github user PaulAngus commented on the pull request: https://github.com/apache/cloudstack/commit/97c716b1253281c8bd7fc4aed05e6fd39c9e9fa2#commitcomment-19285723 We shouldn't skip snapshot tests for XenServer (leaving a test coverage hole) when snapshots are supported. if the issue is with the test itself. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1692: Fix Smoke Test Failures
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1692 To be accurate the tests failed due to CLOUDSTACK-9529 [3]: Marvin Tests Do Not Clean Up Properly As the environment ran out of public IP addresses VRs to use. This only highlights the importance of tests properly cleaning up after themselves. Kind regards, Paul Angus VP Technology paul.an...@shapeblue.com www.shapeblue.com From: Rohit Yadav [mailto:notificati...@github.com] Sent: 05 October 2016 09:23 To: apache/cloudstack Cc: Paul Angus ; Mention Subject: Re: [apache/cloudstack] Fix Smoke Test Failures (#1692) Tests from xenserver failed due to an environmental issue, I'll rekick another round once outstanding issues are resolved. Tests on kvm failed due to corner case/env specific cleanup/setup issues. Tests on vmware are mostly passing now. â You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<https://github.com/apache/cloudstack/pull/1692#issuecomment-251612450>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AEll7Eo91vk40NIUCxJK0fhnZwmSbV-xks5qw153gaJpZM4KGZx8>. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1711: Xenserver7 Support
Github user PaulAngus commented on the issue:
https://github.com/apache/cloudstack/pull/1711
test_primary_storage.py fails iSCSI test.
**Marvin resports:** createstoragepool failed, due to: errorCode: 530,
errorText:Failed to add data store: Failed to access storage pool
**cloudstack-management log reports:**
Unable to setup heartbeat sr on SR 10e0c167-06fa-f654-4b3e-ecd7c5800447 due
to #7# volume hb-0988b965-ce0e-4dd2-af70-318ea063a008 is not created
Running **setup_heartbeat_sr.sh** manually returns:
> Command not permitted while global/metadata_read_only is set
from
http://discussions.citrix.com/topic/378957-command-not-permitted-while-globalmetadata-read-only-is-set/
adding --config global{metadata_read_only=0} to line 75 (lvcreate
VG_XenStorage-$2 -n hb-$1 --size 4M) resolves the issue.
I don't know if adding that global config switch is persisted or if it has
other side effects.
But it allows iSCSI to attach and hb volume to be created...
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] cloudstack issue #1711: Xenserver7 Support
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1711 Results from smoke tests (with above iscsi fix in) Test completed. 41 look ok, 7 have error(s) in addition to the the known VPC issues there are: test_07_resize_fail | `Failure` | 45.55 | test_volumes.py test_04_extract_template | `Failure` | 5.24 | test_templates.py test_01_create_template | `Error` | 70.56 | test_templates.py test_04_extract_Iso | `Error` | 5.09 | test_iso.py I'll look at these (tomorrow as i'm with customer all day today)... Test | Result | Time (s) | Test File --- | --- | --- | --- test_05_rvpc_multi_tiers | `Failure` | 514.00 | test_vpc_redundant.py test_04_rvpc_network_garbage_collector_nics | `Failure` | 1356.43 | test_vpc_redundant.py test_01_create_redundant_VPC_2tiers_4VMs_4IPs_4PF_ACL | `Failure` | 547.92 | test_vpc_redundant.py test_07_resize_fail | `Failure` | 45.55 | test_volumes.py test_04_extract_template | `Failure` | 5.24 | test_templates.py test_03_vpc_privategw_restart_vpc_cleanup | `Failure` | 441.18 | test_privategw_acl.py test_02_vpc_privategw_static_routes | `Failure` | 441.04 | test_privategw_acl.py test_01_create_template | `Error` | 70.56 | test_templates.py test_network_rules_acquired_public_ip_3_Load_Balancer_Rule | `Error` | 765.83 | test_network.py test_network_rules_acquired_public_ip_2_nat_rule | `Error` | 743.10 | test_network.py test_network_rules_acquired_public_ip_1_static_nat_rule | `Error` | 739.12 | test_network.py test_04_extract_Iso | `Error` | 5.09 | test_iso.py runTest | `Error` | 0.00 | test_dynamicroles.py test_01_vpc_site2site_vpn | Success | 295.18 | test_vpc_vpn.py test_01_vpc_remote_access_vpn | Success | 136.28 | test_vpc_vpn.py test_01_redundant_vpc_site2site_vpn | Success | 511.64 | test_vpc_vpn.py test_02_VPC_default_routes | Success | 304.54 | test_vpc_router_nics.py test_01_VPC_nics_after_destroy | Success | 597.62 | test_vpc_router_nics.py test_03_create_redundant_VPC_1tier_2VMs_2IPs_2PF_ACL_reboot_routers | Success | 779.36 | test_vpc_redundant.py test_02_redundant_VPC_default_routes | Success | 1046.48 | test_vpc_redundant.py test_09_delete_detached_volume | Success | 15.67 | test_volumes.py test_08_resize_volume | Success | 70.67 | test_volumes.py test_06_download_detached_volume | Success | 20.33 | test_volumes.py test_05_detach_volume | Success | 100.26 | test_volumes.py test_04_delete_attached_volume | Success | 10.19 | test_volumes.py test_03_download_attached_volume | Success | 15.24 | test_volumes.py test_02_attach_volume | Success | 10.66 | test_volumes.py test_01_create_volume | Success | 392.32 | test_volumes.py test_03_delete_vm_snapshots | Success | 280.22 | test_vm_snapshots.py test_02_revert_vm_snapshots | Success | 186.25 | test_vm_snapshots.py test_01_create_vm_snapshots | Success | 100.73 | test_vm_snapshots.py test_deploy_vm_multiple | Success | 277.93 | test_vm_life_cycle.py test_deploy_vm | Success | 0.02 | test_vm_life_cycle.py test_advZoneVirtualRouter | Success | 0.02 | test_vm_life_cycle.py test_10_attachAndDetach_iso | Success | 26.65 | test_vm_life_cycle.py test_09_expunge_vm | Success | 125.14 | test_vm_life_cycle.py test_08_migrate_vm | Success | 70.92 | test_vm_life_cycle.py test_07_restore_vm | Success | 0.06 | test_vm_life_cycle.py test_06_destroy_vm | Success | 10.13 | test_vm_life_cycle.py test_03_reboot_vm | Success | 10.14 | test_vm_life_cycle.py test_02_start_vm | Success | 20.21 | test_vm_life_cycle.py test_01_stop_vm | Success | 30.24 | test_vm_life_cycle.py test_CreateTemplateWithDuplicateName | Success | 141.00 | test_templates.py test_08_list_system_templates | Success | 0.02 | test_templates.py test_07_list_public_templates | Success | 0.03 | test_templates.py test_05_template_permissions | Success | 0.04 | test_templates.py test_03_delete_template | Success | 5.10 | test_templates.py test_02_edit_template | Success | 90.06 | test_templates.py test_10_destroy_cpvm | Success | 226.72 | test_ssvm.py test_09_destroy_ssvm | Success | 199.07 | test_ssvm.py test_08_reboot_cpvm | Success | 146.70 | test_ssvm.py test_07_reboot_ssvm | Success | 144.08 | test_ssvm.py test_06_stop_cpvm | Success | 136.68 | test_ssvm.py test_05_stop_ssvm | Success | 169.02 | test_ssvm.py test_04_cpvm_internals | Success | 1.32 | test_ssvm.py test_03_ssvm_internals | Success | 3.63 | test_ssvm.py test_02_list_cpvm_vm | Success | 0.08 | test_ssvm.py test_01_list_sec_storage_vm | Success | 0.11 | test_ssvm.py test_01_snapshot_root_disk | Success | 21.17 | test_snapshots.py test_04_change_offering_small | Success | 120.96 | test_service_offerings.py test_03_delete_service_offering | Succ
[GitHub] cloudstack pull request #1650: fix macchinina template specificied for vmwar...
Github user PaulAngus closed the pull request at: https://github.com/apache/cloudstack/pull/1650 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack pull request #1688: Marvin Tests: Fix VPC network offering select...
Github user PaulAngus closed the pull request at: https://github.com/apache/cloudstack/pull/1688 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1749: Updates for SAN-assisted snapshots
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1749 I'll have a look, but seems unlikely, given the scope of the changes 'storage' and that this is the only this PR having a problem... + isn't: /var/cloudstack/mnt/VM/6814905206562.ca65d71/template/tmpl/2/214 going to be on the mgmt server? shouldn't the SSVM be doing this kind of stuff? #nondeveloper_sticking_his_oar_in. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1749: Updates for SAN-assisted snapshots
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1749 Storage from above test run has been cleaned up, however: Permissions look fine in general, but i note that mgmt server created files (ie systemvm.iso) are created as user 996 which i assume is 'cloud'.  In general, other templates and templates.properties files look to be created as root with permissions: 0666 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1711: XenServer 7 Support
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1711 I ran above smoke tests overnight - results look very promising. I know that there 'was' an error in the snapshot test, I'll check to see if the failure was a result of that.. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1741 Any failures relating to rVPC maybe can't be completely ignored but the design of the rVPC means that it will not work under XenServer or vSphere, so tests will fail under those hypervisors. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1741 hi @remibergsma , the design puts the same MAC on the two VPC routers. XenServer doesn't seem to like this. (ESXi hosts give a specific warning). @swill have you pulled in the updated marvin smoke tests? We had all green on KVM tests with the updated component test suite --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1741 I ran the failing test suite against 4.9.2 - and everything passed Test Site 2 Site VPN Across redundant VPCs ... === TestName: test_01_redundant_vpc_site2site_vpn | Status : SUCCESS === ok Test Remote Access VPN in VPC ... === TestName: test_01_vpc_remote_access_vpn | Status : SUCCESS === ok Test Site 2 Site VPN Across VPCs ... === TestName: test_01_vpc_site2site_vpn | Status : SUCCESS === ok -- Ran 3 tests in 747.901s OK so the failures appear to be due to the changes in the PR --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] cloudstack issue #1711: XenServer 7 Support
Github user PaulAngus commented on the issue: https://github.com/apache/cloudstack/pull/1711 @ciroiriarte CloudStack orchestrates XenServers through XAPI - so how VLANs are created is transparent to CloudStack. Are you using Basic or Advanced networks ? Advanced networks are the generally the way to go isolate networks through VLANs. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
