summaryrefslogtreecommitdiffstats
path: root/qa (follow)
Commit message (Collapse)AuthorAgeFilesLines
* qa: Enable flake8 tox and fix failuresThomas Bechtold2019-12-1265-184/+80
| | | | | | | | There were a couple of problems found by flake8 in the qa/ directory (most of them fixed now). Enabling flake8 during the usual check runs hopefully avoids adding new issues in the future. Signed-off-by: Thomas Bechtold <tbechtold@suse.com>
* Merge pull request #32143 from tchaikov/wip-qa-rocksdb-testKefu Chai2019-12-121-1/+1
|\ | | | | | | | | qa/workunits/rados/test_envlibrados_for_rocksdb: accomodate rocksdb c… Reviewed-by: Neha Ojha <nojha@redhat.com>
| * qa/workunits/rados/test_envlibrados_for_rocksdb: accomodate rocksdb cmake ↵Kefu Chai2019-12-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | changes since https://github.com/facebook/rocksdb/commit/bac38c992a3cc504e84a75ecdc085788b2c82ea2, tests are not enabled for non-debug builds. Signed-off-by: Kefu Chai <kchai@redhat.com>
* | qa/tasks/ceph2 -> cephadmSage Weil2019-12-125-6/+5
| | | | | | | | Signed-off-by: Sage Weil <sage@redhat.com>
* | mgr/ssh -> mgr/cephadmSage Weil2019-12-1224-15/+15
| | | | | | | | Signed-off-by: Sage Weil <sage@redhat.com>
* | ceph-daemon-adoption-corpus -> cephadm-adoption-corpusSage Weil2019-12-121-1/+1
| | | | | | | | Signed-off-by: Sage Weil <sage@redhat.com>
* | qa: rename ceph-daemon tests -> cephadmSage Weil2019-12-122-62/+60
| | | | | | | | | | | | Also move the workunit to a better location. Signed-off-by: Sage Weil <sage@redhat.com>
* | qa/tasks: ceph-daemon -> cephadm throughput var names and commentsSage Weil2019-12-122-22/+21
| | | | | | | | Signed-off-by: Sage Weil <sage@redhat.com>
* | mgr/ssh,qa/tasks/ceph2: fix mode to be cephadm-package (vs root)Sage Weil2019-12-123-10/+10
| | | | | | | | | | | | At the same time align the option names with ceph2.py, yay. Signed-off-by: Sage Weil <sage@redhat.com>
* | files,rpm,deb: rename ceph-daemon -> cephadmSage Weil2019-12-1220-29/+29
| | | | | | | | | | | | | | This is just renaming the files and adjusting the packages. Lots of cleanup to do still. Signed-off-by: Sage Weil <sage@redhat.com>
* | Merge PR #31729 into masterPatrick Donnelly2019-12-111-3/+6
|\ \ | | | | | | | | | | | | | | | | | | | | | * refs/pull/31729/head: qa: reduce cache size further mds: obsoleting 'mds_cache_size' Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
| * | qa: reduce cache size furtherPatrick Donnelly2019-12-101-1/+1
| | | | | | | | | | | | | | | | | | 1M isn't low enough to trigger recall/trimming. Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
| * | mds: obsoleting 'mds_cache_size'Ramana Raja2019-12-021-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | Remove last bits of support for 'mds_cache_size'. 'mds_cache_memory_limit' is preferred. Fixes: https://tracker.ceph.com/issues/41951 Signed-off-by: Ramana Raja <rraja@redhat.com>
* | | Merge PR #30201 into masterPatrick Donnelly2019-12-111-18/+54
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * refs/pull/30201/head: qa/tasks: remove subvolume, subvolumegroup and their snapshots with --force qa/tasks: Fix the commands success qa/tasks: Fix raises that doesn't re-raise Reviewed-by: Ramana Raja <rraja@redhat.com> Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
| * | | qa/tasks: remove subvolume, subvolumegroup and their snapshots with --forceJos Collin2019-11-201-0/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * tests 'fs subvolume rm --force' * tests 'fs subvolume snapshot rm --force' * tests 'fs subvolumegroup rm --force' * tests 'fs subvolumegroup snapshot rm --force' Signed-off-by: Jos Collin <jcollin@redhat.com>
| * | | qa/tasks: Fix the commands successJos Collin2019-11-201-11/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | * Raised RuntimeException when the commands, which were expected to fail succeed. * Dropped some commands with --force remove commands, as it is unnecessary. Signed-off-by: Jos Collin <jcollin@redhat.com>
| * | | qa/tasks: Fix raises that doesn't re-raiseJos Collin2019-11-201-8/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | * Fixed raises that doesn't re-raise * Dropped some commands with --force remove commands, as it is unnecessary. Signed-off-by: Jos Collin <jcollin@redhat.com>
* | | | Merge PR #32178 into masterSage Weil2019-12-111-0/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * refs/pull/32178/head: qa/standalone/test_ceph_daemon.sh: disable adoption for the moment Reviewed-by: Sebastian Wagner <swagner@suse.com>
| * | | | qa/standalone/test_ceph_daemon.sh: disable adoption for the momentSage Weil2019-12-111-0/+2
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Sage Weil <sage@redhat.com>
* | | | | Merge PR #32163 into masterPatrick Donnelly2019-12-111-12/+8
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | * refs/pull/32163/head: qa: fix output check to not be sensitive to debugging Reviewed-by: Rishabh Dave <ridave@redhat.com>
| * | | | qa: fix output check to not be sensitive to debuggingPatrick Donnelly2019-12-111-12/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The traceback would screw up the equality checks. Also, use unittest asserts so we can easily see the assert values. Fixes: https://tracker.ceph.com/issues/43247 Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
* | | | | Merge pull request #31839 from alimaredia/wip-s3-test-branch-refactorKefu Chai2019-12-1111-37/+51
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | qa: update s3-test download code for s3-test tasks Reviewed-by: Casey Bodley <cbodley@redhat.com> Reviewed-by: Yuri Weinstein <yweinste@redhat.com>
| * | | | qa: update s3-test download code for s3-test tasksAli Maredia2019-12-1011-37/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Ensure the download code for all tasks running s3-tests is consistent. - Simplify download code to only use the config variable 'force-branch' for the branch being cloned. - make ceph-master the force-branch for all suites using s3-tests. Fixes: https://tracker.ceph.com/issues/43077 Signed-off-by: Ali Maredia <amaredia@redhat.com>
* | | | | Merge PR #31772 into masterSage Weil2019-12-102-6/+7
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * refs/pull/31772/head: qa/standalone/test_ceph_daemon.sh: clone corpus explicitly ceph-daemon: re-enable the OSD standalone test ceph-daemon: move standalone test tgz to corpus ceph-daemon: add ceph-daemon-adoption-corpus.git submodule Reviewed-by: Tim Serong <tserong@suse.com> Reviewed-by: Sebastian Wagner <swagner@suse.com> Reviewed-by: Sage Weil <sage@redhat.com>
| * | | | | qa/standalone/test_ceph_daemon.sh: clone corpus explicitlySage Weil2019-12-101-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When this is run by teuthology we don't have a full ceph source tree checkout with submodules. Signed-off-by: Sage Weil <sage@redhat.com>
| * | | | | ceph-daemon: re-enable the OSD standalone testMichael Fritch2019-12-101-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Michael Fritch <mfritch@suse.com>
| * | | | | ceph-daemon: move standalone test tgz to corpusMichael Fritch2019-12-102-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/42876 Signed-off-by: Michael Fritch <mfritch@suse.com>
* | | | | | Merge pull request #30548 from trociny/wip-41557Jason Dillaman2019-12-101-2/+0
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | rbd-mirror: snapshot mirror mode Reviewed-by: Jason Dillaman <dillaman@redhat.com>
| * | | | | librbd: snapshot mirror modeMykola Golub2019-12-101-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enabling mirroring for an image that does not support journaling assumes snapshot based mirroring, which is supported only when the pool is in the "image" mirror mode. Also for the pool in the "image" mirror mode disabling/enabling journaling feature for a mirroring image will switch snapshot/journal mirror mode. Signed-off-by: Mykola Golub <mgolub@suse.com>
* | | | | | Merge PR #32138 into masterSage Weil2019-12-101-46/+53
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * refs/pull/32138/head: ceph-daemon: combine SUDO and ARGS into a single var Reviewed-by: Sebastian Wagner <swagner@suse.com> Reviewed-by: Sage Weil <sage@redhat.com>
| * | | | | | ceph-daemon: combine SUDO and ARGS into a single varMichael Fritch2019-12-091-46/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - reduce the amount of typing/noise for each CEPH_DAEMON invocation - ensure the `--image` param is passed to each test invocation - allow passing additional args to ceph-daemon via CEPH_DAEMON_ARGS Signed-off-by: Michael Fritch <mfritch@suse.com>
* | | | | | | Merge pull request #32122 from s0nea/wip-dashboard-fix-testsLenz Grimmer2019-12-101-4/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | qa/tasks/mgr/dashboard/test_health: update schema Reviewed-by: Alfonso Martínez <almartin@redhat.com>
| * | | | | | | qa/tasks/mgr/dashboard/test_health: update schemaTatjana Dehler2019-12-091-4/+1
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/43195 Signed-off-by: Tatjana Dehler <tdehler@suse.com>
* | | | | | | Merge pull request #32052 from mgfritch/wip-cd-standalone-tempfilesKefu Chai2019-12-101-9/+5
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ceph-daemon: clean-up tempfiles on EXIT Reviewed-by: Sage Weil <sage@redhat.com> Reviewed-by: Sebastian Wagner <swagner@suse.com>
| * | | | | | | ceph-daemon: remove guesswork to find script fileMichael Fritch2019-12-061-7/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow passing CEPH_DAEMON via the environment or default to using the script from the standard location. Signed-off-by: Michael Fritch <mfritch@suse.com>
| * | | | | | | ceph-daemon: trap on EXITMichael Fritch2019-12-061-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tempfiles were not being removed after a standalone test failure Signed-off-by: Michael Fritch <mfritch@suse.com>
* | | | | | | | Merge pull request #32100 from yehudasa/wip-43184Yehuda Sadeh2019-12-101-8/+7
|\ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | qa: radosgw-admin: remove dependency on bunch package Reviewed-by: Casey Bodley <cbodley@redhat.com>
| * | | | | | | qa: radosgw-admin: remove dependency on bunch packageYehuda Sadeh2019-12-091-8/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: https://tracker.ceph.com/issues/43184 Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
* | | | | | | | Merge PR #27894 into masterPatrick Donnelly2019-12-101-0/+41
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * refs/pull/27894/head: cephfs-shell: Add default list to df cephfs-shell: Add tests for df command cephfs-shell: Fix 'df' command errors Reviewed-by: Rishabh Dave <ridave@redhat.com> Reviewed-by: Patrick Donnelly <pdonnell@redhat.com>
| * | | | | | | | cephfs-shell: Add tests for df commandVarsha Rao2019-11-151-0/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds tests for valid and invalid directory, valid files and testing df without any arguments. Signed-off-by: Varsha Rao <varao@redhat.com>
* | | | | | | | | Merge PR #32098 into masterSage Weil2019-12-101-1/+1
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * refs/pull/32098/head: ceph-daemon: py2: tolerate whitespace before config key name Reviewed-by: Sebastian Wagner <swagner@suse.com>
| * | | | | | | | | ceph-daemon: py2: tolerate whitespace before config key nameSage Weil2019-12-091-1/+1
| | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The py2 ConfigParser doesn't like whitespace before the config option name. (The py3 version doesn't care.) Filter it out before parsing. Signed-off-by: Sage Weil <sage@redhat.com>
* | | | | | | | | Merge pull request #32088 from idryomov/wip-krbd-exclusive-lock-fixupsIlya Dryomov2019-12-092-28/+86
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | qa: krbd_exclusive_option.sh: update for recent kernel changes Reviewed-by: Jason Dillaman <dillaman@redhat.com>
| * | | | | | | | | qa: kernel.sh: unlock before rolling backIlya Dryomov2019-12-091-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "rbd snap rollback" expects an unlocked image, but we may get there locked if object map is enabled (or if lock_on_read is specified in rbd_default_map_options). Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
| * | | | | | | | | qa: krbd_exclusive_option.sh: update for recent kernel changesIlya Dryomov2019-12-091-28/+82
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since 5.3: - a plain "rbd map" acquires the lock, so it's not different from "rbd map -o exclusive" in this regard - if the lock is held by the exclusive peer, I/O is failed right away instead of blocking - lock_timeout option is respected only by "rbd map" and not by I/O Since 5.5: - if the mapping is read-only, the lock isn't acquired Added blacklisting test case, dropped lock_timeout test case. Fixes: https://tracker.ceph.com/issues/43127 Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
* | | | | | | | | | Merge PR #32046 into masterSage Weil2019-12-091-0/+48
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * refs/pull/32046/head: mgr/DaemonServer: fix 'osd ok-to-stop' for EC pools Reviewed-by: Neha Ojha <nojha@redhat.com>
| * | | | | | | | | mgr/DaemonServer: fix 'osd ok-to-stop' for EC poolsSage Weil2019-12-051-0/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We need to pay attention to account for CRUSH_ITEM_NONE entries in the EC PG acting set. Fixes: https://tracker.ceph.com/issues/43151 Signed-off-by: Sage Weil <sage@redhat.com>
* | | | | | | | | | Merge PR #32108 into masterSage Weil2019-12-081-1/+0
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * refs/pull/32108/head: ceph-daemon: remove prepare-host Reviewed-by: Sebastian Wagner <swagner@suse.com>
| * | | | | | | | | | ceph-daemon: remove prepare-hostSage Weil2019-12-081-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I thought I took this out of the PR but somehow it got merged in... must have repushed and old branch and not realized. :/ Signed-off-by: Sage Weil <sage@redhat.com>
* | | | | | | | | | | Merge PR #32101 into masterSage Weil2019-12-081-2/+2
|\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * refs/pull/32101/head: mgr/orchestrator_cli: make rgw rm take realm+zone qa/tasks/mgr/test_orchestrator_cli: fix rgw add test Reviewed-by: Sebastian Wagner <swagner@suse.com>