From 581ec646582c462d9a190de5265670f7146d4d31 Mon Sep 17 00:00:00 2001 From: Doug Hellmann Date: Wed, 15 Feb 2017 22:27:19 -0500 Subject: [PATCH] change error on missing branch to warning We have some deliverables with new (or at least unbranched) repositories for ocata. This is likely to be the case in the future, too, so make the error a warning for now until we can improve the logic. Change-Id: Id2d6dc96501ecfb65945b406f17bd4dcfdaea1c2 Signed-off-by: Doug Hellmann --- openstack_releases/cmds/validate.py | 2 +- openstack_releases/tests/test_validate.py | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/openstack_releases/cmds/validate.py b/openstack_releases/cmds/validate.py index d7a6f6bf18..ac3467141f 100644 --- a/openstack_releases/cmds/validate.py +++ b/openstack_releases/cmds/validate.py @@ -115,7 +115,7 @@ def validate_series_open(deliverable_info, if branch['name'] == expected_branch: # Everything is OK return - mk_error( + mk_warning( 'There is no {} branch defined in {}. Is the {} series open?'.format( expected_branch, previous_deliverable_file, series_name)) diff --git a/openstack_releases/tests/test_validate.py b/openstack_releases/tests/test_validate.py index ff55e60ebe..2e77db7a4e 100644 --- a/openstack_releases/tests/test_validate.py +++ b/openstack_releases/tests/test_validate.py @@ -1476,5 +1476,5 @@ class TestValidateSeriesOpen(base.BaseTestCase): errors.append, ) print(warnings, errors) - self.assertEqual(0, len(warnings)) - self.assertEqual(1, len(errors)) + self.assertEqual(1, len(warnings)) + self.assertEqual(0, len(errors))