Project

General

Profile

Actions

Bug #4643

closed

mkcephfs repeated deprecated warning messages can be avoided for -a option

Added by Tamilarasi muthamizhan about 11 years ago. Updated about 11 years ago.

Status:
Won't Fix
Priority:
Normal
Assignee:
Neil Levine
Category:
-
Target version:
-
% Done:

0%

Source:
Q/A
Tags:
Backport:
Regression:
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

ceph branch: master [ceph version 0.60-409-g2f0e31f (2f0e31f31ee01158040c3f337a66d1ac0ed2dc5b)]

It would be nice to have a single deprecated warning message from mkcephfs at the end of the command when -a option is used.

ubuntu@burnupi13:/etc/ceph$ sudo mkcephfs -a -c ceph.conf -k ceph.keyring
temp dir is /tmp/mkcephfs.nEkyj0KgPX
preparing monmap in /tmp/mkcephfs.nEkyj0KgPX/monmap
/usr/bin/monmaptool --create --clobber --add a 10.214.134.26:6789 --print /tmp/mkcephfs.nEkyj0KgPX/monmap
/usr/bin/monmaptool: monmap file /tmp/mkcephfs.nEkyj0KgPX/monmap
/usr/bin/monmaptool: generated fsid 0e6770a1-027f-495f-84e6-5b329d116e03
epoch 0
fsid 0e6770a1-027f-495f-84e6-5b329d116e03
last_changed 2013-04-03 17:27:30.585629
created 2013-04-03 17:27:30.585629
0: 10.214.134.26:6789/0 mon.a
/usr/bin/monmaptool: writing epoch 0 to /tmp/mkcephfs.nEkyj0KgPX/monmap (1 monitors)

WARNING: mkcephfs is now deprecated in favour of ceph-deploy. Please see:
http://github.com/ceph/ceph-deploy === osd.1 ===
2013-04-03 17:27:30.738829 7fa7c0467780 1 filestore(/var/lib/ceph/osd/ceph-1) limited size xattrs - filestore_xattr_use_omap enabled
2013-04-03 17:27:30.767762 7fa7c0467780 -1 filestore(/var/lib/ceph/osd/ceph-1) could not find 23c2fcde/osd_superblock/0//-1 in index: (2) No such file or directory
2013-04-03 17:27:30.830974 7fa7c0467780 -1 created object store /var/lib/ceph/osd/ceph-1 journal /var/lib/ceph/osd/ceph-1/journal for osd.1 fsid 0e6770a1-027f-495f-84e6-5b329d116e03
2013-04-03 17:27:30.831022 7fa7c0467780 -1 auth: error reading file: /var/lib/ceph/osd/ceph-1/keyring: can't open /var/lib/ceph/osd/ceph-1/keyring: (2) No such file or directory
2013-04-03 17:27:30.831122 7fa7c0467780 -1 created new key in keyring /var/lib/ceph/osd/ceph-1/keyring

WARNING: mkcephfs is now deprecated in favour of ceph-deploy. Please see:
http://github.com/ceph/ceph-deploy === osd.2 ===
2013-04-03 17:27:30.953941 7f6e4c08e780 1 filestore(/var/lib/ceph/osd/ceph-2) limited size xattrs - filestore_xattr_use_omap enabled
2013-04-03 17:27:31.007280 7f6e4c08e780 -1 filestore(/var/lib/ceph/osd/ceph-2) could not find 23c2fcde/osd_superblock/0//-1 in index: (2) No such file or directory
2013-04-03 17:27:31.051023 7f6e4c08e780 -1 created object store /var/lib/ceph/osd/ceph-2 journal /var/lib/ceph/osd/ceph-2/journal for osd.2 fsid 0e6770a1-027f-495f-84e6-5b329d116e03
2013-04-03 17:27:31.051075 7f6e4c08e780 -1 auth: error reading file: /var/lib/ceph/osd/ceph-2/keyring: can't open /var/lib/ceph/osd/ceph-2/keyring: (2) No such file or directory
2013-04-03 17:27:31.051170 7f6e4c08e780 -1 created new key in keyring /var/lib/ceph/osd/ceph-2/keyring

WARNING: mkcephfs is now deprecated in favour of ceph-deploy. Please see:
http://github.com/ceph/ceph-deploy === mds.a ===
creating private key for mds.a keyring /var/lib/ceph/mds/ceph-a/keyring
creating /var/lib/ceph/mds/ceph-a/keyring

WARNING: mkcephfs is now deprecated in favour of ceph-deploy. Please see:
http://github.com/ceph/ceph-deploy
Building generic osdmap from /tmp/mkcephfs.nEkyj0KgPX/conf
/usr/bin/osdmaptool: osdmap file '/tmp/mkcephfs.nEkyj0KgPX/osdmap'
/usr/bin/osdmaptool: writing epoch 1 to /tmp/mkcephfs.nEkyj0KgPX/osdmap
Generating admin key at /tmp/mkcephfs.nEkyj0KgPX/keyring.admin
creating /tmp/mkcephfs.nEkyj0KgPX/keyring.admin
Building initial monitor keyring
added entity mds.a auth auth(auid = 18446744073709551615 key=AQDzyFxRmCUMCRAAeeF1DlrEHYCo6m7AjaiGEA== with 0 caps)
added entity osd.1 auth auth(auid = 18446744073709551615 key=AQDyyFxROHeIMRAA4AAuu83ZcNUTU+rR/+Nj0A== with 0 caps)
added entity osd.2 auth auth(auid = 18446744073709551615 key=AQDzyFxRQGsLAxAAWGIWxODTfKKS4zcVRMz7Ww== with 0 caps)

WARNING: mkcephfs is now deprecated in favour of ceph-deploy. Please see:
http://github.com/ceph/ceph-deploy === mon.a ===
/usr/bin/ceph-mon: created monfs at /var/lib/ceph/mon/ceph-a for mon.a

WARNING: mkcephfs is now deprecated in favour of ceph-deploy. Please see:
http://github.com/ceph/ceph-deploy
placing client.admin keyring in ceph.keyring

Actions #1

Updated by Neil Levine about 11 years ago

  • Status changed from New to Won't Fix

1. My previous fix was a 1 liner. Not sure how to make this only appear once only without a fair bit of hacking :-)
2. I think it's OK to have the message come up repeatedly as if it just at the end of a long list of STDOUT, it may get missed.

Actions #2

Updated by Dan Mick about 11 years ago

Actually, is there any reason this isn't just output at the beginning of mkcephfs? Why does it need to be at the end?

Actions #3

Updated by Neil Levine about 11 years ago

It could do but I fear that if it is just displayed at the top, people may miss it. It's not pretty but having it on exit, even if repeatedly, will ensure people read it.

Actions #4

Updated by Sage Weil about 11 years ago

Neil Levine wrote:

It could do but I fear that if it is just displayed at the top, people may miss it. It's not pretty but having it on exit, even if repeatedly, will ensure people read it.

It would be easy to make it only do the trap when -a is specified... which means only the calling process will spit out the message.

Actions

Also available in: Atom PDF