rpc hang sometimes after oslo try to reconnect to the servers

Bug #1288844 reported by Mehdi Abaakouk
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Critical
Mehdi Abaakouk

Bug Description

Hi,

When we use ceilometer with kombu (rabbitmq), sometimes when the server gone and ceilometer try to reconnect to it, the connection can hang.

This is because, ceilometer monkeypatch only socket and not select, or when kombu detect monkeypatching on the socket module, it assume the select is monkeypatched too and use the real select.select on a eventlet socket, and hang.

Ceilometer must enable monkeypatch for select too.

Cheers

Mehdi Abaakouk (sileht)
Changed in ceilometer:
assignee: nobody → Mehdi Abaakouk (sileht)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (master)

Fix proposed to branch: master
Review: https://review.openstack.org/78654

Changed in ceilometer:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/78654
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=10300af4f55b179f7aea909a8d752ed4ac4c2d70
Submitter: Jenkins
Branch: master

commit 10300af4f55b179f7aea909a8d752ed4ac4c2d70
Author: Mehdi Abaakouk <email address hidden>
Date: Thu Mar 6 17:28:11 2014 +0100

    Enable monkeypatch for select module

    When we use ceilometer with kombu (rabbitmq), sometimes when the server
    is gone and ceilometer tries to reconnect to it, the connection can hang.

    This is because, ceilometer monkeypatch only socket and not select, or
    when kombu detect monkeypatching on the socket module, it assume the
    select is monkeypatched too and use the real select.select on a eventlet
    socket, and hang.

    Ceilometer must enable monkeypatch for select too.

    Change-Id: Ib41dcde6e5f32755ababf9227827aa8368399c9a
    Closes-bug: 1288844

Changed in ceilometer:
status: In Progress → Fix Committed
Mark McLoughlin (markmc)
Changed in ceilometer:
importance: Undecided → Critical
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: none → icehouse-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: icehouse-rc1 → 2014.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.