Double Builds for Blimp Integration Test Infrastructure on FYI bot

scripts/slave/recipe_modules/pgo/api.py

Index: scripts/slave/recipe_modules/pgo/api.py
diff --git a/scripts/slave/recipe_modules/pgo/api.py b/scripts/slave/recipe_modules/pgo/api.py
index f938ab1401aec28385f767b7702b56e10f5c91aa..784101eb6727a36ca3f2c0f3a37ac98685d75e3b 100644
--- a/scripts/slave/recipe_modules/pgo/api.py
+++ b/scripts/slave/recipe_modules/pgo/api.py
@@ -56,7 +56,7 @@ class PGOApi(recipe_api.RecipeApi):
         self.m.properties['mastername'],
         self.m.properties['buildername'],
         use_goma=False,
-        phase=1)
+        phase=0)

[shenghuazhang, 2016/10/08 00:21:08]

The 'win_pgo' config keys are set to be '0', '1'. And the phase type defaults to
be 'str'. So set the phase '0', '1' instead of '1', '2'. 
Does this correct?

[Dirk Pranke, 2016/10/11 00:18:23]

Nope; this'd break the old branches, i.e., this module needs to not change at
all. 

Fix this client-side instead as I noted in my comments on the other CL just now.

(And sorry for the back-and-forth on this).

[shenghuazhang, 2016/10/12 22:42:04]

Oh yeah I see. This is very clear, I will change it back to the initial way.
Thanks for the explanation :)

     # Remove the profile files from the previous builds.
     self.m.file.rmwildcard('*.pg[cd]', str(self.m.chromium.output_dir))
     self.m.chromium.compile(name='Compile: Instrumentation phase.')
@@ -95,7 +95,7 @@ class PGOApi(recipe_api.RecipeApi):
         self.m.properties['mastername'],
         self.m.properties['buildername'],
         use_goma=False,
-        phase=2)
+        phase=1)

[shenghuazhang, 2016/10/08 00:21:08]

Relates here

[Dirk Pranke, 2016/10/11 00:18:23]

same answer :).

[shenghuazhang, 2016/10/12 22:42:04]

Done.

     self.m.chromium.compile(name='Compile: Optimization phase.')
 
   def compile_pgo(self, bot_config):