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:

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

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

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