Chromium Code Reviews| Index: scripts/slave/recipe_modules/goma/api.py |
| diff --git a/scripts/slave/recipe_modules/goma/api.py b/scripts/slave/recipe_modules/goma/api.py |
| index 97c12f3f6e200a5787d3a682631de8b9f410a1b0..c4f6117edad150678dc6afd54a9ab20b4c9cccfe 100644 |
| --- a/scripts/slave/recipe_modules/goma/api.py |
| +++ b/scripts/slave/recipe_modules/goma/api.py |
| @@ -14,6 +14,7 @@ class GomaApi(recipe_api.RecipeApi): |
| self._goma_ctl_env = {} |
| self._cloudtail_pid = None |
| + self._goma_jobs = None |
| @property |
| def service_account_json_path(self): |
| @@ -31,6 +32,40 @@ class GomaApi(recipe_api.RecipeApi): |
| assert self._goma_dir |
| return self.m.path.join(self._goma_dir, 'jsonstatus') |
| + @property |
| + def recommended_goma_jobs(self): |
| + """ |
| + Return the recommended number of jobs for parallel build using Goma. |
| + |
| + This function caches the _goma_jobs. |
| + """ |
| + if self._goma_jobs: |
| + return self._goma_jobs |
| + |
| + # Use 80 for linux not to load goma backend. |
| + if self.m.platform.is_linux: |
| + return 80 |
|
shinyak
2016/09/07 04:45:54
Instead of using constant 80, could you limit max
tikuta
2016/09/07 04:53:08
Done.
|
| + |
| + step_result = self.m.python.inline( |
| + 'calculate the number of recommended jobs', |
| + """ |
| + import multiprocessing |
| + |
| + try: |
| + jobs = min(200, multiprocessing.cpu_count() * 10) |
| + except NotImplementedError: |
| + jobs = 50 |
| + |
| + print jobs |
| + """, |
| + stdout=self.m.raw_io.output(), |
| + step_test_data=( |
| + lambda: self.m.raw_io.test_api.stream_output('50\n')) |
| + ) |
| + self._goma_jobs = int(step_result.stdout) |
| + |
| + return self._goma_jobs |
| + |
| def ensure_goma(self, canary=False): |
| with self.m.step.nest('ensure_goma'): |
| with self.m.step.context({'infra_step': True}): |