[GitHub] cloudstack issue #1928: Update hypervisor capabilities for xenserver 7

2017-02-16 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1928 @syed I'll work on the upgrade path today and include your fix, 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

[GitHub] cloudstack issue #1928: Update hypervisor capabilities for xenserver 7

2017-02-14 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1928 @syed simply moving this to a file is not enough. Please see Upgrade4910to4920.java and the class usage to see how upgrade paths are implemented. We'll need a Upgrade4920to4930 upgrade path,

[GitHub] cloudstack issue #1928: Update hypervisor capabilities for xenserver 7

2017-02-14 Thread syed
Github user syed commented on the issue: https://github.com/apache/cloudstack/pull/1928 Thanks @rhtyd for the review. I've moved it to the schema4920to4930.sql file. @karuturi Can we also push this for 4.10? --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack issue #1928: Update hypervisor capabilities for xenserver 7

2017-02-12 Thread rhtyd
Github user rhtyd commented on the issue: https://github.com/apache/cloudstack/pull/1928 @syed since 4.9.2.0 is already released, please created 4.9.2.0->4.9.3.0 upgrade path and move the changes to the suggested sql file. This needs to go into master/4.10 as well /cc @karuturi

[GitHub] cloudstack issue #1928: Update hypervisor capabilities for xenserver 7

2017-02-03 Thread pdion891
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1928 Please merge it into master as well, it also mean we need to make sure master does have the same file `setup/db/db/schema-4920to4930.sql` too. We will do some testing around xenserver7

[GitHub] cloudstack issue #1928: Update hypervisor capabilities for xenserver 7

2017-02-02 Thread pdion891
Github user pdion891 commented on the issue: https://github.com/apache/cloudstack/pull/1928 I think this should be push into master as well to make sure it will exist in future version. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] cloudstack issue #1711: XenServer 7 Support

2017-01-29 Thread syed
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

2017-01-29 Thread ciroiriarte
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

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 are the

[GitHub] cloudstack issue #1711: XenServer 7 Support

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

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

2017-01-21 Thread serg38
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

2017-01-21 Thread ciroiriarte
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 pull request #1711: XenServer 7 Support

2016-12-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1711 --- 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

[GitHub] cloudstack issue #1711: XenServer 7 Support

2016-12-21 Thread rhtyd
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

2016-12-21 Thread rhtyd
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

2016-12-21 Thread syed
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

2016-12-21 Thread syed
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

2016-12-21 Thread rhtyd
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

2016-12-21 Thread rhtyd
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

2016-12-21 Thread blueorangutan
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

2016-12-20 Thread blueorangutan
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

2016-12-20 Thread rhtyd
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

2016-12-19 Thread blueorangutan
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

2016-12-19 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

[GitHub] cloudstack issue #1711: XenServer 7 Support

2016-12-19 Thread rhtyd
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

2016-12-19 Thread koushik-das
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

2016-12-19 Thread syed
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

2016-12-18 Thread rhtyd
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

2016-12-13 Thread blueorangutan
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

2016-12-13 Thread blueorangutan
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

2016-12-13 Thread karuturi
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

2016-12-13 Thread karuturi
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

2016-12-13 Thread blueorangutan
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

2016-12-09 Thread Rohit Yadav
Hi Marty, The XenServer 7 PR could not be validated/tested to be included for 4.9.1.0/4.10.0.0. I'll cut first set of RCs, depending on if we have the results for this next week, there is a slim chance of including this in 4.9.1.0. Since the PR changes, don't require db changes -- we may

[GitHub] cloudstack issue #1711: XenServer 7 Support

2016-12-09 Thread blueorangutan
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

2016-12-09 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

[GitHub] cloudstack issue #1711: XenServer 7 Support

2016-12-09 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

[GitHub] cloudstack issue #1711: XenServer 7 Support

2016-12-09 Thread blueorangutan
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

2016-12-09 Thread Marty Godsey
: 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

