Refactor "chromium_*" recipe modules and the recipes that use them. |
|||||||
Issue descriptionThe most core Chromium recipes import and use the "chromium", "chromium_android", and "chromium_tests" recipe modules to checkout, build, and test Chromium. These recipe modules have grown organically, and have proven to be unwieldy and confusing for people, especially those who are not familiar with them, to work with. Part of the Chromium crossover effort will be to improve the usability of these recipes by refactoring them. The specific approach still needs to be worked out, but an overview of the effort will be: - Split the modules into Infrastructure- and Chromium-specific components. - Identify Chromium team members who will own the Chromium side of these components. - Design and implement refactoring of these respective modules.
,
Dec 7 2016
The way I'm thinking about the recipes these days (and the responsibilities of the infra sub-teams), I don't think there is a "Chromium side" of the recipes per se that would be owned solely by chromium devs. I think the right place to draw the line is in the src-side interfaces (the *.json files, the Test step APIs, the MB configs, etc.). I mostly want infra folks (either foundation or crossover) to own all the recipe-side code. I think this is consistent with what we've discussed recently. Agree?
,
Dec 14 2016
I'm going to reassign this to me for now ...
,
Dec 16 2016
,
Dec 16 2016
,
Dec 16 2016
why is this a postmortem-related thing?
,
Dec 16 2016
Just flying by, but in light of the current team organization, I think the owners in comment #0 would be "foundation" and "crossover", not "infra" and "chromium".
,
Dec 16 2016
I think actually that the answer is "all crossover", but maybe there's some stuff that we can pull out of the chromium recipes and modules that we'd want to share for non-chromium teams, like a new steps api and logic to handle retrying w/o patches. I'm not sure ...
,
Dec 16 2016
,
Jan 4 2017
@dnj - let's talk about this when you have some time. I'd like to make sure my current thinking about this stuff lines up with yours.
,
Jan 15 2017
We talked about this, but I didn't update the bug. Our conclusion was that this CL as it stands is a bit vague, and we don't really want to think of things as "crossover" or not. However, we should clean up these recipes and see if we can make them as generic as possible, with clean declarative APIs for repo-side configuration. I'm closing this bug; we'll work on that cleanup elsewhere in other tasks.
,
Jan 25 2017
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by andyb...@chromium.org
, Nov 9 2016Owner: d...@chromium.org
Status: Assigned (was: Untriaged)