Rethink and unify ios and chromium builder / test configs |
|||||
Issue descriptionThis is an umbrella issue for a large-scale project of revamping many of the configs that go into defining what a builder runs. In scope: - mb_config.pyl - testing/buildbot/*.pyl - recipe-side configs, both config.py scripts in the recipe modules, and various by-master configs defining which config.py settings to apply for which builders. - possibly some LUCI config bits, if there is duplication of settings. - ios/build/bots configs Out of scope: - proper LUCI configs for the services. These are tracked in issue 833946.
,
Dec 5
Current very preliminary thought: to use starlark and follow similar design idea as for lucicfg, but generate configs consumed by the recipes. In particular, this may require redesigning the recipes to read a bulk of their configs from chromium/src, instead of having those recipe-side in build.git.
,
Dec 5
I'm getting close to the point of just converting ios to the existing chromium configs, independently of the config work elsewhere.
,
Dec 6
Re: #3 - I'm not opposed to that, but it may degrade the config experience for the ios folks until this effort is finished. I was hoping to avoid that.
,
Dec 6
Yeah, and I understand that POV, but I'm coming to view that as less and less of a blocker given the ongoing maintenance burden presented by the uniqueness of ios's current infra configuration.
,
Dec 6
Filed issue 912681 to track the ios -> chromium migration. I'd like to keep this bug focused on configs.
,
Dec 7
The following revision refers to this bug: https://chromium.googlesource.com/infra/experimental/+/6a4ed8b60e3282ad9ea55916a2080fa3307d4f4a commit 6a4ed8b60e3282ad9ea55916a2080fa3307d4f4a Author: Sergey Berezin <sergeyberezin@google.com> Date: Fri Dec 07 00:40:54 2018 Set up a directory for experimenting with chromium configs Bug: 912367 Change-Id: Id322851df9743a6e9f2eb054874aaa62e215bed2 Reviewed-on: https://chromium-review.googlesource.com/c/1366917 Reviewed-by: Sergey Berezin <sergeyberezin@chromium.org> [add] https://crrev.com/6a4ed8b60e3282ad9ea55916a2080fa3307d4f4a/sergeyberezin/build_configs/README.md
,
Dec 10
,
Dec 19
The following revision refers to this bug: https://chromium.googlesource.com/infra/experimental/+/83f3bbd84daa51bac46c4a54ad7ed47a48c24354 commit 83f3bbd84daa51bac46c4a54ad7ed47a48c24354 Author: Sergey Berezin <sergeyberezin@google.com> Date: Wed Dec 19 03:07:15 2018 Add a primitive lucicfg module and config The intent is to test how a simple graph-based config can be created from the first principles using lucicfg. Bug: 912367 Change-Id: I5d93bbb7d00ed708e3be757e3413ed213f999d6f Reviewed-on: https://chromium-review.googlesource.com/c/1382628 Reviewed-by: Sergey Berezin <sergeyberezin@chromium.org> [add] https://crrev.com/83f3bbd84daa51bac46c4a54ad7ed47a48c24354/sergeyberezin/build_configs/test.star [add] https://crrev.com/83f3bbd84daa51bac46c4a54ad7ed47a48c24354/sergeyberezin/build_configs/chromium.star
,
Dec 21
The following revision refers to this bug: https://chromium.googlesource.com/infra/experimental/+/6d3bcf7922faa07e38121d217d2ed1270de6ee46 commit 6d3bcf7922faa07e38121d217d2ed1270de6ee46 Author: Sergey Berezin <sergeyberezin@google.com> Date: Fri Dec 21 00:05:34 2018 [builder_configs] Implement core chromium config constructors Bug: 912367 Change-Id: I45796194803a2ef33b306cd81953f77fdf887ae2 Reviewed-on: https://chromium-review.googlesource.com/c/1387846 Reviewed-by: Sergey Berezin <sergeyberezin@chromium.org> [add] https://crrev.com/6d3bcf7922faa07e38121d217d2ed1270de6ee46/sergeyberezin/build_configs/chromiumlib/chromium.star [delete] https://crrev.com/83f3bbd84daa51bac46c4a54ad7ed47a48c24354/sergeyberezin/build_configs/chromium.star [delete] https://crrev.com/83f3bbd84daa51bac46c4a54ad7ed47a48c24354/sergeyberezin/build_configs/test.star
,
Dec 21
The following revision refers to this bug: https://chromium.googlesource.com/infra/experimental/+/68277362c6b65ddbc0562c74b09c33f4ae5a2b91 commit 68277362c6b65ddbc0562c74b09c33f4ae5a2b91 Author: Sergey Berezin <sergeyberezin@google.com> Date: Fri Dec 21 00:20:15 2018 [builder_configs] Add sample configs to test the format. Bug: 912367 Change-Id: I16dc3e33352a5fbdafe9b92d6f249d505f5fff68 Reviewed-on: https://chromium-review.googlesource.com/c/1387847 Reviewed-by: Sergey Berezin <sergeyberezin@chromium.org> [add] https://crrev.com/68277362c6b65ddbc0562c74b09c33f4ae5a2b91/sergeyberezin/build_configs/targets/targets.star [add] https://crrev.com/68277362c6b65ddbc0562c74b09c33f4ae5a2b91/sergeyberezin/build_configs/configs.star [add] https://crrev.com/68277362c6b65ddbc0562c74b09c33f4ae5a2b91/sergeyberezin/build_configs/targets/platforms.star [add] https://crrev.com/68277362c6b65ddbc0562c74b09c33f4ae5a2b91/sergeyberezin/build_configs/builders/builders.star
,
Dec 21
The following revision refers to this bug: https://chromium.googlesource.com/infra/experimental/+/2f42c57d2a0ba9bfd65db8b3c4f67744b6fd5156 commit 2f42c57d2a0ba9bfd65db8b3c4f67744b6fd5156 Author: Sergey Berezin <sergeyberezin@google.com> Date: Fri Dec 21 01:18:57 2018 [builder_configs] Add a starter script to convert existing configs. Also, update the list of targets from the actual waterfalls.pyl. Bug: 912367 Change-Id: Ia6b106bcf9906c0427d94aa09171bfc76fd4c6da Reviewed-on: https://chromium-review.googlesource.com/c/1387848 Reviewed-by: Sergey Berezin <sergeyberezin@chromium.org> [modify] https://crrev.com/2f42c57d2a0ba9bfd65db8b3c4f67744b6fd5156/sergeyberezin/build_configs/targets/targets.star [add] https://crrev.com/2f42c57d2a0ba9bfd65db8b3c4f67744b6fd5156/sergeyberezin/build_configs/convert_configs.py
,
Jan 11
Setting defect without priority to default.
,
Jan 15
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by sergeybe...@chromium.org
, Dec 5