Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cloudstack.apache.org
From: rashmidixit <git@git.apache.org>
To: dev@cloudstack.apache.org
Reply-To: dev@cloudstack.apache.org
References: <git-pr-1889-cloudstack@git.apache.org>
In-Reply-To: <git-pr-1889-cloudstack@git.apache.org>
Subject: [GitHub] cloudstack pull request #1889: CLOUDSTACK-9718: Revamp the dropdown showing ...
Content-Type: text/plain
Message-Id: <20170406123148.6A7E2DFC31@git1-us-west.apache.org>
Date: Thu,  6 Apr 2017 12:31:48 +0000 (UTC)
archived-at: Thu, 06 Apr 2017 12:31:50 -0000

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