Actions
Bug #3434
closedUnknown variables in test_xattr_support
% Done:
0%
Source:
Tags:
Backport:
Regression:
Severity:
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):
Description
From https://github.com/ceph/obsync/blob/master/obsync#L157
When catching IOError, there are references to pa
and exc
, I think these are supposed to be path
and e
, respectively. It doesn't work, currently.
###### Extended Attributes ####### def test_xattr_support(path): test_file = path + "/$TEST" f = open(test_file, 'w') f.close try: xattr.set(test_file, "test", "123", namespace=xattr.NS_USER) if xattr.get(test_file, "test", namespace=xattr.NS_USER) != "123": raise ObsyncPermanentException("test_xattr_support: failed to set an xattr and " + \ "read it back.") except IOError, e: print >>stderr, "**** ERRROR: You do not appear to have xattr support " + \ "at %s ****" % pa raise ObsyncPermanentException(exc) finally: os.unlink(test_file)
Updated by Greg Farnum about 11 years ago
- Assignee set to Ian Colle
Giving to Ian for proper distribution. :)
Updated by Ian Colle about 11 years ago
- Assignee changed from Ian Colle to Matthew Wodrich
Updated by Steve Hoeksema over 10 years ago
I'm curious why this is set to Won't Fix - the bug still exists in master.
Updated by Matthew Wodrich over 10 years ago
I was unaware this was assigned to me on this tracker. The good news is that this has been fixed, I believe, in the repo with continuing development (on https://github.com/dreamhost/obsync). I seem to recall running into this issue and patching it. Try the current version on master there.
Actions