Prepare NEWS file for 2.12.0

Mention inherited changes as well as the fixes made to stable-2.12
since rc2.

Signed-off-by: Petr Pudlak <pudlak@google.com>
Reviewed-by: Klaus Aehlig <aehlig@google.com>
Suffix bump for 2.12.0

Signed-off-by: Petr Pudlak <pudlak@google.com>
Reviewed-by: Klaus Aehlig <aehlig@google.com>
Generate domain-name unique names in genEmptyCluster

It is a prerequisite that all nodes in a cluster have different names.
In genEmptyCluster, this is achieved by adding a unique suffix to each
name. However, this still allows to have both, 'n-3' and 'n-3.n-4' as
names in the same cluster---and those names are considered equal, as
they might both the legitimate short hands of 'n-3.n-4.example.com'.

To avoid these kinds of name collisions, make genEmptyCluster only
use the first part of the FQDN and suffix it.

Signed-off-by: Klaus Aehlig <aehlig@google.com>
Reviewed-by: Helga Velroyen <helgav@google.com>
Improve retries on RPC communication to daemons

Before the communication was retried "Broken Pipe".

Now retry it on any network error and wait between the retries (to
account for situations like a deamon restart).

Rename the function to better capture the new semantics.

Signed-off-by: Petr Pudlak <pudlak@google.com>
Reviewed-by: Helga Velroyen <helgav@google.com>
Also debug-log the end of local config writes

...to better estimate the cost of it.

Signed-off-by: Klaus Aehlig <aehlig@google.com>
Reviewed-by: Helga Velroyen <helgav@google.com>
Update the release date and add a fixed bug in NEWS

Signed-off-by: Petr Pudlak <pudlak@google.com>
Reviewed-by: Klaus Aehlig <aehlig@google.com>
.gitignore scripts/gnt-filter

This command was added in d8f5462, but the corresponding generated file
was not added to the list of files to be ignored. Do this now.

Signed-off-by: Klaus Aehlig <aehlig@google.com>
Reviewed-by: Petr Pudlak <pudlak@google.com>
Merge branch 'stable-2.11' into stable-2.12

* stable-2.11
  Generate domain-name unique names in genEmptyCluster

Signed-off-by: Klaus Aehlig <aehlig@google.com>
Reviewed-by: Petr Pudlak <pudlak@google.com>
Merge branch 'stable-2.12' into master

* stable-2.11
  Generate domain-name unique names in genEmptyCluster

* stable-2.12
  Update the release date and add a fixed bug in NEWS
  Improve retries on RPC communication to daemons
  Suffix bump for 2.12.0
  Prepare NEWS file for 2.12.0

Conflicts:
	NEWS: take ALL the entries
	configure.ac: ignore suffix bump

Signed-off-by: Klaus Aehlig <aehlig@google.com>
Reviewed-by: Petr Pudlak <pudlak@google.com>
Add systemd unit files to EXTRA_DIST

This patch add systemd unit files used by ganeti-*.service to
EXTRA_DIST.

Signed-off-by: Jun Futagawa <jfut@integ.jp>
Reviewed-by: Apollon Oikonomopoulos <apoikos@gmail.com>
Signed-off-by: Klaus Aehlig <aehlig@google.com>
Reviewed-by: Klaus Aehlig <aehlig@google.com>
Set SO_REUSEADDR in ConfD

Signed-off-by: Klaus Aehlig <aehlig@google.com>
Reviewed-by: Petr Pudlak <pudlak@google.com>
Readd instance removal

The cleanup that was removed in commit c57cdced still needs to exist,
albeit outside the finally block.

Signed-off-by: Hrvoje Ribicic <riba@google.com>
Reviewed-by: Klaus Aehlig <aehlig@google.com>
Break up parallel performance QA tests

With all the parallel performance QA tests being run under the same
"parallel-performance" test name, activating individual ones for
debugging was impractical. This patch introduces separate names for
the QA tests.

Signed-off-by: Hrvoje Ribicic <riba@google.com>
Reviewed-by: Klaus Aehlig <aehlig@google.com>
Make parallel performance rename test more serial

While the rename operation still acquired the BGL lock, the parallel
performance rename test was effectively serialized. With the removal
of the BGL, the second rename fails as it hits a check for the
existence of the renamed instance before the rename is done.
This patch explicitly serializes the two operations by making them run
on success rather then submitting them almost immediately.

Signed-off-by: Hrvoje Ribicic <riba@google.com>
Reviewed-by: Klaus Aehlig <aehlig@google.com>
Change monitoring test invocation

This patch changes the invocation of the monitoring tests so they would
accurately show when a test has been skipped.

Signed-off-by: Hrvoje Ribicic <riba@google.com>
Reviewed-by: Klaus Aehlig <aehlig@google.com>
Remove check for "complex disk layout" from LUMoveInstance

The check is unnecessary because earlier the disk template is checked in
a more restrictive way. Newly the check works on disk types, but gets
rid of the redundancy.

Signed-off-by: Aaron Karper <akarper@google.com>
Signed-off-by: Michele Tartara <mtartara@google.com>
Reviewed-by: Michele Tartara <mtartara@google.com>
Add documentation for LUInstance{Move,Failover,Migrate}

The documentation explains how LUMoveInstance, LUInstanceFailover, and
LUInstanceMigrate are related.

Signed-off-by: Aaron Karper <akarper@google.com>
Signed-off-by: Michele Tartara <mtartara@google.com>
Reviewed-by: Michele Tartara <mtartara@google.com>
Add a stable version of cover

Work around the limitation of QuickChecks `cover` to ignore
all coverage labels that do not occur in the actual sample.

Signed-off-by: Klaus Aehlig <aehlig@google.com>
Reviewed-by: Petr Pudlak <pudlak@google.com>
Reviewed-by: Niklas Hambuechen <niklash@google.com>
reaname variables to avoid name shadowing

...as name shadowing is a fatal warning in our style.
Enjoy the more readable one character variable names
as in the QuickCheck source code.

Signed-off-by: Klaus Aehlig <aehlig@google.com>
Reviewed-by: Petr Pudlak <pudlak@google.com>
Test: Rename genCount to genSlotCount

Describes better what it does.

Signed-off-by: Niklas Hambuechen <niklash@google.com>
Reviewed-by: Klaus Aehlig <aehlig@google.com>