actually try to getattr not just check if it exists, defaulting to None

Signed-off-by: Alfredo Deza <adeza@redhat.com>

diff --git a/radosgw_agent/client.py b/radosgw_agent/client.py
index c27dd03980edf89cb6bcd082568ec41cd6edc168..b05d2d3c217169520155f21de160dc27a5a74cab 100644 (file)
--- a/radosgw_agent/client.py
+++ b/radosgw_agent/client.py
@@ -278,7 +278,7 @@ def is_versioned(obj):
     if not hasattr(obj, 'VersionedEpoch'):
         # overly paranoid here, an object that is not versioned should *never*
         # have a `VersionedEpoch` attribute
-        if hasattr(obj, 'version_id'):
+        if getattr(obj, 'version_id', None):
             if obj.version_id is None:
                 return False
             return True  # probably will never get here