From 33129e8fd7a15b7e5c7d118e6ca265e676931913 Mon Sep 17 00:00:00 2001 From: Brant Knudson Date: Sat, 31 Jan 2015 08:25:50 -0600 Subject: [PATCH] Change hacking check to verify all oslo imports The hacking check was verifying that specific oslo imports weren't using the oslo-namespaced package. Since all the oslo libraries used are now changed to use the new package name the hacking check can be simplified. bp drop-namespace-packages Change-Id: I6466e857c6eda0add6918e9fb14dc9296ed98600 --- keystone/hacking/checks.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/keystone/hacking/checks.py b/keystone/hacking/checks.py index 74df764239..d3b5fd073d 100644 --- a/keystone/hacking/checks.py +++ b/keystone/hacking/checks.py @@ -399,10 +399,7 @@ class CheckForLoggingIssues(BaseASTChecker): def check_oslo_namespace_imports(logical_line, blank_before, filename): oslo_namespace_imports = re.compile( - r"(((from)|(import))\s+oslo\.(" - "(config)|(db)|(messaging)|(serialization)|(utils)|(i18n)))|" - "(from\s+oslo\s+import\s+(" - "(config)|(db)|(messaging)|(serialization)|(utils)|(i18n)))") + r"(((from)|(import))\s+oslo\.)|(from\s+oslo\s+import\s+)") if re.match(oslo_namespace_imports, logical_line): msg = ("K333: '%s' must be used instead of '%s'.") % (