Project

General

Profile

Actions

Bug #3434

closed

Unknown variables in test_xattr_support

Added by Steve Hoeksema over 11 years ago. Updated over 10 years ago.

Status:
Won't Fix
Priority:
Normal
Category:
obsync
Target version:
-
% 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)
Actions #1

Updated by Greg Farnum about 11 years ago

  • Assignee set to Ian Colle

Giving to Ian for proper distribution. :)

Actions #2

Updated by Ian Colle about 11 years ago

  • Assignee changed from Ian Colle to Matthew Wodrich
Actions #3

Updated by Sage Weil over 10 years ago

  • Status changed from New to Won't Fix
Actions #4

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.

Actions #5

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

Also available in: Atom PDF