Re: Issue 1076 in ganeti: disk adoption should make sure, that the disk is not allready in use

2016-05-03 Thread ganeti
Comment #3 on issue 1076 by bpfo...@google.com: disk adoption should make sure, that the disk is not allready in use https://code.google.com/p/ganeti/issues/detail?id=1076 Agreed. Sounds like a good idea, but it'll need to be reimplemented as AddInstanceDisk was moved into wconfd in commit

Re: Issue 1175 in ganeti: ext3 filesystem hardcoded in fstab of import script

2016-05-03 Thread ganeti
Updates: Status: Fixed Comment #2 on issue 1175 by bpfo...@google.com: ext3 filesystem hardcoded in fstab of import script https://code.google.com/p/ganeti/issues/detail?id=1175 Thanks for the report. Fixed in

Re: [PATCH debootstrap] Use OSP_FILESYSTEM for fstab and mk2efs in import

2016-05-03 Thread 'Viktor Bachraty' via ganeti-devel
LGTM On Tue, May 3, 2016 at 1:52 PM, 'Brian Foley' via ganeti-devel < ganeti-devel@googlegroups.com> wrote: > These were missed previously when adding OSP_FILESYSTEM support, and > may cause problems if OSP_FILESYSTEM is not compatible with the default > filesystem chosen by mke2fs. > > Thanks

[PATCH debootstrap] Use OSP_FILESYSTEM for fstab and mk2efs in import

2016-05-03 Thread 'Brian Foley' via ganeti-devel
These were missed previously when adding OSP_FILESYSTEM support, and may cause problems if OSP_FILESYSTEM is not compatible with the default filesystem chosen by mke2fs. Thanks to hostingnuggets for reporting and suggesting the fix in #1175. Signed-off-by: Brian Foley ---

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

Re: [MERGE] stable-2.17 to master

2016-05-03 Thread 'Viktor Bachraty' via ganeti-devel
LGTM On Tue, May 3, 2016 at 12:12 PM, 'Brian Foley' via ganeti-devel < ganeti-devel@googlegroups.com> wrote: > commit e986d58e14865c4082c634ce1c925562617611be > Merge: 8b2ec2f 0b57a03 > Author: Brian Foley > Date: Tue May 3 11:56:08 2016 +0100 > > Merge branch

[MERGE] stable-2.17 to master

2016-05-03 Thread 'Brian Foley' via ganeti-devel
commit e986d58e14865c4082c634ce1c925562617611be Merge: 8b2ec2f 0b57a03 Author: Brian Foley Date: Tue May 3 11:56:08 2016 +0100 Merge branch 'stable-2.17' * stable-2.15 Use socat method string compatible with <1.73 & >=1.73 Reduce heap when parsing &

Re: [MERGE] stable-2.16 to stable-2.17

2016-05-03 Thread 'Viktor Bachraty' via ganeti-devel
LGTM On Tue, May 3, 2016 at 11:47 AM, 'Brian Foley' via ganeti-devel < ganeti-devel@googlegroups.com> wrote: > commit 394c8ffa24f93223049d11c6e7f676d04d5c4754 > Merge: b9de363 cf10806 > Author: Brian Foley > Date: Tue May 3 11:32:16 2016 +0100 > > Merge branch

[MERGE] stable-2.16 to stable-2.17

2016-05-03 Thread 'Brian Foley' via ganeti-devel
commit 394c8ffa24f93223049d11c6e7f676d04d5c4754 Merge: b9de363 cf10806 Author: Brian Foley Date: Tue May 3 11:32:16 2016 +0100 Merge branch 'stable-2.16' into stable-2.17 * stable-2.15 Use socat method string compatible with <1.73 & >=1.73 Reduce

Re: [MERGE] stable-2.15 to stable-2.16

2016-05-03 Thread 'Viktor Bachraty' via ganeti-devel
LGTM On Tue, May 3, 2016 at 11:18 AM, 'Brian Foley' via ganeti-devel < ganeti-devel@googlegroups.com> wrote: > commit 3afc662c93d2d5292874b76fcbb6aa7eb1b74f57 > Merge: eb3f956 d5d747d > Author: Brian Foley > Date: Tue May 3 10:59:05 2016 +0100 > > Merge branch

[MERGE] stable-2.15 to stable-2.16

2016-05-03 Thread 'Brian Foley' via ganeti-devel
commit 3afc662c93d2d5292874b76fcbb6aa7eb1b74f57 Merge: eb3f956 d5d747d Author: Brian Foley Date: Tue May 3 10:59:05 2016 +0100 Merge branch 'stable-2.15' into stable-2.16 * stable-2.15 Use socat method string compatible with <1.73 & >=1.73 Reduce

[PATCH stable-2.16] Implement localized cache for lvs commands

2016-05-03 Thread 'Federico Morg Pareschi' via ganeti-devel
This patch optimizes unnecessary calls to the lvs command when running gnt-cluster verify (among other operations) by caching multiple series of lvs statements into a single operation and storing the results in a dictionary with device paths used as keys. Signed-off-by: Federico Morg Pareschi

[PATCH master] Configurable ceph authentication user id

2016-05-03 Thread David Mohr
On our cluster we did not want to deploy the ceph admin keyring everywhere, so we created a new identity for ganeti. This patch adds the configuration option to specify the user id to be passed to rbd commands. Comments are welcome, I will update the patch as needed. David Mohr (1): New

[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 |