[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 proj

[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, after

[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 have

[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 Gi