Project

General

Profile

Actions

Bug #4877

closed

ceph-deploy pushy library compatibility issue between python 2.7 and 2.6.6

Added by Anonymous about 11 years ago. Updated about 11 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
Regression:
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

The is in issue with connection close when the going from ceph-deploy running on a system with python 2.7 to a target system running python 2.6.6. Problem was seen during work on Bug #4859:

ubuntu@gary-ubuntu-01:~/test-cd$ ceph-deploy install --dev next burnupi27
curl: (22)Exception in thread Thread-1 (most likely raised during interpreter shutdown):
Traceback (most recent call last):
File "/usr/lib/python2.7/threading.py", line 551, in __bootstrap_inner
File "/usr/lib/python2.7/threading.py", line 504, in run
File "/usr/local/lib/python2.7/dist-packages/pushy/protocol/baseconnection.py", line 272, in serve_forever
<type 'exceptions.AttributeError'>: 'NoneType' object has no attribute 'util'

Actions #1

Updated by Dan Mick about 11 years ago

The opinion is that this is after the work is done, so it just an annoyance, and perhaps we can cleanly close the pushy connection (since that seems to be missing). Working on that now.

Actions #2

Updated by Dan Mick about 11 years ago

  • Status changed from New to 12
Actions #3

Updated by Dan Mick about 11 years ago

  • Status changed from 12 to In Progress
Actions #4

Updated by Dan Mick about 11 years ago

  • Assignee set to Dan Mick
Actions #5

Updated by Anonymous about 11 years ago

I pushed a wip-4862 ceph-deploy branch. This puts a pushy close at the end of each connection. There is also an update to the local sudo transport hack that I did. This creates a subclass instead of just forcing an override of the init.

This solved the one case I was able to reproduce reliably.

Actions #6

Updated by Dan Mick about 11 years ago

  • Assignee changed from Dan Mick to Anonymous

I think you've fixed this now, right, Gary?

Actions #7

Updated by Anonymous about 11 years ago

Resolved with the following commit:

commit 51dfee667fbce36bb888dc8af75243941f33e2b9
Author: Gary Lowell <>
Date: Fri May 3 09:28:50 2013 -0700

ceph-deploy: Call pushy close when done with connection.

Signed-off-by: Gary Lowell <>

Actions #8

Updated by Anonymous about 11 years ago

  • Status changed from In Progress to Resolved
Actions

Also available in: Atom PDF