gluster-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


From: Peter Portante
Subject: Re: [Gluster-devel] Looping import in Swift w/cae3216a
Date: Tue, 6 Nov 2012 14:50:09 -0500 (EST)

Thanks for noticing, Pete.

Unfortunately, I have a number of commits backed up behind the review of the 
existing one which will address this problem that you have identified below 
(which we missed in the review due to lack of testing on my part ... bugger 
all).

Sorry,

-peter


----- Original Message -----
> From: "Kaleb Keithley" <address@hidden>
> To: "Pete Zaitcev" <address@hidden>
> Cc: "Peter Portante" <address@hidden>, address@hidden
> Sent: Tuesday, November 6, 2012 2:43:51 PM
> Subject: Re: [Gluster-devel] Looping import in Swift w/cae3216a
> 
> The swift tree is in transition while we review and merge bite-sized
> pieces into the source.
> 
> (And the swift.diff file is eventually going away.)
> 
> ----- Original Message -----
> From: "Pete Zaitcev" <address@hidden>
> To: "Peter Portante" <address@hidden>
> Cc: address@hidden, "Kaleb KEITHLEY" <address@hidden>,
> "Vijay Bellur" <address@hidden>
> Sent: Tuesday, November 6, 2012 2:16:09 PM
> Subject: Looping import in Swift w/cae3216a
> 
> 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
> 
> _______________________________________________
> Gluster-devel mailing list
> address@hidden
> https://lists.nongnu.org/mailman/listinfo/gluster-devel
> 



reply via email to

[Prev in Thread] Current Thread [Next in Thread]