From bb317a93233d216679f4960727b2f127b905824b Mon Sep 17 00:00:00 2001 From: Ralph Bean Date: Thu, 27 Oct 2016 15:59:41 -0400 Subject: [PATCH] Fix typo bug in our existence check. This gave a traceback every time before. We need a trailing comma there in order to make that an `iterable`, otherwise our `in` check doesn't make any sense. --- module_build_service/views.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/module_build_service/views.py b/module_build_service/views.py index 08e16b60..6b3a8342 100644 --- a/module_build_service/views.py +++ b/module_build_service/views.py @@ -139,7 +139,7 @@ class ModuleBuildAPI(MethodView): log.debug('Checking whether module build already exist.') # TODO: make this configurable, we might want to allow # resubmitting any stuck build on DEV no matter the state - if module.state not in (models.BUILD_STATES['failed']): + if module.state not in (models.BUILD_STATES['failed'],): log.error('Module (state=%s) already exists. ' 'Only new or failed builds are allowed.' % module.state)