- May 17, 2013
-
-
Balazs Lecz authored
The OS log dir is used by noded, not masterd. Signed-off-by:
Balazs Lecz <leczb@google.com> Reviewed-by:
Guido Trotter <ultrotter@google.com>
-
Bernardo Dal Seno authored
Also document a couple more fixes. Signed-off-by:
Bernardo Dal Seno <bdalseno@google.com> Reviewed-by:
Thomas Thrainer <thomasth@google.com>
-
Thomas Thrainer authored
The master IP setup script now checks if the master IP is actually configured on the machine before trying to remove the IP. This fixes issue 460. Signed-off-by:
Thomas Thrainer <thomasth@google.com> Reviewed-by:
Guido Trotter <ultrotter@google.com>
-
- May 14, 2013
-
-
Bernardo Dal Seno authored
So far QA only recreated the whole set of disks at once. Signed-off-by:
Bernardo Dal Seno <bdalseno@google.com> Reviewed-by:
Helga Velroyen <helgav@google.com>
-
Bernardo Dal Seno authored
Just a very basic test that adds and then removes a disk. Signed-off-by:
Bernardo Dal Seno <bdalseno@google.com> Reviewed-by:
Helga Velroyen <helgav@google.com>
-
Bernardo Dal Seno authored
cmdlib.LUInstanceSetParams now uses helper functions to create and wipe disks, so that when the creation of a disk fails, any leftover device is cleaned up. As a bonus, exceptions raised by _CreateBlockDev() are caught correctly. Now cmdlib._CreateDisks() is used every time there are disks to create. Signed-off-by:
Bernardo Dal Seno <bdalseno@google.com> Reviewed-by:
Helga Velroyen <helgav@google.com>
-
Bernardo Dal Seno authored
Now even recreate-disks wipes the newly-created disks, if the flag is set. Signed-off-by:
Bernardo Dal Seno <bdalseno@google.com> Reviewed-by:
Helga Velroyen <helgav@google.com>
-
Bernardo Dal Seno authored
The wrapper handles errors by logging them and cleaning up freshly-created disks. Also, the correct disk is used in the error message when an error happens in cmdlib._CreateDisks() and the resulting disk clean-up fails. Signed-off-by:
Bernardo Dal Seno <bdalseno@google.com> Reviewed-by:
Helga Velroyen <helgav@google.com>
-
Bernardo Dal Seno authored
Since 9b221ea4, _CreateBlockDev() doesn't raise OpExecError any more. Yet some code was left in place to catch it. By removing that code we have two advantages: 1. Dead code is removed. 2. If for whatever reason _CreateBlockDev() raises OpExecError, the exception is not silently dropped and we notice (so we can fix it). Signed-off-by:
Bernardo Dal Seno <bdalseno@google.com> Reviewed-by:
Guido Trotter <ultrotter@google.com>
-
Michael Hanselmann authored
In issue 353 Sascha Lucas reported that disks are not wiped when added through “gnt-instance modify”. This patch adds this functionality and updates the docstring for “_WipeDisks”. Signed-off-by:
Michael Hanselmann <hansmi@google.com> Reviewed-by:
Iustin Pop <iustin@google.com> (cherry picked from commit 965e0e6a) Signed-off-by:
Bernardo Dal Seno <bdalseno@google.com> Reviewed-by:
Guido Trotter <ultrotter@google.com>
-
- May 07, 2013
-
-
Thomas Thrainer authored
Starting from Debian Wheezy, /var/run is a symlink to /run. If rsync does not treat receiving symlinks to directories as directories, it would remove the original symlink and create a new, empty directory where only the files to sync are put in. In the case of /var/run, this behavior severly impacts the running system (as, for example, several PID can't be found anymore). This is a cherry-pick of commit 412e7387. Signed-off-by:
Thomas Thrainer <thomasth@google.com> Reviewed-by:
Klaus Aehlig <aehlig@google.com>
-
- May 03, 2013
-
-
Guido Trotter authored
Signed-off-by:
Guido Trotter <ultrotter@google.com> Reviewed-by:
Klaus Aehlig <aehlig@google.com> Reviewed-by:
Helga Velroyen <helgav@google.com>
-
Helga Velroyen authored
This fixes issue 432. If a job summary contained a non-ascii character, gnt-job list crashed badly so far. This patch fixes the crash and makes gnt-job list display unicode characters properly. Signed-off-by:
Helga Velroyen <helgav@google.com> Reviewed-by:
Guido Trotter <ultrotter@google.com>
-
- May 02, 2013
-
-
Helga Velroyen authored
This fixes issue 432: when setting the volume group name, the presence of the volume group was checked on all nodes. This patch now restricts the check only to the vm-capable ones. Signed-off-by:
Helga Velroyen <helgav@google.com> Reviewed-by:
Guido Trotter <ultrotter@google.com>
-
Helga Velroyen authored
Inter-cluster instance moves depends on exporting an instance. For file-based instance, exporting is not supported. Therefore, this patch disables inter-cluster instance moves for file-based instances and informs the user with an expressive error message. Signed-off-by:
Helga Velroyen <helgav@google.com> Reviewed-by:
Michele Tartara <mtartara@google.com>
-
Helga Velroyen authored
Currently, when creating an instance with file storage fails, the error message is rather general. This patch adds a hint on what to do as a possible fix. Signed-off-by:
Helga Velroyen <helgav@google.com> Reviewed-by:
Guido Trotter <ultrotter@google.com>
-
Guido Trotter authored
Move the "main" 2.7 section up to rc1, and just leave the delta in beta3. Then add a new beta3 to rc1 delta. Signed-off-by:
Guido Trotter <ultrotter@google.com> Reviewed-by:
Michele Tartara <mtartara@google.com>
-
Guido Trotter authored
- Mention the file-storage-paths config file - Add information about sharedfile and ext storage types - Change one manpage reference to a proper link Signed-off-by:
Guido Trotter <ultrotter@google.com> Reviewed-by:
Michele Tartara <mtartara@google.com>
-
Guido Trotter authored
It was in NEWS, but not mentioned at all in the upgrade notes. Signed-off-by:
Guido Trotter <ultrotter@google.com> Reviewed-by:
Klaus Aehlig <aehlig@google.com>
-
Guido Trotter authored
-enable-kvm and -disable-kvm are deprecated in favor of accel=kvm passed as a machine specification. This fixes compatibility with newer qemu versions. Signed-off-by:
Guido Trotter <ultrotter@google.com> Reviewed-by:
Helga Velroyen <helgav@google.com>
-
Guido Trotter authored
We had parameter validation when the runtime is generated, but in the middle of the function. Move up, before any parameter is accessed. Signed-off-by:
Guido Trotter <ultrotter@google.com> Reviewed-by:
Thomas Thrainer <thomasth@google.com>
-
Guido Trotter authored
Newer versions of kvm (from 1.4) don't support -nographic with -daemonize. When -display exists, use -display none instead. This fixes Issue 389. Signed-off-by:
Guido Trotter <ultrotter@google.com> Reviewed-by:
Thomas Thrainer <thomasth@google.com>
-
- May 01, 2013
-
-
Guido Trotter authored
- Remove mention of python 2.4 specific modules, as that version is not supported anymore - Add bitarray and ipaddr - Remove a double space - Add mandatory C library packages too - Mention the option of using drbd modules directly from your kernel This also fixes Issue 372. Signed-off-by:
Guido Trotter <ultrotter@google.com> Reviewed-by:
Helga Velroyen <helgav@google.com>
-
- Apr 30, 2013
-
-
Michele Tartara authored
Mon-collector kept being compiled even if monitoring was disabled during configure. Fixes Issue 441. Signed-off-by:
Michele Tartara <mtartara@google.com> Reviewed-by:
Guido Trotter <ultrotter@google.com>
-
Michele Tartara authored
Some data collectors require confd data information. Therefore, monitoring is dependent on confd, but this was not checked. This commit introduces such a check and partially fixes Issue 441. Signed-off-by:
Michele Tartara <mtartara@google.com> Reviewed-by:
Helga Velroyen <helgav@google.com>
-
Iustin Pop authored
Since we use the primitive string type for group UUIDs, the group fields have a bug where we pass the group name as filter for node tests, whereas the nodes themselves use the group UUID. This results in zero node count, empty node list, and no instances being reported as assigned to groups. The patch fixes this and adds a test for the node count. It does some test generation improvement, and also cleans up whitespace issues in Test/G/Q/Query.hs (the functions case_queryNode_allfields, prop_queryGroup_noUnknown and case_queryGroup_allfields are unchanged but simply have indentation fixed). Signed-off-by:
Iustin Pop <iustin@google.com> Reviewed-by:
Guido Trotter <ultrotter@google.com> Cherry-pick of e7124835, fixes issue 436 Signed-off-by:
Klaus Aehlig <aehlig@google.com> Reviewed-by:
Guido Trotter <ultrotter@google.com> Conflicts: test/hs/Test/Ganeti/Objects.hs test/hs/Test/Ganeti/Query/Query.hs
-
Helga Velroyen authored
This patch extends the man page of the 'gnt-instance' command by describing the newly implemented hypervisor parameter 'vif_type'. Signed-off-by:
Helga Velroyen <helgav@google.com> Reviewed-by:
Guido Trotter <ultrotter@google.com>
-
Helga Velroyen authored
This patch fixes issue 247. It empowers the user to specify explicitly the vif type of a nic configuration of a Xen HVM instance. This includes the option of setting it to '' on the commandline, causing Ganeti to leave out the 'type' parameter in the vif configuration completely. Signed-off-by:
Helga Velroyen <helgav@google.com> Reviewed-by:
Guido Trotter <ultrotter@google.com>
-
- Apr 29, 2013
-
-
Klaus Aehlig authored
So far, any successful query was only logged at debug level, thus leaving no traces in the logs, unless debugging was enabled. Therefore, we add a notes about successful completion at info level. Also, repeat a failed request at WARN level, so that, even if debugging disabled, the request is available together with the error message in the logs. Signed-off-by:
Klaus Aehlig <aehlig@google.com> Reviewed-by:
Helga Velroyen <helgav@google.com>
-
Klaus Aehlig authored
Signed-off-by:
Klaus Aehlig <aehlig@google.com> Reviewed-by:
Helga Velroyen <helgav@google.com>
-
Thomas Thrainer authored
Explain the relationship of the ip and network parameter in `gnt-instance add --net` better. Signed-off-by:
Thomas Thrainer <thomasth@google.com> Reviewed-by:
Guido Trotter <ultrotter@google.com>
-
Guido Trotter authored
The current error message ends up saying something like '192.168.5.5' != 'network1', which is sort of non-intuitive (yes, of course they are different). Fix this by explaining better what the erroneous condition actually is. Signed-off-by:
Guido Trotter <ultrotter@google.com> Reviewed-by:
Thomas Thrainer <thomasth@google.com>
-
- Apr 26, 2013
-
-
Klaus Aehlig authored
Currently, if converting an instance from plain to drbd fails after renaming the original LVs, the instance is left in an inconsistent state. This commit tries to undo the renaming if a failure occurs on assembling a DRBD device, e.g., when device minor numbers are exhausted. (Issue 229) Signed-off-by:
Klaus Aehlig <aehlig@google.com> Reviewed-by:
Thomas Thrainer <thomasth@google.com>
-
- Apr 25, 2013
-
-
Guido Trotter authored
Currently this is checked only when distcheck-release is called. Check it in check-news instead, for all versions before the "current" one (as defined by configure.ac) Signed-off-by:
Guido Trotter <ultrotter@google.com> Reviewed-by:
Michele Tartara <mtartara@google.com>
-
- Apr 24, 2013
-
-
Bernardo Dal Seno authored
Signed-off-by:
Bernardo Dal Seno <bdalseno@google.com> Reviewed-by:
Klaus Aehlig <aehlig@google.com>
-
- Apr 23, 2013
-
-
Guido Trotter authored
The renew-crypto entry had a few obscure sentences. Improve it by rephrasing it. Signed-off-by:
Guido Trotter <ultrotter@google.com> Reviewed-by:
Klaus Aehlig <aehlig@google.com>
-
- Apr 22, 2013
-
-
Guido Trotter authored
Signed-off-by:
Guido Trotter <ultrotter@google.com> Reviewed-by:
Michele Tartara <mtartara@google.com>
-
Guido Trotter authored
- Most entries for beta1 are moved to the top level - The beta1 to beta2 delta is left Signed-off-by:
Guido Trotter <ultrotter@google.com> Reviewed-by:
Michele Tartara <mtartara@google.com>
-
Michele Tartara authored
During the creation of an instance, if the creation of disks fails, some partially created disks might remain lying around. There is an already implemented cleanup procedure, but it requires the disks to be listed in the disks_created variable, which is updated after the creation of the disk itself to prevent it from removing disks already existing previously. This patch introduces a better tracking of partially created disks, so that they can be removed without risking removing already existing ones. Signed-off-by:
Michele Tartara <mtartara@google.com> Reviewed-by:
Guido Trotter <ultrotter@google.com>
-
- Apr 18, 2013
-
-
Michele Tartara authored
Specify that the need to manually modify the metavg only applies to clusters with DRBD enabled, and is not always required. Signed-off-by:
Michele Tartara <mtartara@google.com> Reviewed-by:
Guido Trotter <ultrotter@google.com>
-