Re: [PATCH master] New configuration option for the rbd user-id

2016-08-22 Thread 'Brian Foley' via ganeti-devel
On Sun, Aug 21, 2016 at 08:58:01PM -0600, David Mohr wrote: > The user id is used by ceph to determine the keyring to use for > authentication. By default the admin keyring is used, which may > not be desirable. Example usage: > > $ gnt-cluster modify -D rbd:user-id=foobar > > Signed-off-by:

[PATCH master] New configuration option for the rbd user-id

2016-08-21 Thread David Mohr
The user id is used by ceph to determine the keyring to use for authentication. By default the admin keyring is used, which may not be desirable. Example usage: $ gnt-cluster modify -D rbd:user-id=foobar Signed-off-by: David Mohr --- lib/storage/bdev.py| 48

Re: [PATCH master] New configuration option for the rbd user-id

2016-08-08 Thread 'Brian Foley' via ganeti-devel
On Sat, Jul 23, 2016 at 09:18:34PM -0600, David Mohr wrote: > The user id is used by ceph to determine the keyring to use for > authentication. By default the admin keyring is used, which may > not be desirable. Example usage: > > $ gnt-cluster modify -D rbd:user-id=foobar > > Signed-off-by:

Re: [PATCH master] New configuration option for the rbd user-id

2016-07-23 Thread David Mohr
This was in reply to https://groups.google.com/forum/#!topic/ganeti-devel/dWVgZFk-MdM ; I seem to have screwed up the message-id when using send-email. It has the fix for the unit tests. However, the check-local target doesn't pass on my system. It doesn't seem to be related to my patch, but

[PATCH master] New configuration option for the rbd user-id

2016-07-23 Thread David Mohr
The user id is used by ceph to determine the keyring to use for authentication. By default the admin keyring is used, which may not be desirable. Example usage: $ gnt-cluster modify -D rbd:user-id=foobar Signed-off-by: David Mohr --- lib/storage/bdev.py| 44

Re: [PATCH master] New configuration option for the rbd user-id

2016-06-23 Thread 'Brian Foley' via ganeti-devel
On Thu, Jun 23, 2016 at 06:59:34AM -0700, David Mohr wrote: >Hi Brian, >this is the squashed commit of the actual code changes; without this >the documentation update is slightly misleading ;-) Woops. I guess I was a bit too quick off the mark! >Thanks for applying it! I'll be

Re: [PATCH master] New configuration option for the rbd user-id

2016-06-23 Thread 'Brian Foley' via ganeti-devel
On Thu, Jun 23, 2016 at 03:57:50PM +0200, David Mohr wrote: > The user id is used by ceph to determine the keyring to use for > authentication. By default the admin keyring is used, which may > not be desirable. Example usage: > > $ gnt-cluster modify -D rbd:user-id=foobar > > Signed-off-by:

Re: [PATCH master] New configuration option for the rbd user-id

2016-06-23 Thread David Mohr
Hi Brian, this is the squashed commit of the actual code changes; without this the documentation update is slightly misleading ;-) Thanks for applying it! I'll be back soon with more patches. ~David On Thursday, June 23, 2016 at 3:57:52 PM UTC+2, David Mohr wrote: > > The user id is used by

[PATCH master] New configuration option for the rbd user-id

2016-06-23 Thread David Mohr
The user id is used by ceph to determine the keyring to use for authentication. By default the admin keyring is used, which may not be desirable. Example usage: $ gnt-cluster modify -D rbd:user-id=foobar Signed-off-by: David Mohr --- lib/storage/bdev.py | 44

Re: [PATCH master] New configuration option for the rbd user id

2016-05-09 Thread 'Brian Foley' via ganeti-devel
On Sun, May 08, 2016 at 12:19:28PM -0700, David Mohr wrote: >On Tuesday, May 3, 2016 at 6:20:43 AM UTC-6, Brian Foley wrote: > > Hi David, > Thanks for the patch. > I'm afraid we don't use Ceph internally, so we don't really have a > good way > of testing this. I

Re: [PATCH master] New configuration option for the rbd user id

2016-05-08 Thread David Mohr
On Tuesday, May 3, 2016 at 6:20:43 AM UTC-6, Brian Foley wrote: > Hi David, > > Thanks for the patch. > > I'm afraid we don't use Ceph internally, so we don't really have a good > way > of testing this. I assume you've tested it and it's working for you > locally. > Hi Brian, thank you

Re: [PATCH master] New configuration option for the rbd user id

2016-05-03 Thread 'Brian Foley' via ganeti-devel
On Mon, May 02, 2016 at 06:10:28PM -0600, David Mohr wrote: > The user id is used by ceph to determine the keyring to use for > authentication. By default the admin keyring is used, which may > not be desirable. > > Signed-off-by: David Mohr Hi David, Thanks for the patch. I'm

[PATCH master] New configuration option for the rbd user id

2016-05-03 Thread David Mohr
The user id is used by ceph to determine the keyring to use for authentication. By default the admin keyring is used, which may not be desirable. Signed-off-by: David Mohr --- lib/storage/bdev.py | 45 - src/Ganeti/Constants.hs |