[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-9057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15000413#comment-15000413
 ] 

ASF GitHub Bot commented on CLOUDSTACK-9057:
--------------------------------------------

GitHub user DaanHoogland opened a pull request:

    https://github.com/apache/cloudstack/pull/1061

    CLOUDSTACK-9057 remove old system vm upgrade code

    the upgrade step to 4.5 system templates is removed for 4.6 in this. 
validation would be to install any 4.4.2 or lower version and upgrade to 4.6. 
It would only require the templates for 4.6 and not the ones for 4.5. as it 
would before this patch. only unittests performed hence for all practical 
purposes untested.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/DaanHoogland/cloudstack CLOUDSTACK-9057

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/1061.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1061
    
----
commit d8e7e7d99355dcb79aec1e16ac94ae546a8bed41
Author: Daan Hoogland <[email protected]>
Date:   2015-11-11T14:02:28Z

    CLOUDSTACK-9057 remove old system vm upgrade code

----


> upgrade to 4.6 requires 4.5 templates
> -------------------------------------
>
>                 Key: CLOUDSTACK-9057
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9057
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>    Affects Versions: 4.4.2
>            Reporter: Daan Hoogland
>            Assignee: Daan Hoogland
>            Priority: Minor
>
> the upgrade coe requiring 4.5 system templates can be removed.
> There is a ticket somewhere to make the upgrade code more generic. for this 
> the maps in the method(s) have to be moved to a central location. The update 
> call can be an API using those maps. this for prosperity, now remove!



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to