* refs/pull/36987/head:
mgr: PyModuleRegistry::unregister_client() can run endlessly
Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
Reviewed-by: Kefu Chai <kchai@redhat.com>
Since the v1.4.0 release there have been a few improvements to Zstandard
including improved compression ratios, faster compression, and faster
decompression.
Signed-off-by: Bryan Stillwell <bstillwell@godaddy.com>
qa/tests: running rados with -p 80 to produce better test results on …
Reviewed-by: Josh Durgin <jdurgin@redhat.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
qa/cephadm: Add iSCSI
Reviewed-by: Adam King <adking@redhat.com>
Reviewed-by: Georgios Kyratsas <gkyratsas@suse.com>
Reviewed-by: Matthew Oliver <moliver@suse.com>
Reviewed-by: Michael Fritch <mfritch@suse.com>
the data read from stdin is used as the input parameter for calls like
`cluster.osd_command(...)` and `cluster.mon_command`. all of them
expect a bytes `inbuf`. in Python2, this sys.stdin.read() returns a str,
and we don't differentiate str from byte back then. but we need enforce
the type now for better readablity and type correctness.
Signed-off-by: Kefu Chai <kchai@redhat.com>
monitors' names are not numbers, they are referenced by names in
general. their ranks are numbers though, but that's different story.
Signed-off-by: Kefu Chai <kchai@redhat.com>
for silencing warnings like:
WARNING: Cannot resolve forward reference in type annotations of "rados.Rados.conf_get": name 'unicode' is not defined
because cython < 3.0 with language_level = "3", translates "str" to "unicode"
to be python2 compatible, but we've migrated to python3. and the specified
"language_level" is "3'. see also
https://github.com/cython/cython/issues/1370
Signed-off-by: Kefu Chai <kchai@redhat.com>
* use PEP484 style typo annotations
* drop homebrew `requires()` decorator
* drop tests for `requires()` decorator
* add test for type annotations
Signed-off-by: Kefu Chai <kchai@redhat.com>
no need to construct a list as the cmd. and more importantly,
the document of Rados.mon_command() reads:
Rados.mon_command(self, cmd, inbuf, timeout=0, target=None):
...
:param cmd: JSON formatted string.
Signed-off-by: Kefu Chai <kchai@redhat.com>
otherwise what monitor receives would be something like
mon_command({ " p r e f i x " : " o s d m a p " ...
difficult to parse/grep by machine/human.
Signed-off-by: Kefu Chai <kchai@redhat.com>
keyring is removed during daemon remove, which obsoletes the need to
update the keyring caps after get-or-create
Signed-off-by: Michael Fritch <mfritch@suse.com>
to prevent duplicate maintenance operations due to RPC hiccups.
Fixes: https://tracker.ceph.com/issues/46803
Signed-off-by: Mykola Golub <mgolub@suse.com>
Right now, only client IDs are stashed and restored but with the recent
changes (addition of more attributes to mount objects, specifically),
this is not enough. Saving and restoring these details before and after
tests respectively ensures that mount commands rus smoothly. Not doing
this typically leads to mount command failure for the second test in the
testsuite under execution since the client IDs are saved and restored in
CephFSTestCase.setUp and CephFSTestCase.tearDown respectively but the
rest of the details are not.
Signed-off-by: Rishabh Dave <ridave@redhat.com>
Add testsuite for testing authorization on Ceph cluster with multiple
file systems and enable it to be executable with Teuthology framework.
Also add helper methods required to setup the test environment for
multi-FS tests.
Signed-off-by: Rishabh Dave <ridave@redhat.com>
osd: Introduce new PGOpQueueable class for recovery push/reply messages.
Reviewed-by: Josh Durgin <jdurgin@redhat.com>
Reviewed-by: Samuel Just <sjust@redhat.com>
Reviewed-by: Neha Ojha <nojha@redhat.com>
Reviewed-by: David Zafman <dzafman@redhat.com>
Reviewed-by: xie xingguo <xie.xingguo@zte.com.cn>
