Statistics
| Branch: | Tag: | Revision:

root / qa / qa_instance.py @ 6a654276

History | View | Annotate | Download (24.5 kB)

# Date Author Comment
a77e3d33 02/08/2013 03:35 pm Michael Hanselmann

QA: Release instance in ganeti-qa

Release instances directly from “ganeti-qa” instead of doing so in the
three tests removing instances. This is in line with how nodes are
released and makes it easier to find and debug double-release issues.

Signed-off-by: Michael Hanselmann <>...

f006f110 02/04/2013 05:40 pm Bernardo Dal Seno

QA: Clean up after "instance-remove-drbd-offline" test

This test used to leave a node with orphan logical volumes and unused DRBD
minors. Now, the expected leftovers are cleaned up after the test, and
cluster-verify is run.

Signed-off-by: Bernardo Dal Seno <>...

5de31440 02/01/2013 12:25 pm Bernardo Dal Seno

QA: Instance tests check the configuration before running

Instance test functions check the current configuration, and they will run
the actual tests only if the configuration supports them. This will be
used for refactoring in following patches.

Signed-off-by: Bernardo Dal Seno <>...

c99200a3 02/01/2013 12:25 pm Bernardo Dal Seno

QA: Use lists of nodes as argument to instance tests

Some instance test functions took two node arguments, some took one, and
some took two but the second argument could be None. This patch makes such
functions uniform by using a list of nodes as an argument. This simplifies...

deaa347b 02/01/2013 12:20 pm Bernardo Dal Seno

QA fix: TestInstanceExportWithRemove() updates configuration

When the instance is removed from the cluster by this test, it's also freed
in the QA configuration.

Signed-off-by: Bernardo Dal Seno <>
Reviewed-by: Michael Hanselmann <>

e4c346a5 02/01/2013 12:20 pm Bernardo Dal Seno

QA cleanup: Removed instance-disk-failure test

The test was broken, out of sync with the rest of the code, and prone to
crashes. Until someone does a better job, it's better to remove the
test. This simplifies further refactoring.

Signed-off-by: Bernardo Dal Seno <>...

906a0346 02/01/2013 12:20 pm Bernardo Dal Seno

QA: Added disk template to config

The disk templates used by the instances are tracked in the QA
configuration. This will be used to selectively enable tests according to
the configuration.

Signed-off-by: Bernardo Dal Seno <>
Reviewed-by: Michael Hanselmann <>

1ba25bad 12/19/2012 07:26 pm Guido Trotter

Fix QA for --offline semantics modification

The previous patch series made it simpler to deal with ERROR_up offline
instances, but didn't fix qa, that had the old expectations. This fixes
that, and adds more testing about the expected behavior.

Signed-off-by: Guido Trotter <>...

b07afbb3 10/19/2012 05:55 pm Iustin Pop

Add a small QA check in instance renames for tags update

We also need to change the signature for _GetInstanceInfo, since we
don't have access to the instance dictionary in the rename instance
test.

Signed-off-by: Iustin Pop <>
Reviewed-by: Michael Hanselmann <>

e85be22a 10/11/2012 04:48 pm Bernardo Dal Seno

Add option to disable iallocator use in QA

By default some QA tests use iallocators. It is now possible to disable
iallocators in QA, so that QA doesn't force having iallocators set up.
The affected tests have now alternatives pathways.

Signed-off-by: Bernardo Dal Seno <>...

6a74f584 10/11/2012 12:38 pm Michael Hanselmann

Merge branch 'devel-2.6'

  • devel-2.6:
    verify-disks: Explicitely state nothing has to be done
    Add list of design documents implemented in version 2.6
    Better list of replace-disks arguments + typos fixed
    jqueue: Look at archived jobs when watching
    Show old primary/secondary node on disk replacement...
946e553b 10/08/2012 02:20 pm Bernardo Dal Seno

Fixed QA for replace-disks

Bug introduced in commit cc4b14f07: Part of a statement was missing. This
makes pylint-qa fail.

Signed-off-by: Bernardo Dal Seno <>
Reviewed-by: Michael Hanselmann <>

cc4b14f0 10/05/2012 07:23 pm Bernardo Dal Seno

Instance QA uses default iallocator

