cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From rashmidixit <...@git.apache.org>
Subject [GitHub] cloudstack pull request #1889: CLOUDSTACK-9718: Revamp the dropdown showing ...
Date Fri, 17 Mar 2017 03:24:19 GMT
GitHub user rashmidixit reopened a pull request:

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

    CLOUDSTACK-9718: Revamp the dropdown showing lists of hosts available for migration in
a Zone

    Refer to [CLOUDSTACK-9718](https://issues.apache.org/jira/browse/CLOUDSTACK-9718) for
more details.
    
    Problem: All the hosts suitable for VM Migration are not shown in the UI. This could
    confuse the user as the target host might never be shown in the UI.
    
    Root Cause: The API (findHostsForMigration) always returned page 1 results which would
    be always <= default.page.size global parameter. Therefore, in case of large
    no. of hosts where the result can map to multiple pages, this issue would arise.
    
    Solution: 1. Replace drop-down with listView widget.
    2. Allow lazy-loading of records on listView's scroll.
    3. Show additional parameters (CPU/Memory used) to assist admin in decision making.
    4. Provide 'Search by host name' to limit the results.
    
    Conflicts:
    	client/WEB-INF/classes/resources/messages.properties
    	client/WEB-INF/classes/resources/messages_ja_JP.properties
    	client/WEB-INF/classes/resources/messages_zh_CN.properties
    	ui/css/cloudstack3.css
    	ui/dictionary2.jsp

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

    $ git pull https://github.com/Accelerite/cloudstack CLOUDSTACK-9718

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

    https://github.com/apache/cloudstack/pull/1889.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 #1889
    
----
commit e64306de19a011f7fef3d3a92d7d2bbc6792e215
Author: rashmidixit <rashmi_dixit@persistent.co.in>
Date:   2017-03-16T10:35:22Z

     CLOUDSTACK-9718: Revamp the dropdown showing lists of hosts available for migration in
a Zone
    
        Reviewed-By: Rashmi Dixit
    
        Problem: All the hosts suitable for VM Migration are not shown in the UI. This could
        confuse the user as the target host might never be shown in the UI.
    
        Root Cause: The API (findHostsForMigration) always returned page 1 results which would
        be always <= default.page.size global parameter. Therefore, in case of large
        no. of hosts where the result can map to multiple pages, this issue would arise.
    
        Solution: 1. Replace drop-down with listView widget.
        2. Allow lazy-loading of records on listView's scroll.
        3. Show additional parameters (CPU/Memory used) to assist admin in decision making.
        4. Provide 'Search by host name' to limit the results.
    
    Added change where if there are no hosts found, an empty row with message will
    appear.

----


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message