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 <[email protected]>
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 <[email protected]>
commit ae0f169123158c23898316a32b413f4647d0c456
Author: Rohit Yadav <[email protected]>
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 <[email protected]>
----
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
