CEPH filesystem development
 help / color / mirror / Atom feed
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.


  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