Re: [libvirt] [PATCH v5 4/4] qemu/rbd: improve rbd device specification

2011-11-16 Thread Daniel P. Berrange
On Tue, Nov 15, 2011 at 05:05:27PM -0700, Eric Blake wrote: On 10/31/2011 07:29 PM, Josh Durgin wrote: From: Sage Weil s...@newdream.net +if (sec) { +char *base64 = NULL; + +secret = (char *)conn-secretDriver-getValue(sec, secret_size, 0, +

Re: Assert in OSDMap::Incremental::decode with recent git

2011-11-16 Thread Josh Pieper
on vstart, since that's supposed to use the new mon bootstrapping pieces as well. Josh, can you turn up monitor debugging and send me the log/post it somewhere? Presumably the big refactor Sage referred to broke something here. http://joshp.no-ip.com:8080/2016-ceph-mon.2.log I've inlined

Re: [PATCH v5 4/4] qemu/rbd: improve rbd device specification

2011-11-16 Thread Eric Blake
On 11/15/2011 06:37 PM, Josh Durgin wrote: The command line that we pass to qemu gets logged. But what happens if the secret was marked as ephemeral - could we be violating the premise of not exposing passwords to too broad an audience? Or are we already safe in that the log entries created

Re: rbd format check

2011-11-16 Thread Sage Weil
On Tue, 15 Nov 2011, Josh Durgin wrote: I added a simple check for the old version in the wip-rbd-layering branch of ceph and ceph-client.git. If that looks good and you want to push it upstream, maybe grab the rollback removal from wip-rollback as well. On the librbd side, let's add a dout(0

Re: rbd format check

2011-11-16 Thread Josh Durgin
On 11/16/2011 10:40 AM, Sage Weil wrote: On Tue, 15 Nov 2011, Josh Durgin wrote: I added a simple check for the old version in the wip-rbd-layering branch of ceph and ceph-client.git. If that looks good and you want to push it upstream, maybe grab the rollback removal from wip-rollback as well.

Re: rbd format check

2011-11-16 Thread Sage Weil
On Wed, 16 Nov 2011, Josh Durgin wrote: On 11/16/2011 10:40 AM, Sage Weil wrote: On Tue, 15 Nov 2011, Josh Durgin wrote: I added a simple check for the old version in the wip-rbd-layering branch of ceph and ceph-client.git. If that looks good and you want to push it upstream,

[no subject]

2011-11-16 Thread Wido den Hollander
When upgrading or installing we do not want to stop or start the init scripts. This could break upgrades but also do harmfull stuff we don't want. Let the sysadmin decide when to (re)start the daemons -- To unsubscribe from this list: send the line unsubscribe ceph-devel in the body of a

[PATCH] debian init: Do not stop or start daemons when installing or upgrading

2011-11-16 Thread Wido den Hollander
Signed-off-by: Wido den Hollander w...@widodh.nl --- debian/rules |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/debian/rules b/debian/rules index a20258d..4f3fe62 100755 --- a/debian/rules +++ b/debian/rules @@ -98,7 +98,7 @@ binary-arch: build install

Re: [PATCH] debian init: Do not stop or start daemons when installing or upgrading

2011-11-16 Thread Sage Weil
Ooh, that would certainly make things less annoying for me as well. Unless that's contrary to some debian policy/best practice, Laszlo? sage On Wed, 16 Nov 2011, Wido den Hollander wrote: Signed-off-by: Wido den Hollander w...@widodh.nl --- debian/rules |2 +- 1 files changed, 1

Re: [PATCH] debian init: Do not stop or start daemons when installing or upgrading

2011-11-16 Thread Wido den Hollander
On 11/16/2011 08:53 PM, Sage Weil wrote: Ooh, that would certainly make things less annoying for me as well. Unless that's contrary to some debian policy/best practice, Laszlo? We could also go for --no-restart-on-upgrade. I ran into this when upgrading the debs on my machines. The init

phprados update

2011-11-16 Thread Wido den Hollander
Hi, Some time ago the API of librados changed thus breaking phprados. I tried implementing the C++ API of librados in PHP but I got pretty stuck there with the new IoCTX stuff. So I went to the C API and started implementing a C-only phprados with just using the C functions. I just

Re: rbd format check

2011-11-16 Thread Josh Durgin
On 11/16/2011 11:10 AM, Sage Weil wrote: On Wed, 16 Nov 2011, Josh Durgin wrote: On 11/16/2011 10:40 AM, Sage Weil wrote: On Tue, 15 Nov 2011, Josh Durgin wrote: I added a simple check for the old version in the wip-rbd-layering branch of ceph and ceph-client.git. If that looks good and you