All of lore.kernel.org
 help / color / mirror / Atom feed
From: Abhishek L <abhishek.lekshmanan@gmail.com>
To: Yehuda Sadeh <yehuda@redhat.com>
Cc: ceph-devel <ceph-devel@vger.kernel.org>
Subject: Re: [PATCH 0/3] WIP-8112 radosgw usage & man update
Date: Wed, 13 Aug 2014 16:25:54 +0530	[thread overview]
Message-ID: <87fvh0bq54.fsf@gmail.com> (raw)
In-Reply-To: <CABBk=J8ceLh4i_DiGRS4yq1rhdNr5ssookFu8__c5ohxyTjqpg@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 692 bytes --]

Hi 

Actually there was a minor error in the above patchset, the help options
should have come before `generic_server_usage()` as the function exits
after that. I have updated this as a PR at https://github.com/ceph/ceph/pull/2248

This should hopefully close the issues http://tracker.ceph.com/issues/8112 
and related http://tracker.ceph.com/issues/8864. However as mentioned in #8112
teuthology shows some apache conf file location options being passed for
apache conf & rgw ops log etc. I'm not sure whether those (and a few
others options related to rgw seen in common/config_opts.h) should be
displayed in help. Let me know if any other options are to be documented
we can close this.


[-- Attachment #2.1: Type: text/plain, Size: 15 bytes --]

-- 
Abhishek

[-- Attachment #2.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 472 bytes --]

      reply	other threads:[~2014-08-13 11:12 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-24 15:00 [PATCH 0/3] WIP-8112 radosgw usage & man update Abhishek Lekshmanan
2014-07-24 15:00 ` [PATCH 1/3] rgw: format help options to align with the rest Abhishek Lekshmanan
2014-07-24 15:00 ` [PATCH 2/3] rgw: list all available options during help() Abhishek Lekshmanan
2014-07-24 15:00 ` [PATCH 3/3] doc: update radosgw man page with available opts Abhishek Lekshmanan
2014-07-24 20:15 ` [PATCH 0/3] WIP-8112 radosgw usage & man update Yehuda Sadeh
2014-08-13 10:55   ` Abhishek L [this message]

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=87fvh0bq54.fsf@gmail.com \
    --to=abhishek.lekshmanan@gmail.com \
    --cc=ceph-devel@vger.kernel.org \
    --cc=yehuda@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.