[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-11-07 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @PaulAngus @jburwell I've added more changes and rebased. now My tests are now running without errors. I did add the ` --config global{metadata_read_only=0} ` as suggested by @PaulAngus and did not

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-28 Thread ArmedGuy
Github user ArmedGuy commented on the issue: https://github.com/apache/cloudstack/pull/1711 @PaulAngus --config is not persistent, but helpfully overrides metadata_read_only = 1 which is set in /etc/lvm/lvm.conf on XS7. But I second that `--config global{metadata_read_only=0}` is need

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-27 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @jburwell I have reviewed all the smoke tests and I have not found at any place we skipping because of a version mismatch so we are good there. I still have to figure out why some of the tests are

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-26 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed +1 -- I would like to see a full component test run for this PR on XS 7 and smoke tests for XS 6.5. Also, we might want to consider a PR for 4.9 and then another PR for master like we di

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-26 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @jburwell Got it. I want to stabilize the tests and once everything looks green here. I will open a PR against 4.9 --- If your project is set up for it, you can reply to this email and have your r

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-26 Thread jburwell
Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed given that XenServer 6.5 doesn't run on newer hardware, XenServer users don't have a realistic LTS release. Therefore, I propose that this PR be targeted to the 4.9 branch and forward me

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-26 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @rhtyd @PaulAngus I will verify the failed tests (except VPC and private gateway) and update this. It would be nice if we can get this into 4.10 --- If your project is set up for it, you can repl

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 Trillian test result (tid-192) Environment: CUSTOM-XenServer (x2), Advanced Networking with Mgmt server 6 Total time taken: 35929 seconds Marvin logs: https://github.com/blueora

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-26 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 Trillian test result (tid-183) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 6 Total time taken: 32182 seconds Marvin logs: https://github.com/blueoran

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-25 Thread koushik-das
Github user koushik-das commented on the issue: https://github.com/apache/cloudstack/pull/1711 One small comment, rest of the code LGTM. --- 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 f

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-25 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 @rhtyd a Trillian-Jenkins matrix job (centos6 mgmt + xs65sp1, centos7 mgmt + vmware55u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests --- If your project is set up for it

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-25 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1711 @blueorangutan test matrix --- 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 wi

RE: [GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-25 Thread Paul Angus
: dev@cloudstack.apache.org Subject: [GitHub] cloudstack issue #1711: Xenserver7 Support Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @PaulAngus would it be possible to run another round of tests on this? --- If your project is set up for it

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-25 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @PaulAngus would it be possible to run another round of tests on 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 proje

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-20 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-69 --- 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 do

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-20 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-20 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1711 @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

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-20 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @PaulAngus `test_volumes:TestCreateVolume.test_01_create_volume` I've tested the failed checks. I found a problem with the marvin check that was doing size check. I've modified it

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-20 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @PaulAngus I'm running the tests that failed for you on my setup. I will update with what I find. --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-19 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 #1711: Xenserver7 Support

2016-10-19 Thread PaulAngus
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 **clo

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-18 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @koushik-das Thank you for the review! I've added the guest OS mappings and hypervisor capabilities. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-17 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-63 --- 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 do

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-17 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1711 There was an env issue with the debian package job, it has been re-fired. --- 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 pr

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-17 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 Packaging result: ✔centos6 ✔centos7 ✖debian. JID-62 --- 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 do

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-17 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 @rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your reply appe

[GitHub] cloudstack issue #1711: Xenserver7 Support

2016-10-17 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1711 Thanks @syed for your PR. I'll kick some tests following merge of PR #1708. @PaulAngus can add support for XenServer 7 in Trillian, thanks. @blueorangutan package --- If your project