Ubuntu
ceph package

The '--print' option to ceph-authtool is incorrect

Bug #1056462 reported by Andrew Glen-Young on 2012-09-25

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	ceph (Ubuntu)	Fix Released	Low	Unassigned

Bug Description

Running `ceph-authtool keyringfile --print` throws an error claiming that it does not know the option however, the help indicates that this is the valid long-option for '-p'.

Looking at the source code, the command expects '--print-key' and using this works as expected.

Either the help is incorrect or the '--print-key' option is incorrect.

How to reproduce:

    $ ceph-authtool --help
    no command specified
    usage: ceph-authtool keyringfile [OPTIONS]...
    where the options are:
      -l, --list will list all keys and capabilities present in
                                    the keyring
      -p, --print will print an encoded key for the specified
                                    entityname. This is suitable for the
                                    'mount -o secret=..' argument
    [...]

    $ ceph-authtool /etc/ceph/keyring --print
    ceph-authtool: unexpected '--print'
    [...]

$ ceph-authtool /etc/ceph/keyring -p
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX==

$ ceph-authtool /etc/ceph/keyring --print-key
XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX==

See the trivial patch attached.

Tags:

Related branches

lp:ubuntu/raring-proposed/ceph

Revision history for this message

Andrew Glen-Young (aglenyoung) wrote on 2012-09-25:

Correct help message to provide correct long option '--print-key' Edit (774 bytes, text/plain)

Revision history for this message

Clint Byrum (clint-fewbar) wrote on 2012-09-25:

This is fixed upstream in this commit:

http://ceph.com/git/?p=ceph.git;a=commit;h=4e40a7849442d633c9801639fbdaf27a97f39888

Revision history for this message

Clint Byrum (clint-fewbar) wrote on 2012-09-25:

This is fixed upstream in version 0.51

Revision history for this message

Ubuntu Foundations Team Bug Bot (crichton) wrote on 2012-09-26:

The attachment "Correct help message to provide correct long option '--print-key'" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags:

added: patch

James Page (james-page) on 2012-10-06

Changed in ceph (Ubuntu):
status:	New → Confirmed
importance:	Undecided → Low
status:	Confirmed → Triaged

Revision history for this message

Sage Weil (sage-newdream) wrote on 2012-10-08:

I've also queued this patch in the stable branch so that it'll be included in the next stable point release (0.48.3).

Revision history for this message

Launchpad Janitor (janitor) wrote on 2012-11-19:

This bug was fixed in the package ceph - 0.54-0ubuntu1

---------------
ceph (0.54-0ubuntu1) raring; urgency=low

  * New upstream release, preview for next upstream LTS release
    (LP: #1056462):
    - d/librbd-dev.install: Pickup new features.h file.
    - d/control: Suggest parted to support upstart integration.
    - d/{control,obsync.install}: Drop obsync package inline with
      upstream.
    - d/control: Add xfsprogs to Depends for ceph to support XFS
      as default filesystem (LP: #1028981).
    - d/patches/*: Drop all patches; either included upstream or
      no longer required.
    - d/ceph.lintian-overrides: Added additional overrides for
      new rados class libraries.
  * d/watch: Switch back to using distributed tarball's rather than
    pulling from github so that version resolution works correctly
    (LP: #1079941).
  * Tidied up redundant packaging files from debian directory.
-- James Page <email address hidden> Mon, 19 Nov 2012 10:15:42 +0000