simplify Chrome branch process in Chrome OS builds |
|
Issue descriptioncreating a new Chrome OS branch currently involves creating a "sticky chrome" ebuild so that the uprev logic knows which version more precisely to base on in the branch. but looking at the way the uprev works, it seems to (at least in the common case?) already work correctly. if we look at branch R40 for example: cc02abd819efd6a2e4d2474df8ef6e5a84cf13f9: 40.0.2214.0_rc was bumped to 40.0.2214.2_rc 187e1b8be6ca8a580d11e3148c4885a3e4b3f430: 40.0.2214.2_rc was bumped to 40.0.2214.3_rc 6317473d4477978389d496da44507d793513156c: 40.0.2214.0 was created to stick to "40.0.2214.x" so if the point of creating the sticky ebuild is to stick the branch to 40.0.2214.x, but the branch is already doing that (since cros_mark_chrome_as_stable is called with "stable_release" mode), do we need to create the sticky ebuild at all ? what scenarios aren't being handled here ?
,
Dec 7
i'm not suggesting it create a sticky ebuild, but that it not need one in the first place. it might require some changes to the stable script. i'm interested in what requirements the TPM side has wrt sticky ebuilds. on a branch, when would you ever do something other than stick it to e.g. "40.0.2214.x" ? when i looked through the uprev script, it's making queries to some API end points to get version numbers for the branch ...
,
Dec 7
I think all we need to do is be able to specify what upstream Chrome branch we try to find new Chrome versions from, kind of like how we have a constant in Chromite to define the upstream Android branch we pull ARC++ container images from (e.g. ANDROID_NYC_BUILD_BRANCH and ANDROID_PI_BUILD_BRANCH). If the branch or builder scripts can automatically locate what the associated Chrome release branch is, that would be great. A heuristic like the highest build revision on the current milestone might do it in most cases (only the release branch is likely to go higher than .5 or so, and I am not sure if we are guaranteed to get the right one if we say the highest branch version for the milestone), though this is not perfect, so we would probably want to pull from some Chrome side source, or just set it manually. |
|
►
Sign in to add a comment |
|
Comment 1 by bhthompson@google.com
, Dec 7