From 63c60410684c833bd79d23be04caac7d5f586690 Mon Sep 17 00:00:00 2001 From: Zac Dover Date: Wed, 18 May 2022 01:16:16 +1000 Subject: [PATCH] doc/dev: refine basic-workflow.rst integ. test This is one in a set of PRs meant to keep the Basic Workflow in the Developer guide current. It refines the English in the "Integration Tests AKA ceph-qa-suite" section of "Basic Workflow". Several other small updates like this are expected. I intend to avoid refining half of the page in one commit, as I did last month when I refined the first half of the basic workflow. Co-authored-by: Anthony D'Atri Signed-off-by: Zac Dover --- doc/dev/developer_guide/basic-workflow.rst | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/doc/dev/developer_guide/basic-workflow.rst b/doc/dev/developer_guide/basic-workflow.rst index 1647983d47d6..8f91c2021da6 100644 --- a/doc/dev/developer_guide/basic-workflow.rst +++ b/doc/dev/developer_guide/basic-workflow.rst @@ -383,9 +383,8 @@ determine the cause of your specific failure. Integration tests AKA ceph-qa-suite ----------------------------------- -Since Ceph is complex, it may be necessary to test your fix to -see how it behaves on real clusters running on physical or virtual -hardware. Tests designed for this purpose live in the `ceph/qa +It may be necessary to test your fix on real Ceph clusters that run on physical +or virtual hardware. Tests designed for this purpose live in the `ceph/qa sub-directory`_ and are run via the `teuthology framework`_. .. _`ceph/qa sub-directory`: https://github.com/ceph/ceph/tree/master/qa/ @@ -395,10 +394,12 @@ sub-directory`_ and are run via the `teuthology framework`_. The Ceph community has access to the `Sepia lab `_ where `integration tests`_ can be run on physical hardware. -Other developers may add tags like "needs-qa" to your PR. This allows PRs that -need testing to be merged into a single branch and tested all at the same time. -Since teuthology suites can take hours (even days in some cases) to run, this -can save a lot of time. + +Other contributors might add tags like `needs-qa` to your PR. This allows PRs +to be merged into a single branch and then efficiently tested together. +Teuthology test suites can take hours (and even days in some cases) to +complete, so batching tests reduces contention for resources and saves a lot of +time. To request access to the Sepia lab, start `here `_. -- 2.47.3