New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: update links to point to ceph/qa instead of ceph-qa-suite #13397

Merged
merged 3 commits into from Mar 8, 2017

Conversation

Projects
None yet
5 participants
@jan--f
Member

jan--f commented Feb 13, 2017

Also fix two broken links to install task.

Signed-off-by: Jan Fajerski jfajerski@suse.com

@jan--f

This comment has been minimized.

Member

jan--f commented Feb 13, 2017

@smithfarm you wanted to have a look?

* `rgw <https://github.com/ceph/ceph/tree/master/qa/suites/rgw>`_ suite
* `CephFS <https://github.com/ceph/ceph/tree/master/qa/suites/fs>`_ suite
* `rados <https://github.com/ceph/ceph/tree/master/qa/suites/rados>`_ suite
* `rbd <https://github.com/ceph/ceph/tree/master/qa/suites/rbd>`_ suite
and a many others such as

This comment has been minimized.

@smithfarm

smithfarm Feb 13, 2017

Contributor

and many others (drop the indefinite article)

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Feb 13, 2017

@jan--f: LGTM. I pushed a commit modifying the very last section of doc/dev/index.rst - please take a look.

@liewegas

This comment has been minimized.

Member

liewegas commented Feb 14, 2017

retest this please

@jan--f

This comment has been minimized.

Member

jan--f commented Feb 14, 2017

@smithfarm looks good. Test failures seem unrelated.

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Feb 14, 2017

@jan--f Looks like make check is failing on all master PRs atm.

@dmick

This comment has been minimized.

Member

dmick commented Feb 14, 2017

include fc28dca

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Feb 14, 2017

@dmick Thanks, rebased to grab that.

@dmick

This comment has been minimized.

Member

dmick commented Mar 3, 2017

someone want to merge this?

This common use case is currently provided for by the following command::
- exec:
client.0:
- sleep 1000000000 # forever

This comment has been minimized.

@tchaikov

tchaikov Mar 3, 2017

Contributor

would be

tasks:
 - interactive

a better choice?

This comment has been minimized.

@smithfarm

smithfarm Mar 3, 2017

Contributor

@tchaikov Good idea! I read the documentation at https://github.com/ceph/teuthology/blob/master/teuthology/task/interactive.py#L13-L29 from which I suppose that this causes an interactive Python shell to be run, but I'm not sure how to connect to that shell?

This comment has been minimized.

@tchaikov

tchaikov Mar 3, 2017

Contributor

i am able to have the interactive python shell using following command in sepia.

teuthology ~/test.yaml --lock --machine-type mira

where test.yaml is

roles:
    - [client.0]
repo: git://git.ceph.com/ceph.git
tasks:
    - interactive:

@smithfarm smithfarm self-assigned this Mar 3, 2017

jan--f and others added some commits Feb 13, 2017

doc: update links to point to ceph/qa instead of ceph-qa-suite
Also fix two broken links to install task and two typos.

Signed-off-by: Jan Fajerski <jfajerski@suse.com>
doc: rewrite Deploy a cluster for manual testing section
Signed-off-by: Nathan Cutler <ncutler@suse.com>
doc: mention interactive task in developer guide
Signed-off-by: Nathan Cutler <ncutler@suse.com>
@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Mar 8, 2017

@tchaikov I added a commit mentioning the interactive task. Please take a look.

@tchaikov tchaikov merged commit f46b327 into ceph:master Mar 8, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@smithfarm smithfarm deleted the SUSE:doc-fix-qa-links branch Mar 8, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment