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