Browse code

zuul job: write the enable_plugin lines last

Plugins must be the last items in the local.conf file
otherwise the configuration set in the rest of the file
is not applied to them (for example a different value of DEST.)

Change-Id: Ia001badca179c3f3436d5ecd26b0755a3f3a3078

Luigi Toscano authored on 2019/02/27 02:39:51
Showing 1 changed files
... ...
@@ -214,11 +214,14 @@ class LocalConf(object):
214 214
         self.base_dir = base_dir
215 215
         self.projects = projects
216 216
         self.project = project
217
-        if plugins:
218
-            self.handle_plugins(plugins)
219 217
         if services or base_services:
220 218
             self.handle_services(base_services, services or {})
221 219
         self.handle_localrc(localrc)
220
+        # Plugins must be the last items in localrc, otherwise
221
+        # the configuration lines which follows them in the file are
222
+        # not applied to the plugins (for example, the value of DEST.)
223
+        if plugins:
224
+            self.handle_plugins(plugins)
222 225
         if localconf:
223 226
             self.handle_localconf(localconf)
224 227