Re: [PATCH stable-2.6 4/6] Human readable hcheck functionality

2012-06-25 Thread René Nussbaumer
On Mon, Jun 25, 2012 at 10:26 AM, Agata Murawska agatamuraw...@google.com wrote: On Fri, Jun 22, 2012 at 1:39 PM, René Nussbaumer r...@google.com wrote: On Fri, Jun 22, 2012 at 10:46 AM, Agata Murawska agatamuraw...@google.com wrote: Full hcheck functionality is implemented in this patch.

Re: [PATCH stable-2.6 2/6] Generalize functions commonly used between hspace and hcheck

2012-06-25 Thread Agata Murawska
On Fri, Jun 22, 2012 at 11:32 AM, René Nussbaumer r...@google.com wrote: On Fri, Jun 22, 2012 at 10:46 AM, Agata Murawska agatamuraw...@google.com wrote: As hspace and hcheck both use machine readable options, they require similar functions - which are now generalized and moved to CLI and

Re: [PATCH master] Add summary field to OpNodeModifyStorage

2012-06-25 Thread René Nussbaumer
On Mon, Jun 25, 2012 at 1:14 PM, Iustin Pop ius...@google.com wrote: ---  lib/opcodes.py |    1 +  1 files changed, 1 insertions(+), 0 deletions(-) LGTM! René

[PATCH master] Release Ganeti 2.6.0rc1

2012-06-25 Thread René Nussbaumer
--- configure.ac |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/configure.ac b/configure.ac index d2b2483..2a50634 100644 --- a/configure.ac +++ b/configure.ac @@ -2,7 +2,7 @@ m4_define([gnt_version_major], [2]) m4_define([gnt_version_minor], [6])

Re: [PATCH master] Release Ganeti 2.6.0rc1

2012-06-25 Thread Iustin Pop
On Mon, Jun 25, 2012 at 03:41:36PM +0200, René Nussbaumer wrote: --- configure.ac |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/configure.ac b/configure.ac index d2b2483..2a50634 100644 --- a/configure.ac +++ b/configure.ac @@ -2,7 +2,7 @@

Disks are not getting annotated before passed to bdev

2012-06-25 Thread Constantinos Venetsanopoulos
Hello team, I'm testing Ganeti 2.6.0beta2 and seem to have hit a problem: Disk parameters are not getting annotated before several rpc calls inside cmdlib, which causes Ganeti to break in templates that need to find disk parameters during their Attach or Remove methods. This does not cause

[PATCH stable-2.6 1/3] Update to the install file - hinotify dependency

2012-06-25 Thread Agata Murawska
Hinotify was not marked as a dependency for hconfd. Signed-off-by: Agata Murawska agatamuraw...@google.com --- INSTALL |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/INSTALL b/INSTALL index db29b26..bd62a91 100644 --- a/INSTALL +++ b/INSTALL @@ -159,15 +159,17 @@

[PATCH stable-2.6 2/3] Fixes in man page for hinfo

2012-06-25 Thread Agata Murawska
Signed-off-by: Agata Murawska agatamuraw...@google.com --- man/hinfo.rst |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/man/hinfo.rst b/man/hinfo.rst index d5109dd..8169261 100644 --- a/man/hinfo.rst +++ b/man/hinfo.rst @@ -1,5 +1,5 @@ HINFO(1) Ganeti | Version

[PATCH stable-2.6 3/3] Fix for man entry for htools - added hinfo

2012-06-25 Thread Agata Murawska
Signed-off-by: Agata Murawska agatamuraw...@google.com --- man/htools.rst |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/man/htools.rst b/man/htools.rst index 95205d1..05bd3a7 100644 --- a/man/htools.rst +++ b/man/htools.rst @@ -24,6 +24,9 @@ SYNOPSIS **hscan**

Release of Ganeti 2.6.0 rc1

2012-06-25 Thread René Nussbaumer
Hi Ganetians, Ganeti 2.6.0 rc1 just has been released. A lot of bug fixes has made it into this release from beta 2: - Fixed daemon-util with non-root user models - Fixed creation of plain instances with --no-wait-for-sync - Fix wrong iv_names when running cfgupgrade - Export more information in

Re: [PATCH stable-2.6 3/3] Fix for man entry for htools - added hinfo

2012-06-25 Thread René Nussbaumer
On Mon, Jun 25, 2012 at 4:38 PM, Agata Murawska agatamuraw...@google.com wrote: Signed-off-by: Agata Murawska agatamuraw...@google.com --- LGTM René

Re: Release of Ganeti 2.6.0 rc1

2012-06-25 Thread René Nussbaumer
Hi again On Mon, Jun 25, 2012 at 4:44 PM, René Nussbaumer r...@google.com wrote: Ganeti 2.6.0 rc1 just has been released. A lot of bug fixes has made it into this release from beta 2: Sorry, I forgot the link to the tarball: http://ganeti.googlecode.com/files/ganeti-2.6.0~rc1.tar.gz Enjoy :)

Re: [PATCH stable-2.6 2/3] Fixes in man page for hinfo

2012-06-25 Thread René Nussbaumer
On Mon, Jun 25, 2012 at 4:38 PM, Agata Murawska agatamuraw...@google.com wrote: Signed-off-by: Agata Murawska agatamuraw...@google.com --- LGTM René

Re: [PATCH stable-2.6 1/3] Update to the install file - hinotify dependency

2012-06-25 Thread René Nussbaumer
On Mon, Jun 25, 2012 at 4:38 PM, Agata Murawska agatamuraw...@google.com wrote: Hinotify was not marked as a dependency for hconfd. Signed-off-by: Agata Murawska agatamuraw...@google.com --- LGTM! Thanks a lot. René

Re: [PATCH master 1/2] QA function do destroy disks

2012-06-25 Thread Bernardo Dal Seno
On Fri, Jun 22, 2012 at 6:11 PM, Michael Hanselmann han...@google.com wrote: 2012/6/20 Bernardo Dal Seno bdals...@google.com: +  # FIXME This works with no more than 2 secondaries +  re_nodelist = re.compile( +    r([^,()]+)(?:\s+\([^)]+\))?(?:,([^,()]+)(?:\s+\([^)]+\))?)?$) This seems very,

Re: [PATCH master 2/2] QA: Added test for gnt-instance recreate-disks

2012-06-25 Thread Bernardo Dal Seno
On Fri, Jun 22, 2012 at 6:13 PM, Michael Hanselmann han...@google.com wrote: 2012/6/20 Bernardo Dal Seno bdals...@google.com: --- a/qa/qa_instance.py +++ b/qa/qa_instance.py +def TestRecreateDisks(instance, pnode, snode, othernodes): +  gnt-instance recreate-disks + +  @param instance:

Re: [PATCH master 1/2] QA function do destroy disks

2012-06-25 Thread Bernardo Dal Seno
Also, fix for the docstring: diff --git a/qa/qa_instance.py b/qa/qa_instance.py index 4621543..d16033b 100644 --- a/qa/qa_instance.py +++ b/qa/qa_instance.py @@ -87,6 +87,7 @@ def _DestroyInstanceVolumes(instance): This is used to simulate HW errors (dead nodes, broken disks...); the