[GitHub] cloudstack pull request: CLOUDSTACK-8395: vmops plugin should work...

2015-04-21 Thread xenserverarmy
Github user xenserverarmy commented on the pull request: https://github.com/apache/cloudstack/pull/186#issuecomment-94880329 This looks solid, but I'd update line 457 to tick-quote the vmchain as you've done elsewhere --- If your project is set up for it, you can reply to

[GitHub] cloudstack-docs-rn pull request: add note on XenServer: we depend ...

2015-05-23 Thread xenserverarmy
Github user xenserverarmy commented on the pull request: https://github.com/apache/cloudstack-docs-rn/pull/21#issuecomment-104945193 I agree that we should add this to the 4.4 and 4.5 notes, but would suggest an addition relating to the config-timeout. The default has changed with

[GitHub] cloudstack pull request: Fix/citrix resource base packages

2015-05-27 Thread xenserverarmy
Github user xenserverarmy commented on the pull request: https://github.com/apache/cloudstack/pull/317#issuecomment-105936788 One suggestion. In the future we may have pure Xen support, so if possible we should ensure all naming for XenServer is "XenServer" and not Xen.

[GitHub] cloudstack pull request: CLOUDSTACK-8987 call s3xen/swiftxen plugi...

2015-10-26 Thread xenserverarmy
Github user xenserverarmy commented on the pull request: https://github.com/apache/cloudstack/pull/970#issuecomment-151157372 @koushik-das One of the goals of a8212d9ef458dd7ac64b021e6fa33fcf64b3cce0 was to ensure that we have clarity in which hypervisor we support since Xen

[GitHub] cloudstack pull request: Rename xapi plugins for s3 and swift to m...

2015-10-28 Thread xenserverarmy
Github user xenserverarmy commented on the pull request: https://github.com/apache/cloudstack/pull/982#issuecomment-151967711 @remibergsma How did the tests work out? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If