diff options
-rw-r--r-- | tests/test_jjb_runner.py | 16 | ||||
-rw-r--r-- | turbo_hipster/task_plugins/real_db_upgrade/handle_results.py | 7 |
2 files changed, 18 insertions, 5 deletions
diff --git a/tests/test_jjb_runner.py b/tests/test_jjb_runner.py index fd60eb7..bdcee2d 100644 --- a/tests/test_jjb_runner.py +++ b/tests/test_jjb_runner.py @@ -26,21 +26,27 @@ from turbo_hipster.lib import utils class TestTaskRunner(base.TestWithGearman): log = logging.getLogger("TestTaskRunner") - def setUp(self): - super(TestTaskRunner, self).setUp() + def _grab_jjb(self): # Grab a copy of JJB's config temp_path = self.useFixture(fixtures.TempDir()).path cmd = 'git clone git://git.openstack.org/openstack-infra/config' utils.execute_to_log(cmd, '/dev/null', cwd=temp_path) - self.jjb_config_dir = os.path.join( + return os.path.join( temp_path, 'config', 'modules/openstack_project/files/jenkins_job_builder/config' ) - def test_job_can_shutdown_th(self): + def test_jjb_pep8_job(self): + self.skipTest("This is buggy atm.") + # We can only do this if we have the slave scripts installed in + # /usr/local/jenkins/slave_scripts/ + if not os.path.isdir('/usr/local/jenkins/slave_scripts/'): + self.skipTest("Slave scripts aren't installed") + + jjb_config_dir = self._grab_jjb() self._load_config_fixture('jjb-config.yaml') # set jjb_config to pulled in config - self.config['plugins'][0]['jjb_config'] = self.jjb_config_dir + self.config['plugins'][0]['jjb_config'] = jjb_config_dir self.start_server() zuul = fakes.FakeZuul(self.config['zuul_server']['gearman_host'], diff --git a/turbo_hipster/task_plugins/real_db_upgrade/handle_results.py b/turbo_hipster/task_plugins/real_db_upgrade/handle_results.py index 2126fda..cf8c6df 100644 --- a/turbo_hipster/task_plugins/real_db_upgrade/handle_results.py +++ b/turbo_hipster/task_plugins/real_db_upgrade/handle_results.py @@ -230,6 +230,13 @@ def check_log_file(log_file, git_path, dataset): for migration in lp.migrations: migration.setdefault('stats', {}) + # check migration completed + if not 'duration' in migration: + success = False + messages.append('WARNING - Migration %s->%s failed to complete' + % (migration['from'], migration['to'])) + continue + # Check total time if not check_migration(migration, 'maximum_migration_times', migration['duration'], dataset['config']): |