rgw: policy acl format should be xml

Signed-off-by: Orit Wasserman <owasserm@redhat.com>

diff --git a/tasks/radosgw_admin.py b/tasks/radosgw_admin.py
index 7cea724ae23f8755c9ecb7c139d1f883671987cd..142b8f22385d12cd160d12023f15ad430b66e06b 100644 (file)
--- a/tasks/radosgw_admin.py
+++ b/tasks/radosgw_admin.py
@@ -931,7 +931,7 @@ def task(ctx, config):
 
     (err, out) = rgwadmin(ctx, client,
         ['policy', '--bucket', bucket.name, '--object', key.key],
-        check_status=True)
+        check_status=True, format='xml')
 
     acl = get_acl(key)
 
@@ -942,7 +942,7 @@ def task(ctx, config):
 
     (err, out) = rgwadmin(ctx, client,
         ['policy', '--bucket', bucket.name, '--object', key.key],
-        check_status=True)
+        check_status=True, format='xml')
 
     acl = get_acl(key)
 

diff --git a/tasks/util/rgw.py b/tasks/util/rgw.py
index e5fba9f82f9a96f11d1483d9eddc6708bd6ef129..cf06443ef34292cb9e72f837d0d14f642f694e6c 100644 (file)
--- a/tasks/util/rgw.py
+++ b/tasks/util/rgw.py
@@ -15,7 +15,8 @@ def multi_region_enabled(ctx):
     # use that as an indicator that we're testing multi-region sync
     return 'radosgw_agent' in ctx
 
-def rgwadmin(ctx, client, cmd, stdin=StringIO(), check_status=False):
+def rgwadmin(ctx, client, cmd, stdin=StringIO(), check_status=False,
+             format='json'):
     log.info('rgwadmin: {client} : {cmd}'.format(client=client,cmd=cmd))
     testdir = teuthology.get_testdir(ctx)
     pre = [
@@ -24,7 +25,7 @@ def rgwadmin(ctx, client, cmd, stdin=StringIO(), check_status=False):
         '{tdir}/archive/coverage'.format(tdir=testdir),
         'radosgw-admin'.format(tdir=testdir),
         '--log-to-stderr',
-        '--format', 'json',
+        '--format', format,
         '-n',  client,
         ]
     pre.extend(cmd)