diff --git a/openstack_releases/cmds/validate.py b/openstack_releases/cmds/validate.py index 1d18d4db55..2950387067 100644 --- a/openstack_releases/cmds/validate.py +++ b/openstack_releases/cmds/validate.py @@ -793,9 +793,7 @@ def validate_new_releases(deliverable_info, deliverable_name, repository_settings = deliverable_info.get('repository-settings', {}) for repo in actual_repos: if repo not in repository_settings: - # TODO(dhellmann): Turn this into a warning after the T - # series is open. - mk_warning( + mk_error( 'release %s includes repository %s ' 'that is not in the repository-settings section' % (final_release['version'], repo) diff --git a/openstack_releases/tests/test_validate.py b/openstack_releases/tests/test_validate.py index 1cf7e19f8e..51975ae067 100644 --- a/openstack_releases/tests/test_validate.py +++ b/openstack_releases/tests/test_validate.py @@ -1340,8 +1340,8 @@ class TestValidateNewReleases(base.BaseTestCase): errors.append, ) print(warnings, errors) - self.assertEqual(1, len(warnings)) - self.assertEqual(0, len(errors)) + self.assertEqual(0, len(warnings)) + self.assertEqual(1, len(errors)) def test_missing_repo_info(self): # The tag is missing a repository that is in