]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
pybind/rados: drop change_auid
authorSage Weil <sage@redhat.com>
Sat, 11 Aug 2018 19:00:31 +0000 (14:00 -0500)
committerSage Weil <sage@redhat.com>
Fri, 31 Aug 2018 14:27:36 +0000 (09:27 -0500)
Signed-off-by: Sage Weil <sage@redhat.com>
src/pybind/rados/rados.pyx
src/test/pybind/test_rados.py

index d92148660b4b7d79c30f355c7a84919047cfd941..44149a21c688511ab9f705bc5b42aaa24c2db879 100644 (file)
@@ -201,7 +201,6 @@ cdef extern from "rados/librados.h" nogil:
     int rados_ioctx_create(rados_t cluster, const char *pool_name, rados_ioctx_t *ioctx)
     int rados_ioctx_create2(rados_t cluster, int64_t pool_id, rados_ioctx_t *ioctx)
     void rados_ioctx_destroy(rados_ioctx_t io)
-    int rados_ioctx_pool_set_auid(rados_ioctx_t io, uint64_t auid)
     void rados_ioctx_locator_set_key(rados_ioctx_t io, const char *key)
     void rados_ioctx_set_namespace(rados_ioctx_t io, const char * nspace)
 
@@ -2544,26 +2543,6 @@ cdef class Ioctx(object):
         if self.state != "open":
             raise IoctxStateError("The pool is %s" % self.state)
 
-    def change_auid(self, auid):
-        """
-        Attempt to change an io context's associated auid "owner."
-
-        Requires that you have write permission on both the current and new
-        auid.
-
-        :raises: :class:`Error`
-        """
-        self.require_ioctx_open()
-
-        cdef:
-            uint64_t _auid = auid
-
-        with nogil:
-            ret = rados_ioctx_pool_set_auid(self.io, _auid)
-        if ret < 0:
-            raise make_ex(ret, "error changing auid of '%s' to %d"
-                          % (self.name, auid))
-
     @requires(('loc_key', str_type))
     def set_locator_key(self, loc_key):
         """
index 76ee286f8124179a63c41633b5fa23b206a34b55..522751504f1f2a51487cc8f07ede346b619a868b 100644 (file)
@@ -3,7 +3,7 @@ from nose import SkipTest
 from nose.tools import eq_ as eq, ok_ as ok, assert_raises
 from rados import (Rados, Error, RadosStateError, Object, ObjectExists,
                    ObjectNotFound, ObjectBusy, requires, opt,
-                   ANONYMOUS_AUID, ADMIN_AUID, LIBRADOS_ALL_NSPACES, WriteOpCtx, ReadOpCtx,
+                   LIBRADOS_ALL_NSPACES, WriteOpCtx, ReadOpCtx,
                    LIBRADOS_SNAP_HEAD, LIBRADOS_OPERATION_BALANCE_READS, LIBRADOS_OPERATION_SKIPRWLOCKS, MonitorLog)
 import time
 import threading
@@ -179,11 +179,6 @@ class TestRados(object):
         finally:
             self.rados.delete_pool(poolname)
 
-    def test_create_auid(self):
-        self.rados.create_pool('foo', 100)
-        assert self.rados.pool_exists('foo')
-        self.rados.delete_pool('foo')
-
     def test_eexist(self):
         self.rados.create_pool('foo')
         assert_raises(ObjectExists, self.rados.create_pool, 'foo')
@@ -310,10 +305,6 @@ class TestIoctx(object):
                    'num_objects_degraded': 0,
                    'num_rd': 0})
 
-    def test_change_auid(self):
-        self.ioctx.change_auid(ANONYMOUS_AUID)
-        self.ioctx.change_auid(ADMIN_AUID)
-
     def test_write(self):
         self.ioctx.write('abc', b'abc')
         eq(self.ioctx.read('abc'), b'abc')
@@ -971,10 +962,6 @@ class TestIoctx2(object):
                    'num_objects_degraded': 0,
                    'num_rd': 0})
 
-    def test_change_auid(self):
-        self.ioctx2.change_auid(ANONYMOUS_AUID)
-        self.ioctx2.change_auid(ADMIN_AUID)
-
 
 class TestObject(object):