[GitHub] cloudstack issue #1711: XenServer 7 Support

2016-12-09 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

[GitHub] cloudstack issue #1711: XenServer 7 Support

2016-12-09 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

[GitHub] cloudstack issue #1711: XenServer 7 Support

2016-12-09 Thread syed
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

2016-12-08 Thread rhtyd
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

2016-12-08 Thread karuturi
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

2016-12-08 Thread rhtyd
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

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

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

2016-12-04 Thread karuturi
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

2016-12-04 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1711 @syed I cannot find your repo to create a PR. ![image](https://cloud.githubusercontent.com/assets/186833/20875605/10c4f0c8-bae2-11e6-8747-a60c5a300c19.png) I updated the changes Here

[GitHub] cloudstack issue #1711: XenServer 7 Support

2016-12-02 Thread syed
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

2016-12-02 Thread rhtyd
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

2016-12-02 Thread syed
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 pull request #1711: XenServer 7 Support

2016-12-02 Thread syed
Github user syed commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1711#discussion_r90680829 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/discoverer/XcpServerDiscoverer.java --- @@ -410,6 +409,8 @@ else if

[GitHub] cloudstack issue #1711: XenServer 7 Support

2016-11-30 Thread jburwell
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

2016-11-30 Thread syed
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

2016-11-23 Thread rhtyd
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

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 #1711: XenServer 7 Support

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

2016-11-20 Thread blueorangutan
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

2016-11-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

[GitHub] cloudstack issue #1711: XenServer 7 Support

2016-11-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

[GitHub] cloudstack issue #1711: XenServer 7 Support

2016-11-17 Thread syed
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

2016-11-16 Thread jburwell
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

2016-11-16 Thread syed
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

2016-11-16 Thread jburwell
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

2016-11-16 Thread jburwell
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

2016-11-16 Thread syed
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 pull request #1711: XenServer 7 Support

2016-11-16 Thread syed
GitHub user syed reopened a pull request: https://github.com/apache/cloudstack/pull/1711 XenServer 7 Support This PR adds support for XenServer 7. I have manually done the following tests - Create a new cluster with XenServer7 - Add Primary storage: Should create an SR

[GitHub] cloudstack pull request #1711: XenServer 7 Support

2016-11-16 Thread syed
Github user syed closed the pull request at: https://github.com/apache/cloudstack/pull/1711 --- 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

[GitHub] cloudstack pull request #1711: XenServer 7 Support

2016-11-14 Thread syed
Github user syed commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1711#discussion_r87849857 --- Diff: setup/db/db/schema-4910to41000.sql --- @@ -35,4 +35,192 @@ INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, hypervi

[GitHub] cloudstack pull request #1711: XenServer 7 Support

2016-11-14 Thread syed
Github user syed commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1711#discussion_r87848428 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/XenServer700Resource.java --- @@ -0,0 +1,32 @@ +/* + *

[GitHub] cloudstack pull request #1711: XenServer 7 Support

2016-11-14 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1711#discussion_r87837783 --- Diff: tools/marvin/marvin/lib/utils.py --- @@ -489,9 +490,11 @@ def checkVolumeSize(ssh_handle=None, return FAILED

[GitHub] cloudstack pull request #1711: XenServer 7 Support

2016-11-14 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1711#discussion_r87846677 --- Diff: setup/db/db/schema-4910to41000.sql --- @@ -35,4 +35,192 @@ INSERT IGNORE INTO `cloud`.`guest_os_hypervisor` (uuid, hypervisor_type, hypervi

[GitHub] cloudstack pull request #1711: XenServer 7 Support

2016-11-14 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1711#discussion_r87836142 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/XenServer700Resource.java --- @@ -0,0 +1,32 @@ +/* + *

[GitHub] cloudstack pull request #1711: XenServer 7 Support

2016-11-14 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1711#discussion_r87758702 --- Diff: scripts/vm/hypervisor/xenserver/setup_heartbeat_sr.sh --- @@ -72,7 +72,7 @@ else fi rm $link -f fi -

[GitHub] cloudstack pull request #1711: XenServer 7 Support

2016-11-14 Thread ArmedGuy
Github user ArmedGuy commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1711#discussion_r87756104 --- Diff: scripts/vm/hypervisor/xenserver/setup_heartbeat_sr.sh --- @@ -72,7 +72,7 @@ else fi rm $link -f fi -

[GitHub] cloudstack pull request #1711: XenServer 7 Support

2016-11-14 Thread koushik-das
Github user koushik-das commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1711#discussion_r87754997 --- Diff: scripts/vm/hypervisor/xenserver/setup_heartbeat_sr.sh --- @@ -72,7 +72,7 @@ else fi rm $link -f fi -

[GitHub] cloudstack issue #1711: XenServer 7 Support

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

[GitHub] cloudstack pull request #1711: XenServer 7 Support

2016-11-09 Thread syed
GitHub user syed reopened a pull request: https://github.com/apache/cloudstack/pull/1711 XenServer 7 Support This PR adds support for XenServer 7. I have manually done the following tests - Create a new cluster with XenServer7 - Add Primary storage: Should create an SR

[GitHub] cloudstack pull request #1739: Fix xapi pluginlib for XenServer 7

2016-10-27 Thread ArmedGuy
Github user ArmedGuy closed the pull request at: https://github.com/apache/cloudstack/pull/1739 --- 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

[GitHub] cloudstack issue #1739: Fix xapi pluginlib for XenServer 7

2016-10-27 Thread ArmedGuy
Github user ArmedGuy commented on the issue: https://github.com/apache/cloudstack/pull/1739 Ah, my bad then. Thought I had searched for it. --- 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 #1739: Fix xapi pluginlib for XenServer 7

2016-10-27 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1739 PR #1711 has this and bunch of other changes needed for XS7 support. --- 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 pull request #1739: Fix xapi pluginlib for XenServer 7

2016-10-27 Thread ArmedGuy
GitHub user ArmedGuy opened a pull request: https://github.com/apache/cloudstack/pull/1739 Fix xapi pluginlib for XenServer 7 This makes cloudstack_pluginlib.py compatible with XenServer 7 dom0, as (from what I gathered) simplejson was included by default as "json" in th

Re: XenServer 7

2016-10-18 Thread Syed Ahmed
peblue.com > www.shapeblue.com > 53 Chandos Place, Covent Garden, London WC2N 4HSUK > @shapeblue > > > > > -Original Message- > From: Syed Ahmed [mailto:sah...@cloudops.com] > Sent: 17 October 2016 21:04 > To: dev@cloudstack.apache.org > Subject: Re: XenS

RE: XenServer 7

2016-10-18 Thread Paul Angus
@cloudstack.apache.org Subject: Re: XenServer 7 @Raj I'd be more interested in knowing if there are any corner cases that We've missed. The fix seems to be relatively simple but I wonder if there is something that I've missed as I cannot tests most of the scenarios On Mon, Oct 17, 2016 at 4:00 PM

Re: XenServer 7

2016-10-17 Thread Syed Ahmed
gt; Could you submit a pull request to Syed's branch containing where you've >>> got to (or let us know where your work is publicly accessible) so that we >>> can combine work..? >>> >>> >>> >>> Kind regards, >>> >>> Paul

Re: XenServer 7

2016-10-17 Thread Syed Ahmed
gt;> >> >> >> Kind regards, >> >> Paul Angus >> >> paul.an...@shapeblue.com >> www.shapeblue.com >> 53 Chandos Place, Covent Garden, London WC2N 4HSUK >> @shapeblue >> >> >> >> >> -Original Message-

Re: XenServer 7

2016-10-17 Thread Syed Ahmed
paul.an...@shapeblue.com > www.shapeblue.com > 53 Chandos Place, Covent Garden, London WC2N 4HSUK > @shapeblue > > > > > -Original Message- > From: Rajesh Ramchandani [mailto:rajesh.ramchand...@accelerite.com] > Sent: 17 October 2016 18:37 > To: dev@cloudsta

RE: XenServer 7

2016-10-17 Thread Paul Angus
WC2N 4HSUK @shapeblue -Original Message- From: Rajesh Ramchandani [mailto:rajesh.ramchand...@accelerite.com] Sent: 17 October 2016 18:37 To: dev@cloudstack.apache.org Subject: Re: XenServer 7 Syed - we are working with Citrix on XS7 as well, please drop a note if you need help. Raj

Re: XenServer 7

2016-10-17 Thread Rajesh Ramchandani
Syed - we are working with Citrix on XS7 as well, please drop a note if you need help. Raj On Mon, Oct 17, 2016 at 8:30 AM -0700, "Syed Ahmed" <sah...@cloudops.com<mailto:sah...@cloudops.com>> wrote: Hi Guys, I have some free time on my hands and was trying to a

Re: XenServer 7

2016-10-17 Thread Syed Ahmed
t; > Kind regards, > > Paul Angus > > paul.an...@shapeblue.com > www.shapeblue.com > 53 Chandos Place, Covent Garden, London WC2N 4HSUK > @shapeblue > > > > > -Original Message- > From: Syed Ahmed [mailto:sah...@cloudops.com] > Sent: 17 October

RE: XenServer 7

2016-10-17 Thread Paul Angus
Ahmed [mailto:sah...@cloudops.com] Sent: 17 October 2016 16:29 To: dev@cloudstack.apache.org Subject: XenServer 7 Hi Guys, I have some free time on my hands and was trying to add XenServer 7 support. There seems to be only one line change that was required to add it. I am able to create VMs

RE: XenServer 7

2016-10-17 Thread Marty Godsey
That is amazing. I cannot wait to see this hit mainstream. The performance difference in XenServer 7 is considerably different than 6.5. I have my "core services" cluster running 7 and it works nicely. Core services like DNS, AD, and even ACS management servers but not compute nodes

Re: XenServer 7

2016-10-17 Thread Haijiao
So cool, thats the feature lots of people awaiting and makes 4.10 shinning Sent from my mobile On 2016-10-17 23:28 , Syed Ahmed Wrote: Hi Guys, I have some free time on my hands and was trying to add XenServer 7 support. There seems to be only one line change that was required to add it. I

XenServer 7

2016-10-17 Thread Syed Ahmed
Hi Guys, I have some free time on my hands and was trying to add XenServer 7 support. There seems to be only one line change that was required to add it. I am able to create VMs and do basic stuff like NAT/port forwarding. If anyone is interested in trying this out, I have a branch [1] which I

Re: Apache Cloudstack 4.8 and XenServer 7

2016-09-08 Thread Erik Weber
On Fri, Jul 29, 2016 at 7:53 AM, Marty Godsey wrote: > Should there be any issues running XS7 with CS 4.8? > > Most likely, yes. There are some major changes with the OS in XS7 that is likely to break. I think there are some efforts from Accelerite to fix it, but I don't

Re: Apache Cloudstack 4.8 and XenServer 7

2016-08-04 Thread Syed Ahmed
We've discussed this and there are changes in XS7 which might break Cloudstack. if you do end up testing this, let us know. We can try to fix it. -Syed On Fri, Jul 29, 2016 at 1:53 AM, Marty Godsey wrote: > Should there be any issues running XS7 with CS 4.8? > > Regards,

Re: XenServer 7

2016-08-01 Thread Will Stevens
Is anyone working on supporting Xenserver 7 in ACS? Thanks, *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw @CloudOps_ On Thu, Jul 21, 2016 at 1:33 PM, Marty Godsey <ma...@gonsource.com> wrote

Apache Cloudstack 4.8 and XenServer 7

2016-07-28 Thread Marty Godsey
Should there be any issues running XS7 with CS 4.8? Regards, Marty Godsey

  1   2   >