Browse code
Remove lib/tempest errexit handling
lib/tempest had code to set -o errexit and restore the setting after
configure_tempest had run. This code didn't work, but it is no longer
necessary since we set errexit for all of devstack (see change 88521;
Change-Id Ia477e04567046a7fcdedf699423165bc577b72ad).
This changeset removes the errexit handling code from lib/tempest.
Change-Id: I2b0bc3c55a5d9bbb688f1e2e8e0fe133830a4172
Roger Luethi authored on 2014/04/30 14:03:24Showing 1 changed files
| ... | ... |
@@ -76,7 +76,6 @@ function configure_tempest {
|
| 76 | 76 |
local num_images |
| 77 | 77 |
local image_uuid |
| 78 | 78 |
local image_uuid_alt |
| 79 |
- local errexit |
|
| 80 | 79 |
local password |
| 81 | 80 |
local line |
| 82 | 81 |
local flavors |
| ... | ... |
@@ -92,11 +91,6 @@ function configure_tempest {
|
| 92 | 92 |
# TODO(afazekas): |
| 93 | 93 |
# sudo python setup.py deploy |
| 94 | 94 |
|
| 95 |
- # This function exits on an error so that errors don't compound and you see |
|
| 96 |
- # only the first error that occurred. |
|
| 97 |
- errexit=$(set +o | grep errexit) |
|
| 98 |
- set -o errexit |
|
| 99 |
- |
|
| 100 | 95 |
# Save IFS |
| 101 | 96 |
ifs=$IFS |
| 102 | 97 |
|
| ... | ... |
@@ -383,8 +377,6 @@ function configure_tempest {
|
| 383 | 383 |
|
| 384 | 384 |
# Restore IFS |
| 385 | 385 |
IFS=$ifs |
| 386 |
- #Restore errexit |
|
| 387 |
- $errexit |
|
| 388 | 386 |
} |
| 389 | 387 |
|
| 390 | 388 |
# create_tempest_accounts() - Set up common required tempest accounts |