[GitHub] cloudstack issue #2026: CLOUDSTACK-9861: Expire VM snapshots after configure...

2017-04-10 Thread PaulAngus
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

[GitHub] cloudstack issue #1813: CLOUDSTACK-9604: Root disk resize support for VMware...

2017-03-09 Thread PaulAngus
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

[GitHub] cloudstack issue #1878: CLOUDSTACK-9717: [VMware] RVRs have mismatching MAC ...

2017-03-09 Thread PaulAngus
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

[GitHub] cloudstack issue #1990: CLOUDSTACK-9674: Baremetal deployment improvements

2017-03-09 Thread PaulAngus
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

[GitHub] cloudstack issue #1980: CLOUDSTACK-9805: Display VR list in network details

2017-03-01 Thread PaulAngus
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

[GitHub] cloudstack issue #1711: XenServer 7 Support

2017-01-28 Thread PaulAngus
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

[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2017-01-25 Thread PaulAngus
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

[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2017-01-20 Thread PaulAngus
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

[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2017-01-20 Thread PaulAngus
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

[GitHub] cloudstack issue #1711: XenServer 7 Support

2016-11-22 Thread PaulAngus
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

[GitHub] cloudstack issue #1749: Updates for SAN-assisted snapshots

2016-11-09 Thread PaulAngus
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

[GitHub] cloudstack issue #1749: Updates for SAN-assisted snapshots

2016-11-09 Thread PaulAngus
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

[GitHub] cloudstack pull request #1650: fix macchinina template specificied for vmwar...

2016-10-26 Thread PaulAngus
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

[GitHub] cloudstack pull request #1688: Marvin Tests: Fix VPC network offering select...

2016-10-26 Thread PaulAngus
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

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-20 Thread PaulAngus
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

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-10-05 Thread PaulAngus
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

[GitHub] cloudstack pull request #:

2016-10-04 Thread PaulAngus
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

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-10-03 Thread PaulAngus
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

[GitHub] cloudstack issue #1692: Fix Smoke Test Failures

2016-09-30 Thread PaulAngus
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

[GitHub] cloudstack issue #1684: CLOUDSTACK-9489: the new config vars that are added ...

2016-09-29 Thread PaulAngus
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

[GitHub] cloudstack pull request #1683: fix ping tests to properly recognise successf...

2016-09-28 Thread PaulAngus
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

[GitHub] cloudstack pull request #1652: Marvin Tests: Fix VPC network offering select...

2016-09-28 Thread PaulAngus
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

[GitHub] cloudstack pull request #1692: fix ping tests to properly recognise successf...

2016-09-26 Thread PaulAngus
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

[GitHub] cloudstack pull request #1691: fix ping tests to properly recognise successf...

2016-09-26 Thread PaulAngus
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

[GitHub] cloudstack pull request #1691: fix ping tests to properly recognise successf...

2016-09-26 Thread PaulAngus
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

[GitHub] cloudstack pull request #1688: Marvin Tests: Fix VPC network offering select...

2016-09-23 Thread PaulAngus
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

[GitHub] cloudstack pull request #1687: Marvintests vpcofferingfix

2016-09-23 Thread PaulAngus
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

[GitHub] cloudstack pull request #1687: Marvintests vpcofferingfix

2016-09-23 Thread PaulAngus
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

[GitHub] cloudstack pull request #1683: fix ping tests to properly recognise successf...

2016-09-21 Thread PaulAngus
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

[GitHub] cloudstack issue #1651: Marvin Tests: fix expected return string for success...

2016-09-21 Thread PaulAngus
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

[GitHub] cloudstack pull request #1651: Marvin Tests: fix expected return string for ...

2016-09-20 Thread PaulAngus
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

[GitHub] cloudstack issue #1652: Marvin Tests: Fix VPC network offering selection .

2016-08-22 Thread PaulAngus
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

[GitHub] cloudstack issue #1651: Marvin Tests: fix expected return string for success...

2016-08-22 Thread PaulAngus
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

[GitHub] cloudstack issue #1651: Marvin Tests: fix expected return string for success...

2016-08-22 Thread PaulAngus
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

[GitHub] cloudstack issue #1652: Marvin Tests: Fix VPC network offering selection .

2016-08-22 Thread PaulAngus
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

[GitHub] cloudstack issue #1641: CLOUDSTACK-9459: the try's catch block was shortenin...

2016-08-22 Thread PaulAngus
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

[GitHub] cloudstack pull request #1652: Marvin Tests: Fix VPC network offering select...

2016-08-19 Thread PaulAngus
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

[GitHub] cloudstack pull request #1651: Marvin Tests: fix expected return string for ...

2016-08-19 Thread PaulAngus
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

[GitHub] cloudstack pull request #1650: fix macchinina template specificied for vmwar...

2016-08-19 Thread PaulAngus
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

[GitHub] cloudstack issue #1649: CLOUDSTACK-9463: Fix dynamic-roles migrate script fo...

2016-08-19 Thread PaulAngus
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

[GitHub] cloudstack issue #1612: packaging: Marvin and integration-tests packages

2016-07-20 Thread PaulAngus
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

[GitHub] cloudstack issue #1612: packaging: Marvin and integration-tests packages

2016-07-20 Thread PaulAngus
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

[GitHub] cloudstack pull request: Remove extraenous log dir and add catalin...

2016-05-14 Thread PaulAngus
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

[GitHub] cloudstack pull request: Remove extraneous log directory and add c...

2016-05-13 Thread PaulAngus
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

[GitHub] cloudstack pull request: Remove extraneous log directory and add c...

2016-05-13 Thread PaulAngus
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

[GitHub] cloudstack pull request: Remove extraneous log directory and add c...

2016-05-13 Thread PaulAngus
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

[GitHub] cloudstack pull request: Remove extraneous log directory and add c...

2016-05-13 Thread PaulAngus
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

[GitHub] cloudstack pull request: Remove extraneous log directory and add c...

2016-05-13 Thread PaulAngus
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

[GitHub] cloudstack pull request: Remove extraneous log directory and add c...

2016-05-13 Thread PaulAngus
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

[GitHub] cloudstack pull request: Remove extraneous log directory and add c...

2016-05-12 Thread PaulAngus
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

[GitHub] cloudstack pull request: Remove extraneous log directory and add c...

2016-05-12 Thread PaulAngus
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

[GitHub] cloudstack pull request: Remove extraneous log directory and add c...

2016-05-10 Thread PaulAngus
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

[GitHub] cloudstack pull request: [4.7] VMware boot order fix

2016-02-05 Thread PaulAngus
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

[GitHub] cloudstack pull request: Remove extraenous log dir and add catalin...

2015-12-03 Thread PaulAngus
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

[GitHub] cloudstack pull request: Remove extraenous log dir and add catalin...

2015-11-18 Thread PaulAngus
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

[GitHub] cloudstack pull request: Remove extraenous log dir and add catalin...

2015-11-18 Thread PaulAngus
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

[GitHub] cloudstack-docs-install pull request: Storage Setup Documentation ...

2015-04-09 Thread PaulAngus
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

[GitHub] cloudstack-docs-install pull request: Storage section of install d...

2015-04-09 Thread PaulAngus
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

[GitHub] cloudstack-docs-install pull request: Storage Setup Documentation ...

2015-04-08 Thread PaulAngus
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

[GitHub] cloudstack-docs-install pull request: Storage Setup Documentation ...

2015-04-08 Thread PaulAngus
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

[GitHub] cloudstack-docs-install pull request: Updates to the vSphere secti...

2015-03-30 Thread PaulAngus
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

[GitHub] cloudstack-docs-install pull request: Updates to the vSphere secti...

2015-03-27 Thread PaulAngus
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