New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 883368 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Oct 4
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug
Build-Toolchain



Sign in to add a comment

Kernel AFDO profiles are probably too old

Project Member Reported by laszio@chromium.org, Sep 12

Issue description

Some of them are older than 3 weeks. We need to

1) look into the quality of profiles to decide if they are really too old.
2a) if so, figure out a way to fix or mitigate the problem.
2b) if not, simply loose the age check.

For now, we need to loose the age check. Otherwise it will block R69, R70 and R71 builds in a few days.
 
Labels: OS-Chrome
Project Member

Comment 2 by sheriffbot@chromium.org, Sep 12

Labels: -Merge-Request-69 Merge-Review-69 Hotlist-Merge-Review
This bug requires manual review: Request affecting a post-stable build
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), kariahda@(iOS), cindyb@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Hi cindyb@ and geohsu@, The CLs only increase days-before-hard-failure for some checks and should be totally safe to land. I'm pretty sure they won't break anything.
does the stale check even make sense on branches ?  the kernel versions are basically fixed at branch time.
Project Member

Comment 6 by sheriffbot@chromium.org, Sep 13

Labels: -Merge-Request-70 Hotlist-Merge-Approved Merge-Approved-70
Your change meets the bar and is auto-approved for M70. Please go ahead and merge the CL to branch 3538 manually. Please contact milestone owner if you have questions.
Owners: benmason@(Android), kariahda@(iOS), geohsu@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 7 by bugdroid1@chromium.org, Sep 13

Labels: merge-merged-release-R70-11021.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/74de15e6fac8c7b43a4764341ed1617f8858242e

commit 74de15e6fac8c7b43a4764341ed1617f8858242e
Author: Ting-Yuan Huang <laszio@chromium.org>
Date: Thu Sep 13 17:20:12 2018

afdo: extend allowed ages of kernel afdo profiles by 2 weeks

TEST=afdo_unittest, afdo_stage_unittest
BUG= chromium:883368 

Change-Id: I21231bdddf1009ac2460b012cb6ac89d890c8642
Reviewed-on: https://chromium-review.googlesource.com/1222106
Tested-by: Ting-Yuan Huang <laszio@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
(cherry picked from commit 6d7245a03cbbe4e16b198891ee43be203a6633ba)
Reviewed-on: https://chromium-review.googlesource.com/1222150

[modify] https://crrev.com/74de15e6fac8c7b43a4764341ed1617f8858242e/cbuildbot/afdo.py

vapier, is there a way to know in chromite to check canary vs branches?
#5: I totally agree but will let Luis and Manoj decide it.
Labels: -Merge-Review-69 Merge-Approved-69
Project Member

Comment 11 by bugdroid1@chromium.org, Sep 13

Labels: merge-merged-release-R69-10895.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/9ce808ac0387e3c4bb8d37eb9d7d379133d30e9d

commit 9ce808ac0387e3c4bb8d37eb9d7d379133d30e9d
Author: Ting-Yuan Huang <laszio@chromium.org>
Date: Thu Sep 13 17:32:59 2018

afdo: extend allowed ages of kernel afdo profiles by 2 weeks

TEST=afdo_unittest, afdo_stage_unittest
BUG= chromium:883368 

Change-Id: I21231bdddf1009ac2460b012cb6ac89d890c8642
Reviewed-on: https://chromium-review.googlesource.com/1222106
Tested-by: Ting-Yuan Huang <laszio@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
(cherry picked from commit 6d7245a03cbbe4e16b198891ee43be203a6633ba)
Reviewed-on: https://chromium-review.googlesource.com/1222151

[modify] https://crrev.com/9ce808ac0387e3c4bb8d37eb9d7d379133d30e9d/cbuildbot/afdo.py

i think we should be able to communicate whether cbuildbot is building the latest master or a branch, but at the very least, we could add an explicit knob to the config like "afdo_kernel_lifespan" which master would set to 2 weeks, but branches would set to None and that'd cause the check to be skipped.
Project Member

Comment 13 by sheriffbot@chromium.org, Sep 17

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Approved-69 -Merge-Approved-70
Status: Verified (was: Untriaged)

Sign in to add a comment