QA for recreate-disk and replace-disks now run also with the default
iallocator.

Signed-off-by: Bernardo Dal Seno <>
Reviewed-by: Michael Hanselmann <>

64be07b1 10/03/2012 06:00 pm Michael Hanselmann

gnt-instance reinstall: Don't always exit with success

If one or more jobs failed the exit status should be set accordingly.

Signed-off-by: Michael Hanselmann <>
Reviewed-by: Bernardo Dal Seno <>

320a5dae 09/27/2012 05:41 pm Bernardo Dal Seno

Always_failover doesn't require --allow-failover anymore

If an administrator sets always_failover, it means that there is no need
for another explicit approval to failover instead of migrating.

Signed-off-by: Bernardo Dal Seno <>
Reviewed-by: Michael Hanselmann <>...

304d9f02 09/18/2012 05:58 pm Michael Hanselmann

Migrate qa/*.py from constants to pathutils

File system paths moved from constants to pathutils.

Signed-off-by: Michael Hanselmann <>
Reviewed-by: Iustin Pop <>

d55408b0 09/14/2012 05:30 pm Bernardo Dal Seno

QA for instance migration made more compact

Also, the value of always_failover is not hard-coded any more, and it's
possible to disable changing the always_failover parameter, if needed.

Signed-off-by: Bernardo Dal Seno <>
Reviewed-by: Michael Hanselmann <>

b5f0b5cc 09/14/2012 05:30 pm Bernardo Dal Seno

Always_failover doesn't require --allow-failover anymore

If an administrator sets always_failover, it means that there is no need
for another explicit approval to failover instead of migrating.

Signed-off-by: Bernardo Dal Seno <>
Reviewed-by: Michael Hanselmann <>

a085d96d 07/25/2012 09:56 pm Bernardo Dal Seno

QA for allocator in recreate-disks

QA for recreate-disks called it twice with a list of node; the first call
now uses an allocator.

Signed-off-by: Adam Ingrassia <>
Signed-off-by: Bernardo Dal Seno <>
Reviewed-by: Iustin Pop <>

112aee5f 07/13/2012 06:00 pm Iustin Pop

Merge branch 'stable-2.6'

  • stable-2.6: (72 commits)
    Make Ganeti 2.6.0 rc3 release
    Allow reinstall even when secondaries are offline
    Prepare NEWS file for Ganeti 2.6.0 rc3
    QA updated to test instance removing when sec. is offline
    Ignore offline node errors when removing disks...
32da72f3 07/05/2012 07:11 pm Iustin Pop

Ensure that --wait-for-sync is used in QA

We don't have a specific test for activate disks, so let's add it in
the cases where we run (incidentally) activate-disks.

Signed-off-by: Iustin Pop <>
Reviewed-by: René Nussbaumer <>

a1cd2ecf 06/27/2012 12:10 pm Bernardo Dal Seno

Fixed doclint warning

Two blank lines are needed between functions.

Signed-off-by: Bernardo Dal Seno <>
Reviewed-by: René Nussbaumer <>

83180411 06/26/2012 07:17 pm Bernardo Dal Seno

QA: Added test for gnt-instance recreate-disks

The test is disabled by default, as it takes a long time.

Signed-off-by: Bernardo Dal Seno <>
Reviewed-by: René Nussbaumer <>
Reviewed-by: Michael Hanselmann <>

82d2f3df 06/26/2012 07:16 pm Bernardo Dal Seno

QA function do destroy disks

This is useful to simulate HW problems during QA.

Signed-off-by: Bernardo Dal Seno <>
Reviewed-by: René Nussbaumer <>
Reviewed-by: Michael Hanselmann <>

d488adb6 06/15/2012 04:48 pm Michael Hanselmann

QA: Test some instance parameters depending on hypervisor

For example, the “initrd_path” parameter is only available with xen-pvm,
whereas “boot_order” is only available with xen-hvm.

Signed-off-by: Michael Hanselmann <>
Reviewed-by: Iustin Pop <>

3bc145d8 06/12/2012 04:52 pm Bernardo Dal Seno

Fixed typos, formatting, etc and added a comment

This patch only touches comments and man pages.

Signed-off-by: Bernardo Dal Seno <>
Reviewed-by: Iustin Pop <>

1c573fb4 06/08/2012 04:58 pm Iustin Pop

Fix QA import of existing instances

Commit f346a7d9 broke the TestInstanceImport test in QA, since it
dropped the "always generate" parameter for backup import. We add an
override mac parameter so that, no matter what is defined in the
config file, imported instances (can) have a different MAC....

f346a7d9 06/07/2012 04:51 pm Michael Hanselmann

QA: Add configuration options for static MAC address

In some QA environments static MAC addresses must be used.

Signed-off-by: Michael Hanselmann <>
Reviewed-by: Iustin Pop <>

51131cad 05/14/2012 06:38 pm Michael Hanselmann

QA: Cleanup after commit 5fa0375e4a

Some tests had wrong assumptions on the instance status and some
needed explicit start or stop commands.

Signed-off-by: Michael Hanselmann <>
Reviewed-by: Iustin Pop <>

5fa0375e 05/10/2012 02:02 pm Michael Hanselmann

QA: Decorate tests with instance checks

Start using “InstanceCheck” decorator where appropriate. Use direct call
for check in places where the instance status changes during the test.

The parameters of “TestInstanceImport” had to be re-ordered to have the...

26a5056d 04/26/2012 06:41 pm Iustin Pop

Add grow-disk tests in QA

Signed-off-by: Iustin Pop <>
Reviewed-by: Michael Hanselmann <>

0fdf247d 04/19/2012 09:04 pm Michael Hanselmann

Convert listing exports to query2

This solves one case where locks are acquired during LUXI queries.
Pretty late into the transition I noticed that OpBackupQuery had a
“use_locking” parameter for a long time, but didn't use it. Since
most of the other changes were already and this allows exports to...

58f0ce16 02/01/2012 08:19 am Michael Hanselmann

LUInstanceSetParams: Allow no-op change of instance offline status

With this patch marking an instance already marked offline (or online)
as offline/online again becomes a no-op. Also removed the unused
INSTANCE_UP variable.

Signed-off-by: Michael Hanselmann <>...

3016bc1f 01/31/2012 07:44 am Michael Hanselmann

OpInstanceSetParams: Merge {off,on}line_inst parameters

Instead of having two separate parameters, a single boolean parameter is
used. Unfortunately we need a third state to say “no change”, so the
value can be None, True or False (similar to other parameters). There...

59d81cb3 01/19/2012 03:14 pm Michael Hanselmann

Merge branch 'devel-2.5'

  • devel-2.5:
    Fix wrong option names in QA and cluster-merge
    Bump version to 2.5.0~rc5, update NEWS
    Add UnescapeAndSplit unittest for multi-escapes
    Fix a bug in command line option parsing code
    cli: Disable abbreviation matching for options...
58ea8d17 01/19/2012 02:42 pm Michael Hanselmann

Fix wrong option names in QA and cluster-merge

Signed-off-by: Michael Hanselmann <>
Reviewed-by: Iustin Pop <>

4c1a464b 01/17/2012 12:08 pm Iustin Pop

Rework instance rename QA tests

After looking again at the QA speeds, it turned out that instance
rename was run 12 times via the command line and 12 times via RAPI;
outside of start/stop which we need for maintenance during QA, this is
the most used test....

42a769f9 12/08/2011 11:53 am Bernardo Dal Seno

Add tests for parameter "always_failover"

Signed-off-by: Bernardo Dal Seno <>
Signed-off-by: Iustin Pop <>
Reviewed-by: Iustin Pop <>

8ccbbe4b 11/22/2011 07:55 pm Guido Trotter

qa: use maximum and minimum memory

test modification of either parameter, but also both at once.

Signed-off-by: Guido Trotter <>
Reviewed-by: Iustin Pop <>

b459a848 08/30/2011 11:24 am Andrea Spadaccini

DeprecationWarning fixes for pylint

In version 0.21, pylint unified all the disable-* (and enable-*)
directives to disable (resp. enable). This leads to a lot of
DeprecationWarning being emitted even if one uses the recommended
version of pylint (0.21.1, as stated in devnotes.rst)....

d0c8c01d 07/25/2011 04:55 pm Iustin Pop

Most boring patch ever

s/'/"/ in (hopefully) the right places.

Signed-off-by: Iustin Pop <>
Reviewed-by: Michael Hanselmann <>

e9c487be 04/06/2011 04:03 pm René Nussbaumer

gnt-instance migrate: Adding --allow-failover option

Signed-off-by: René Nussbaumer <>
Reviewed-by: Iustin Pop <>

9026e935 03/15/2011 12:41 pm René Nussbaumer

TLReplaceDisks: Add check if disks are activated

Previously we failed later with a rather useless error message. This
patch fixes this and tells the user to activate-disks if replace-disks
is in the need of activated disks rather than abort with a cryptic error...

31fe5102 03/03/2011 05:45 pm René Nussbaumer

LUInstanceRename: Fail if renamed hostname mismatch

There's a problem if you run gnt-instance rename with a non FQDN and the
renamed LU tries to resolve the hostname to make it FQDN. It could be
that this resolved hostname was just a CNAME to another name which leads...

cc27265e 02/25/2011 02:25 pm René Nussbaumer

gnt-instance reboot start instance if not yet started

This patch starts the instance when gnt-instance reboot is invoked on a
instance already stopped.

Signed-off-by: René Nussbaumer <>
Reviewed-by: Michael Hanselmann <>

3582eef6 01/12/2011 12:41 pm Iustin Pop

Run pylint over QA code too

Right now, the QA code is not covered by pylint, and this shows at
least one low-impact bug.

This patch does the necessary changes to make QA pylint-clean, and the
changes the makefile to run pylint for it.

Notable changes:
...

2214cf14 12/14/2010 03:23 pm Michael Hanselmann

QA: Extend unittests for query operations, add tests for list-fields

Signed-off-by: Michael Hanselmann <>
Reviewed-by: René Nussbaumer <>

288d6440 12/10/2010 06:26 pm Michael Hanselmann

QA: Improve tests for instance/node list

- Query all known fields
- Random combinations (using a PRNG with a fixed seed) of fields
- Order of result names

Signed-off-by: Michael Hanselmann <>
Reviewed-by: Iustin Pop <>

e5c2accd 12/09/2010 02:37 pm Guido Trotter

Simplify instance rename qa test

The current instance rename qa testing function can only perform
back-and-forth renames, both for command line and rapi. In order to be
able to perform same-name rename tests we change it to be able to
perform simple renames, and then we change qa to call it to perform both...

2f4b4f78 11/29/2010 03:37 pm Iustin Pop

Simplify QA commands

Currently, 95% of the QA commands are executed in the same way: on the
master, based on a command list and with expectancies for succes:

AssertEqual(StartSSH(master['primary'],
utils.ShellQuoteArgs(cmd)).wait(), 0)...
6a343475 11/01/2010 05:02 pm Michael Hanselmann

QA: Test ssconf_instance_list file on rename and creation

This test would've caught the bug fixed in the previous patch.

Signed-off-by: Michael Hanselmann <>
Reviewed-by: René Nussbaumer <>

46f9a948 11/01/2010 05:02 pm Michael Hanselmann

Change qa_utils.ResolveInstanceName to take name

… instead of an object. Allows it to be used in places where
only the name is available.

Signed-off-by: Michael Hanselmann <>
Reviewed-by: René Nussbaumer <>

bc696589 08/10/2010 07:35 pm Michael Hanselmann

gnt-backup: Don't show confusing message w/o target node

“gnt-backup export” requires the target node. Until now, the master
daemon would complain that the “parameter
'OP_BACKUP_EXPORT.target_node' fails validation”. With this patch,
an additional check is done in the client program....

7fb50870 08/10/2010 06:56 pm Michael Hanselmann

QA: Test renaming instance via RAPI

Signed-off-by: Michael Hanselmann <>
Reviewed-by: Iustin Pop <>

938bde86 07/29/2010 04:05 pm Michael Hanselmann

QA: Test instance migration via CLI and RAPI

Signed-off-by: Michael Hanselmann <>
Reviewed-by: Iustin Pop <>

25e2fe5c 07/07/2010 11:06 am Iustin Pop

Merge branch 'devel-2.1'

  • devel-2.1:
    QA, burnin: allow selection of reboot types
    Add a QA option to disable reboots during burnin

Conflicts:
qa/qa-sample.json (trivial)
qa/qa_cluster.py (trivial)

Signed-off-by: Iustin Pop <>
Reviewed-by: Luca Bigliardi <>

1d103c02 07/07/2010 11:04 am Iustin Pop

QA, burnin: allow selection of reboot types

After some more investigation, only the soft reboot type fails for Xen
3.4 (due to the reboot/uptime time counter). As such, it's better to
allow selective testing, since we do want to test in general these
opcodes/the command line script....

801b6a33 07/01/2010 05:31 pm Guido Trotter

Merge branch 'stable-2.1'

  • stable-2.1:
    Bump up version for 2.1.5 release
    RapiClient: fix multi-authentication in Python 2.6
    Remove rapi-user and rapi-pass from qa-sample.json
    qa: fix gnt-instance modify -t drbd
    qa: shutdown instance before trying disk convert...
b73302c4 07/01/2010 02:54 pm Guido Trotter

qa: fix gnt-instance modify -t drbd

We need to pass the secondary node name, not a dict, which is an invalid
value.

Signed-off-by: Guido Trotter <>
Reviewed-by: Michael Hanselmann <>

6b92f2af 04/22/2010 07:12 pm Michael Hanselmann

Merge branch 'devel-2.1'

  • devel-2.1:
    Add a basic unittest for uidpool.RequestUnusedUid
    Factorize LUExportInstance
    Use ints instead of strings to represent user-ids
    QA: fix reimporting instance with different name
    Fix broken commit 9e302a8
    ssh: make quiet configurable...
53a8da8e 04/22/2010 03:09 pm Iustin Pop

QA: fix reimporting instance with different name

Since the original instance still exists, we need to use a different MAC
for its NIC.

Signed-off-by: Iustin Pop <>
Reviewed-by: Michael Hanselmann <>

b701a5df 03/15/2010 05:50 pm Iustin Pop

Merge remote branch 'origin/devel-2.1'

  • origin/devel-2.1:
    Implement QA tests for disk template changes
    Update instance modify documentation
    Implement conversion from drbd to plain
    Implement conversion from plain to drbd
    Abstract check that an instance is down...
7f69aabb 03/15/2010 05:46 pm Iustin Pop

Implement QA tests for disk template changes

The new test depends on the drbd type tests being enabled, and test
conversion to plain and back to drbd.

Signed-off-by: Iustin Pop <>
Reviewed-by: Michael Hanselmann <>

8a28f591 02/15/2010 07:04 pm Michael Hanselmann

Merge branch 'devel-2.1'

  • devel-2.1: (31 commits)
    Add unittest for utils._FingerprintFile
    Release all node locks during disk replace
    Unify a few re.compile calls in DRBD
    Auto-enable early release for offline old nodes
    Skip line-length warnings in man...
8d8d650c 02/12/2010 05:06 pm Michael Hanselmann

Add command line options for instance removal on export

Signed-off-by: Michael Hanselmann <>
Reviewed-by: Iustin Pop <>

18337ca9 02/09/2010 01:34 pm Iustin Pop

Implement instance rename QA tests

Signed-off-by: Iustin Pop <>
Reviewed-by: Michael Hanselmann <>

5645d16b 05/11/2009 05:18 pm Iustin Pop

Fix QA and documentation about no initrd case

In Ganeti 1.2, “none” was used to signify no initrd. In 2.0 we have
changed to “no_” as a prefix (i.e. “-H no_initrd_path”) and thus we
document in the manpage this.

The QA suite is changed accordingly.

Signed-off-by: Iustin Pop <>...

1d693311 12/01/2008 10:52 pm Michael Hanselmann

Update QA scripts to new cluster parameters

There are still issues, especially with "gnt-instance modify" and
resetting values. However, this is a start.

Reviewed-by: ultrotter

ed54b47e 08/15/2008 11:43 am Michael Hanselmann

Remove QA hook functionality

To my knowledge they're used nowhere and it's at least slightly
confusing to people adding new QA checks.

Reviewed-by: ultrotter

76f59a32 06/06/2008 12:32 pm Michael Hanselmann

Forward-port: Fix two problems in QA scripts

- Failover back to original node in instance failure test
- Exclude secondary node from list of potential nodes in
replace-disks test

Reviewed-by: iustinp

8a4e8898 06/06/2008 12:31 pm Michael Hanselmann

Forward-port: Add QA tests for “gnt-instance reboot”

Reviewed-by: ultrotter

7910e7a5 06/06/2008 12:31 pm Michael Hanselmann

Forward-port: Add QA test for “gnt-instance replace-disks”

Reviewed-by: iustinp

4379b1fa 03/27/2008 04:40 pm Michael Hanselmann

Add QA test for “gnt-instance console”

Reviewed-by: iustinp

f9193417 03/25/2008 06:30 pm Iustin Pop

Remove the option to create md/drbd7 instances

This patch removes the options that allow to create local_raid1 or
remote_raid1 instances. It also modifies the documentation and removes
these disk templates from burnin and from qa.

Reviewed-by: imsnah

29df1f02 02/25/2008 03:54 pm Michael Hanselmann

Some changes on disk failure tests

Change comments to printed information, some cleanup. These changes
are from November 2007. The test is not perfect yet.

Reviewed-by: iustinp

c68d1f43 02/14/2008 05:39 pm Michael Hanselmann

Code style updates for QA code.

Reviewed-by: iustinp

c0f74c55 01/28/2008 03:33 pm Iustin Pop

Add QA tests for gnt-instance modify

This patch adds QA tests for most of the possible parameters in the
instance modify operation (exception being the MAC), and modifies the
sample QA file to run this test.

It also tests the no-modification test, but that is a weak one: we only...

e0b62a26 01/21/2008 06:57 pm Guido Trotter

Remove qa tests for gnt-instance start/stop

Those tests were added in the wrong place. This patch removes them. One day
we'll implement proper command line regression testing and they should go in
there.

Reviewed-by: iustinp

ce9fb89d 01/21/2008 06:29 pm Guido Trotter

Test start/stop aliases in qa

This tests both that those two aliases have not been removed and also that
aliases handling hasn't been broken.

Reviewed-by: iustinp

f4bc1f2c 12/03/2007 04:03 pm Michael Hanselmann

Various code style fixes for strings.

- When line wrapping is needed, move spaces to the next line.
- Remove embedded line breaks from error messages.

Reviewed-by: schreiberal

7d7609a3 11/19/2007 08:38 pm Michael Hanselmann

Add basic QA tests for DRBD 8.

Reviewed-by: schreiberal

1e020d1b 11/13/2007 05:04 pm Michael Hanselmann

Fix --node option for “gnt-backup list”.

Reviewed-by: iustinp

e42b5307 11/08/2007 01:51 pm Michael Hanselmann

Insert hook calls.

Reviewed-by: schreiberal

113b8d89 11/06/2007 07:04 pm Michael Hanselmann

Adapt QA tests for removed --secondary-node option.

dfe11bad 11/01/2007 04:31 pm Michael Hanselmann

Cleanup colouring functions.

Make the code somewhat smaller. Disable disk failure test for master for now.

Reviewed-by: schreiberal

b1ffe1eb 11/01/2007 03:54 pm Michael Hanselmann

Cleanup QA scripts.

- Split main() function into several small ones.
- Current work on disk failure tests. This is not yet finished.
- Fix small typo in qa_node.py.

Reviewed-by: schreiberal

e8ae0c20 10/18/2007 04:02 pm Michael Hanselmann

Implement disk failure QA test.

Testing failure on the primary node is currently disabled due to drbd problems.

Reviewed-by: iustinp

283f9d4c 09/27/2007 04:00 pm Michael Hanselmann

Add more QA tests.

Added tests:
- “gnt-cluster getmaster”
- “gnt-cluster version”
- “gnt-instance list”
- “gnt-instance reinstall”

Reviewed-by: schreiberal

5d640672 09/26/2007 09:27 pm Michael Hanselmann

Enhance QA.

- Test “gnt-backup export” and “gnt-backup import”.
- Move “ResolveInstanceName” to qa_utils.py.
- Fix tests for “ganeti-watcher”.
- Make instance shutdown and startup configurable.

Reviewed-by: schreiberal

cec9845c 09/13/2007 03:46 pm Michael Hanselmann

Split QA script into different modules.

Reviewed-by: iustinp