Skip to content
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: missing man pages for ceph-create-keys, ceph-disk-* #4846

Merged
2 commits merged into from Jun 15, 2015

Conversation

smithfarm
Copy link
Contributor

@smithfarm
Copy link
Contributor Author

@tchaikov can you take a look?

@tchaikov
Copy link
Contributor

tchaikov commented Jun 3, 2015

once it is also added into the rpm spec file. and confirmed by the gitbuilders. Reviewed-by.

@smithfarm i'd suggest you to push it to a wip branch to get it tested with the tireless builders.

@smithfarm
Copy link
Contributor Author

@tchaikov: I noticed that http://tracker.ceph.com/issues/10725 is asking for 4 manpages, but it looks like you resolved it after adding only this one . . . (?) Should we open issues for the remaining 3?

@tchaikov
Copy link
Contributor

tchaikov commented Jun 3, 2015

i don't think so. see http://tracker.ceph.com/issues/10725#note-11 .

@smithfarm
Copy link
Contributor Author

@tchaikov I think the make-check bot is paused, see Loic's email to ceph-devel delivered to me today 2015-06-03 at 01:48 CEST.

@tchaikov
Copy link
Contributor

tchaikov commented Jun 3, 2015

@tchaikov I think the make-check bot is paused, see Loic's email to ceph-devel delivered to me today 2015-06-03 at 01:48 CEST.

okay. would you mind pushing it to ceph repo to get it built by gitbuilders?

:program:`ceph-create-keys` is a utility to generate bootstrap keyrings using
the given monitor when it is ready.

it creates following auth entities (or users)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Possibly It

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry didn't notice that this was a backport PR.. Guess it should be fine either way

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@theanalyst might be a chance to fix it though. we could add a note in the commit message for this small edit.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as long as it gets fixed in master too so it doesn't get lost :)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

... or should I say "It" doesn't get lost! 😆

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tchaikov added a commit to tchaikov/ceph that referenced this pull request Jun 4, 2015
to address the comments on ceph#4846

Signed-off-by: Kefu Chai <kchai@redhat.com>
tchaikov added a commit to tchaikov/ceph that referenced this pull request Jun 4, 2015
to address the comments on ceph#4846

Fixes: ceph#10725
Signed-off-by: Kefu Chai <kchai@redhat.com>
@smithfarm
Copy link
Contributor Author

@tchaikov: rebased to include fixes from #4855

@tchaikov
Copy link
Contributor

tchaikov commented Jun 4, 2015

@smithfarm i am sure if i follow you, but the cherry-pick from 27cee2f is missing some bits bit:
SUSE@27cee2f#diff-ca40628478ff1274b73dcd40c384e7f0R640

joaquimrocha pushed a commit to joaquimrocha/ceph that referenced this pull request Jun 5, 2015
to address the comments on ceph#4846

Fixes: ceph#10725
Signed-off-by: Kefu Chai <kchai@redhat.com>
joaquimrocha pushed a commit to joaquimrocha/ceph that referenced this pull request Jun 5, 2015
to address the comments on ceph#4846

Fixes: ceph#10725
Signed-off-by: Kefu Chai <kchai@redhat.com>
tchaikov and others added 2 commits June 6, 2015 08:29
Fixes: ceph#10725
Signed-off-by: Kefu Chai <kchai@redhat.com>
(cherry picked from commit 27cee2f)

Conflicts:
    doc/man/8/ceph-create-keys.rst
    Includes fixes from ceph#4855
This is the generated manpage, taken from a local build of master

Signed-off-by: Nathan Cutler <ncutler@suse.com>
(cherry picked from commit bcda61f)
@smithfarm
Copy link
Contributor Author

@tchaikov: added the missing bit in ceph.spec.in - what else is missing?

@tchaikov
Copy link
Contributor

tchaikov commented Jun 6, 2015

@smithfarm no, thanks! Reviewed-by: Kefu Chai <kchai@redhat.com> and also i pushed to ceph.git/wip-4846, the rpm builders are good.

@ktdreyer
Copy link
Member

ktdreyer commented Jun 8, 2015

LGTM

Reviewed-by: Ken Dreyer <kdreyer@redhat.com>

ghost pushed a commit that referenced this pull request Jun 15, 2015
missing man pages for ceph-create-keys, ceph-disk-*

Reviewed-by: Kefu Chai <kchai@redhat.com>
Reviewed-by: Ken Dreyer <kdreyer@redhat.com>
@ghost ghost merged commit 70bba62 into ceph:hammer Jun 15, 2015
@smithfarm smithfarm deleted the wip-11862-hammer branch June 15, 2015 15:47
@ghost ghost changed the title missing man pages for ceph-create-keys, ceph-disk-* doc: missing man pages for ceph-create-keys, ceph-disk-* Aug 4, 2015
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants