From: Josh Durgin <josh.durgin@inktank.com>
To: Tommi Virtanen <tv@inktank.com>
Cc: Florian Haas <florian@hastexo.com>,
"ceph-devel@vger.kernel.org" <ceph-devel@vger.kernel.org>,
"Plaetinck, Dieter" <dieter@vimeo.com>,
Ross Turk <ross@inktank.com>
Subject: Re: Integration work
Date: Tue, 28 Aug 2012 14:20:54 -0700 [thread overview]
Message-ID: <503D3636.40609@inktank.com> (raw)
In-Reply-To: <CADvuQRHs_t0WT_eERPP_1Eb3wTM0yX5_v+WW3rRnY3JqWw2-3A@mail.gmail.com>
On 08/28/2012 02:15 PM, Tommi Virtanen wrote:
> On Tue, Aug 28, 2012 at 5:03 PM, Florian Haas <florian@hastexo.com> wrote:
>> I for my part, in the documentation space, would love for the admin
>> tools to become self-documenting. For example, I would love a "help"
>> subcommand at any level of the ceph shell, listing the supported
>> subcommands in that level. As in "ceph help", "ceph mon help", "ceph osd
>> getmap help".
>>
>> Even better, the ceph shell could support a general-purpose hook that
>> bash-completion can use (kind of like "hg" does in Mercurial), and this
>> and the above-conjectured help facility could arguably share quite a bit
>> of code.
>
> I would love to see all of that. But, a lot of the "ceph" tool
> functionality is implemented by shoveling strings in and out of the
> monitors. It largely doesn't understand what's happening.
It doesn't need to understand what's happening to give basic usage info
though - the monitors can provide that themselves in the short term
while we don't have an admin api like you describe below.
I added a feature request for this a little while back:
http://www.tracker.newdream.net/issues/2894
> If we were to redo that from scratch, I'd convert that to have some
> sort of API to monitors, and make the cli understand all the relevant
> things. Understandably, that can feel a little bit more rigid; to add
> a command means adding it to both the server and a client, where as
> currently the client is very very generic.
next prev parent reply other threads:[~2012-08-28 21:20 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-28 18:12 Integration work Ross Turk
2012-08-28 18:32 ` Plaetinck, Dieter
2012-08-28 21:03 ` Florian Haas
2012-08-28 21:15 ` Tommi Virtanen
2012-08-28 21:20 ` Josh Durgin [this message]
2012-08-30 14:00 ` João Eduardo Luís
2012-08-28 18:51 ` Dieter Kasper
2012-08-28 18:57 ` Smart Weblications GmbH - Florian Wiessner
2012-08-28 20:05 ` Dieter Kasper
2012-08-28 20:46 ` Tren Blackburn
2012-08-29 7:06 ` Amon Ott
2012-08-29 8:20 ` Sylvain Munaut
2012-08-29 9:53 ` Wido den Hollander
2012-08-29 12:35 ` Sylvain Munaut
2012-08-29 13:40 ` Wido den Hollander
2012-08-29 13:43 ` Tommi Virtanen
2012-08-29 15:19 ` Joseph Glanville
2012-09-01 6:02 ` Ryan Nicholson
2012-09-04 15:52 ` Tommi Virtanen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=503D3636.40609@inktank.com \
--to=josh.durgin@inktank.com \
--cc=ceph-devel@vger.kernel.org \
--cc=dieter@vimeo.com \
--cc=florian@hastexo.com \
--cc=ross@inktank.com \
--cc=tv@inktank.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox