[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @ciroiriarte As @PaulAngus pointed out, you need to use VLANs as the isolation method if you want to use OVS on XenServer --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user ciroiriarte commented on the issue: https://github.com/apache/cloudstack/pull/1711 Well, I chose Security Groups on the Advanced network configuration options, that makes the bridge firewall configuration fail when you try to add hosts. Unchecked that while

[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

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user ciroiriarte commented on the issue: https://github.com/apache/cloudstack/pull/1711 Also, is there any roadmap to add OVS support?, it's the current standard on XS7 and seems I'll have to go back to Linux bridge if I want to move to Cloudstack. I'm moving away of

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user ciroiriarte commented on the issue: https://github.com/apache/cloudstack/pull/1711 Are these messages expected? 2017-01-29 00:02:24,476 WARN [c.c.a.d.ParamGenericValidationWorker] (catalina-exec-10:ctx-6776e3dd ctx-a85fb6b1) (logid:a578d2b0) **Received unknown

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1711 Yes it is in 4.9.2. --- 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: XenServer 7 Support

Github user ciroiriarte commented on the issue: https://github.com/apache/cloudstack/pull/1711 Hi!, is this already included in 4.9.2.0?, I'm installing a new XS7 lab and would like to integrate it with Cloudstack. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1711 Checked tests, the failure was due to missing licence in sql file. I've fixed that and a conflict as upgrade paths were missing, simply adding a sql file does not add it to the upgrade path. I'll

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed yes I see them now. I'll first merge #1851 to get the db upgrade paths and then apply your PR and include your sql changes. --- If your project is set up for it, you can reply to this

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @rhtyd the file is there ... somehow doesn't show in the diff you have to manually load it --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @rhtyd moved the DB changes to `schema-4910to4920.sql` and rebased. --- 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

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed can you move the sql changes to a new path, setup/db/db/schema-4910to4920.sql and make appropriate db changes? Thanks. After this I'm willing to merge this PR. --- If your project is set

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1711 LGTM on test. Some of the failing tests are known intermittent issues or fixed in a separate PR. Merging this now. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 Trillian test result (tid-694) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7 Total time taken: 35068 seconds Marvin logs:

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 @rhtyd a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1711 @blueorangutan test centos7 xenserver-65sp1 --- 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

[GitHub] cloudstack issue #1711: XenServer 7 Support

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

[GitHub] cloudstack issue #1711: XenServer 7 Support

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

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1711 @koushik-das we can ignore that, Trillian's env does not support iscsi mount point for xenserver (from my last memories). @syed I looked at the changes and last week's result, they look good

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user koushik-das commented on the issue: https://github.com/apache/cloudstack/pull/1711 Code changes are good. In the last test run I am seeing test_01_primary_storage_iscsi is failing. Do we know the reason for that? --- If your project is set up for it, you can reply to

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 I think we're good on this. We can run another round of testing. --- 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

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1711 @PaulAngus @karuturi @koushik-das @syed are we good on this PR, the changes look good to me and should not cause regressions. We can fire one round of testing. --- If your project is set up for

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 Trillian test result (tid-676) Environment: xenserver-65sp1 (x2), Advanced Networking with Mgmt server 7 Total time taken: 33149 seconds Marvin logs:

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 @karuturi a Trillian-Jenkins test job (centos7 mgmt + xenserver-65sp1) has been kicked to run smoke tests --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1711 @blueorangutan test centos7 xenserver-65sp1 --- 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

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1711 @blueorangutan help --- 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

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1711 @karuturi I understand these words: "help", "hello", "thanks", "package", "test" Test command usage: test [mgmt os] [hypervisor] [additional tests] Mgmt OS options: ['centos6',

Re: [GitHub] cloudstack issue #1711: XenServer 7 Support

consider this for 4.9.2.0. Regards. From: Marty Godsey <ma...@gonsource.com> Sent: 10 December 2016 02:31:52 To: dev@cloudstack.apache.org; dev@cloudstack.apache.org Subject: RE: [GitHub] cloudstack issue #1711: XenServer 7 Support So will XS7 s

[GitHub] cloudstack issue #1711: XenServer 7 Support

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

[GitHub] cloudstack issue #1711: XenServer 7 Support

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

[GitHub] cloudstack issue #1711: XenServer 7 Support

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

[GitHub] cloudstack issue #1711: XenServer 7 Support

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

RE: [GitHub] cloudstack issue #1711: XenServer 7 Support

So will XS7 support be in this next update? Regards, Marty Godsey Principal Engineer nSource Solutions, LLC -Original Message- From: blueorangutan [mailto:g...@git.apache.org] Sent: Friday, December 9, 2016 4:00 PM To: dev@cloudstack.apache.org Subject: [GitHub] cloudstack issue #1711

[GitHub] cloudstack issue #1711: XenServer 7 Support

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

[GitHub] cloudstack issue #1711: XenServer 7 Support

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

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @rhtyd Rebased and sqashed the commits --- 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

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed @karuturi can you help fix the merge conflict, and rebase against latest 4.9. Thanks. Can commits be logically squashed, and have a JIRA id in their commit summary. --- If your project is

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1711 Yes. I think its ready code wise. since the last test run, lot of code changes happened. If we can run one more round of tests, that would be good. regarding squashing, Please

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed @karuturi @PaulAngus can you comment if this is in a good position to make into 4.9 yet? On Monday, we'll cut the first set of RCs, so if we can satisfactory review and test results only

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 Installed 4.9.1 using the new setup and added XenServer 7 hosts. Everything looking good. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 Thanks @karuturi for the PR. I have merged your changes as well. @jburwell @rhtyd I've created a stored procedure to do idempotent inserts. I cleanup the procedure after the inserts are

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1711 managed to create a PR by clicking more buttons.. somehow, it didn't show your fork on search https://github.com/syed/cloudstack/pull/6 --- If your project is set up for it, you can

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed I cannot find your repo to create a PR.  I updated the changes Here

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 I have rebased this PR against 4.9. I have removed the DB changes, I will add a idempotent version of that after testing everything works. @karuturi You will have to rebase and open a PR

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed yes, please rebase it against 4.9 and change PR's base branch to 4.9? Using @karuturi 's suggestion, you can further simplify the changes. --- If your project is set up for it, you can

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @jburwell Agree with you. The problem that I had last time was making an SQL insert is not very elegant but I understand its importance especailly if we want to port it to 4.9 and move forward

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed yes, you will add the changes to `schema-4910to4920` for the `4.9` port. This will cover anyone upgrading from a version >= 4.9.1.0 to 4.9.2.0. I do not plan to automatically forward

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 Thanks Guys for testing this out. I had a question @jburwell You wanted me to port this to 4.9 and then it would be forward merged to 4.10 and master. How would the DB changes be handled? right

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1711 Good progress :smiley: --- 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

[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

[GitHub] cloudstack issue #1711: XenServer 7 Support

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

[GitHub] cloudstack issue #1711: XenServer 7 Support

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

[GitHub] cloudstack issue #1711: XenServer 7 Support

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

[GitHub] cloudstack issue #1711: XenServer 7 Support

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

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 Sounds good @jburwell --- 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: XenServer 7 Support

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed I think this change will merit a separate.PR for the forward merge to master. So, once we get merged to 4.9, you'll open PR against master for the forward merge. In terms of

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 That sounds good John. Ill reopen this against 4.9 On Wed, Nov 16, 2016 at 18:36 John Burwell wrote: > @syed also, could you please

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed also, could you please change the base branch to 4.9? I would like to include this support in LTS as many XenServer users will be stranded on old hardware without it. --- If your

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user jburwell commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed I would like to hold this item for the next release cycle. We have a very few PRs left to get 4.8.2.0, 4.9.1.0, and 4.10.0.0 out the door. It will also give us time to run all of the

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @jburwell I think this is looking good (unless you have any other comments) What can I do to help get this in? --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack issue #1711: XenServer 7 Support

Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1711 @PaulAngus @jburwell This PR is looking good now. Can we re-run the tests with this PR? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as