14 Commits

Author SHA1 Message Date
Ian Wienand
aee18c749b Enforce function declaration format in bash8
Check that function calls look like ^function foo {$ in bash8, and fix
all existing failures of that check.  Add a note to HACKING.rst

Change-Id: Ic19eecb39e0b20273d1bcd551a42fe400d54e938
2014-02-28 07:59:03 +11:00
Kiall Mac Innes
a16c821024 Skip Nova exercises if Nova is not enabled
This allows for ./exercises.sh to complete sucessfully when nova is not
enabled / installed.

Change-Id: If969e14f5106c15007146e8fad1da27d131828c8
2014-01-12 19:51:33 +00:00
Jenkins
02ef0dbfb0 Merge "add assertions for blind grep" 2013-12-05 00:19:49 +00:00
Adam Spiers
6a5aa7c6a2 Fix some Markdown formatting issues
Address miscellaneous issues with Markdown formatting in comments which
are consumed by shocco when generating the online documentation.

Change-Id: I953075cdbddbf1f119c6c7e35f039e2e54b79078
2013-10-24 17:38:19 +01:00
DennyZhang
23178a997a add assertions for blind grep
blind grep is error-prone.
Add assertions for errors we can not handle

Change-Id: Ibe19085545ecc848498506e8b8ee14e71825b273
2013-10-22 17:12:18 -05:00
Sahid Orentino Ferdjaoui
75e851a6de exercices: aggregates needs to be more flexible
The actual regex checks a result in python format and
because of the change in the bug 1132961, Jekins failed.

I have update the regex to work with the old result
and the new result.

Change-Id: I393e1358f99be5f20d9ac8b3e214355a453ecfcb
Closes-Bug: 1239726
2013-10-16 08:41:22 +00:00
Chris Behrens
c62c2b9b66 Fix various things when n-cell is enabled
1) Use the newer cell_type configuration option instead of overriding
the compute_api_class.

2) A nova-cells service is started for both the API cell (region) and
the compute cell (child).  The screen names were both 'n-cell' which
caused both services to log to the same log file.  The screen names have
been changed to n-cell-region and n-cell-child.

3) Security groups are not supported with cells.  Multiple exercises are
patched to use the 'default' security group instead of creating a new
one.  Additionally, if one uses the 'default' security group, do not try
to delete it, as one can never delete the default.

4) Disable the floating_ips and aggregates exercises when n-cell is
enabled, as they are not supported by cells.

5) Related to #4, disable the floating_ips tests within euca.sh

6) Update the README.md.  No services need to be disabled, and one only
needs to enable the n-cell service.

Change-Id: I9782d1e3cda3c9dd3daefa15c043f5b06473cb87
2013-07-26 11:11:25 -07:00
Nachi Ueno
07115eb5c0 Refactor error logging
It is hard to grep errors in current log. so in this patch,
I'm updating die function which also writes log for
screen_log_dir/error.log.
In future, we may categolize negative fault by using
this error.log.

Change-Id: I70a8cfe67ed408284f5c88c762c6bb8acb8ecdb2
2013-02-28 11:54:24 -08:00
Mate Lakat
c497a05ea2 Fix aggregates exercise
Fixes bug 1131244

aggregates.sh was making assumptions on the working directory. Thus it
failed, if the current working directory is not devstack. This pathch
makes the sourcing to use the full path for openrc.

Change-Id: I10c83c88112ead97b43873e1034b0e76e4f889f4
2013-02-21 15:25:51 +00:00
Dean Troyer
da85cdadd9 Exercise cleanup
* Make common steps consistent
* Make comments consistent

aggregates.sh
boot_from_volume.sh
client-args.sh
client-env.sh
euca.sh
floating_ips.sh
sec_groups.sh
swift.sh
volumes.sh

Change-Id: Ib93dcdfdead93c259e3cd184fbc5ccc0a4a87c9a
2013-02-15 12:34:33 -06:00
Joe Gordon
5c1bedd1ed Update aggregates test for aggregates bp
Instead of implementing availability zones in the service table,
availability zones will be implemented using general aggregate metadata.

So when an aggregate is created it will already have metadata.

Part of blueprint aggregate-based-availability-zones

Change-Id: I0fd22399b99a14087fef63fc91d0baef746efbed
2012-12-12 12:20:01 +00:00
Mate Lakat
178b8402d9 Fix aggregates test with multi host setup
Fixes bug 1046222.

The aggregate exercise assumed that you have only one compute node, thus
it failed with syntax error for two hosts. With this fix, the exercise
will pick the first compute host, and use that for the tests.

Change-Id: I85d76552295d640e1a9d86fbbed781f15529d047
2012-09-05 10:42:10 +01:00
Joe Gordon
b7ef539b12 Improve exercises/aggregates.sh
* Update for blueprint general-host-aggregates
* Test for add/remove hosts
* Now uses nova host-list

Change-Id: Id6fef649c13032cf9148d7152fa2b28654717892
2012-08-01 17:18:39 -07:00
John Garbutt
fd1c87e830 blueprint host-aggregates
add some inital tests for the host-aggregates blueprint

Change-Id: I0c07f2e7fd123bbda6d26f4ff64bea3949e57157
2012-03-28 18:24:52 -05:00