[Gluster-devel] Looping import in Swift w/cae3216a

Pete Zaitcev zaitcev at redhat.com
Tue Nov 6 19:16:09 UTC 2012


Dear Peter:

I pulled the latest Gluster tip today from git, and it containes
a commit cae3216a, "object-storage: use constants for directory names".
In there, we have this:

+++ b/swift/1.4.8/plugins/utils.py
@@ -19,6 +19,7 @@ import errno
 import xattr
 from hashlib import md5
 from swift.common.utils import normalize_timestamp, TRUE_VALUES
+from swift.obj.server import ASYNCDIR
 import cPickle as pickle
.............

But the existing swift.diff adds an opposing import, like so:

+++ b/swift/obj/server.py
@@ -45,6 +46,10 @@ from swift.common.exceptions import ConnectionTimeout, DiskFileError, \
     DiskFileNotExist
 from swift.obj.replicator import tpooled_get_hashes, invalidate_hash, \
     quarantine_renamer
+from swift.plugins.utils import plugin_enabled
+if plugin_enabled():
.............

The result is that "from swift.obj.server import ASYNCDIR" fails.

I suspect that something got inconsistent, for example maybe you have
moved plugin_enabled() into common in your tree.

Greetings,
-- Pete

cc: to signed-off




More information about the Gluster-devel mailing list