Android PFQ missing HWTests |
|
Issue descriptionIt looks like no Android PFQ builds are running HWTest. https://uberchromegw.corp.google.com/i/chromeos/builders/caroline-nyc-android-pfq/builds/125 has HWTest reported. https://uberchromegw.corp.google.com/i/chromeos/builders/caroline-nyc-android-pfq/builds/126 doesn't. https://uberchromegw.corp.google.com/i/chromeos/builders/veyron_jaq-nyc-android-pfq/builds/65 has HWTest reported. https://uberchromegw.corp.google.com/i/chromeos/builders/veyron_jaq-nyc-android-pfq/builds/66 doesn't.
,
May 30 2017
Or if you think we should always run HWTest even if no uprev will happen, please reopen.
,
May 30 2017
To be honest, I think it is confusing. The Android PFQ seems to work differently from the Chrome PFQ. Now this might be a good thing. It appears to me the Android PFQ keeps accumulating OS and Chrome changes without testing them on the actual PFQ boards, but ultimately also without displaying them properly once a hardware test actually runs (the build page will only show the last few OS changes, right)? I don't have strong feelings: not running HWTest has its benefits. But something doesn't seem quite right with the current setup.
,
May 30 2017
Yes, I definitely agree it's confusing... and I guess ExitEarlyException is evil. Maybe it's better if we could mark remaining stages explicitly "skipped", though such cleanup seems to cost more than benefit IMO. Any good idea to clarify this more is welcomed.
,
May 30 2017
I might be misremembering (and have too much on my plate right now to go and veirfy), but I feel like the logic should be the same between Android and Chrome PFQs. I based the original code on the Chrome PFQ, including the skipping logic. Could it just be that NOP Chrome PFQ runs just don't happen as often?
,
May 30 2017
NOP Chrome PFQ runs happen after a successful uprev. So I went to the waterfall: they behave exactly like the Android PFQ, no HWTests are run. So we have consistency. Nothing to see here. WontFix. |
|
►
Sign in to add a comment |
|
Comment 1 by nya@chromium.org
, May 30 2017Status: WontFix (was: Untriaged)