cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From rhtyd <...@git.apache.org>
Subject [GitHub] cloudstack pull request: Fix Nio/CPU issue and CI failures
Date Fri, 13 May 2016 06:27:43 GMT
GitHub user rhtyd reopened a pull request:

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

    Fix Nio/CPU issue and CI failures

    - Reverts ea2286 that introduced a wakeup on each connection loop run.
    - In SSL handshake code removes delegated tasks to be run in separate threads.
    
    /cc @kiwiflyer @swill @jburwell and others for review
    
    @kiwiflyer please help me test this fix and share if it makes the NioConnection robust
now, without having the selector consume a lot of CPU. Thanks.

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

    $ git pull https://github.com/shapeblue/cloudstack nio-fix-aggressive-cpu-use

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

    https://github.com/apache/cloudstack/pull/1543.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 #1543
    
----
commit 540d9572fd491db3ce182d26636fc74ada4e171c
Author: Rohit Yadav <rohit.yadav@shapeblue.com>
Date:   2016-05-13T03:24:24Z

    CLOUDSTACK-9348: Make NioConnectio loop less aggressive
    
    - Reverts ea2286 that introduced a wakeup on each connection loop run.
    - In SSL handshake code removes delegated tasks to be run in separate threads.
    
    Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>

commit ae0f169123158c23898316a32b413f4647d0c456
Author: Rohit Yadav <rohit.yadav@shapeblue.com>
Date:   2016-05-13T03:39:58Z

    CLOUDSTACK-9299: Fix test failures on CI
    
    - Fixes oobm integration test to skip if known ipmitool bug is hit
    - Fixes ProcessTest unit test case to use sleep
    - Removes redundant unit test that covers code in ProcessTest
    
    Signed-off-by: Rohit Yadav <rohit.yadav@shapeblue.com>

----


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