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

Issue 869537 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 1
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Consolidate some platform nightly tests into one nightly suite

Project Member Reported by ka...@chromium.org, Jul 31

Issue description

The idea is to run PrintJob and StageAndRecover (and possibly other autotests) tests as part of one nightly test suite the platform test team is monitoring on daily basis.
 
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 1

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/infra/suite_scheduler/+/2e87621b7add88f714f052a94f7bf482c246affd

commit 2e87621b7add88f714f052a94f7bf482c246affd
Author: Kalin Stoyanov <kalin@chromium.org>
Date: Wed Aug 01 07:05:40 2018

Changed suite name and spec

BUG= chromium:869537 
TEST=None

Change-Id: I971362779c3d60a31300e87e9a558054a52423ba
Reviewed-on: https://chromium-review.googlesource.com/1157191
Commit-Ready: Kalin Stoyanov <kalin@chromium.org>
Tested-by: Kalin Stoyanov <kalin@chromium.org>
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>

[modify] https://crrev.com/2e87621b7add88f714f052a94f7bf482c246affd/configs/suite_scheduler.ini

Labels: -Pri-3 M-69 Merge-Request-69 M-70 Pri-2
Status: Fixed (was: Untriaged)
Requesting merge to M69 for +/1157207, i.e.
https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/1158804

Autotest only change
Cc: cindyb@chromium.org
Project Member

Comment 6 by sheriffbot@chromium.org, Aug 2

Labels: -Merge-Request-69 Hotlist-Merge-Approved Merge-Approved-69
Your change meets the bar and is auto-approved for M69. Please go ahead and merge the CL to branch 3497 manually. Please contact 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
Project Member

Comment 7 by bugdroid1@chromium.org, Aug 2

Labels: merge-merged-release-R69-10895.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/93ee9b434e51ebaf6193a359d561cc0cad5bcdce

commit 93ee9b434e51ebaf6193a359d561cc0cad5bcdce
Author: Kalin Stoyanov <kalin@chromium.org>
Date: Thu Aug 02 15:29:52 2018

Change test suite for two tests.

BUG= chromium:869537 
TEST=None

Change-Id: Ia70947d3871311d561a2a58adc9f645ee7f61e10
Reviewed-on: https://chromium-review.googlesource.com/1157207
Commit-Ready: Kalin Stoyanov <kalin@chromium.org>
Tested-by: Kalin Stoyanov <kalin@chromium.org>
Reviewed-by: Katherine Threlkeld <kathrelkeld@chromium.org>
(cherry picked from commit de4eb9abec1a37c6992cbaba600f29e663c16515)
Reviewed-on: https://chromium-review.googlesource.com/1158804
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
Commit-Queue: Kalin Stoyanov <kalin@chromium.org>

[modify] https://crrev.com/93ee9b434e51ebaf6193a359d561cc0cad5bcdce/test_suites/control.chameleon_peripherals
[modify] https://crrev.com/93ee9b434e51ebaf6193a359d561cc0cad5bcdce/client/site_tests/platform_PrintJob/control
[modify] https://crrev.com/93ee9b434e51ebaf6193a359d561cc0cad5bcdce/server/site_tests/platform_StageAndRecover/control

With the last change I change the suite name but did not create new file.
That is probably why one of the tests is not running having the parrent test suite job still looking for pool:chameleon, though it is not a dependency anymore.
08/02 20:20:48.863 DEBUG|             suite:1060| Discovered 2 tests.
08/02 20:20:48.864 INFO |        server_job:0216| INFO	----	Start platform_test_nightly	timestamp=1533266448	localtime=Aug 02 20:20:48	
08/02 20:20:48.864 DEBUG|             suite:1008| Scheduling platform_StageAndRecover
08/02 20:20:49.172 DEBUG|        retry_util:0207| ending retries with error: <class 'autotest_lib.client.common_lib.error.NoEligibleHostException'>(NoEligibleHostException: No hosts within board:eve satisfy recovery_test, servo, cros-version:eve-release/R70-10932.0.0, board:eve, pool:chameleon.
Traceback (most recent call last):
  File "/usr/local/autotest/frontend/afe/json_rpc/serviceHandler.py", line 109, in dispatchRequest
    results['result'] = self.invokeServiceEndpoint(meth, args)
  File "/usr/local/autotest/frontend/afe/json_rpc/serviceHandler.py", line 147, in invokeServiceEndpoint
    return meth(*args)
  File "/usr/local/autotest/frontend/afe/rpc_handler.py", line 270, in new_fn
    return f(*args, **keyword_args)
  File "/usr/local/autotest/frontend/afe/rpc_utils.py", line 1172, in replacement
    return func(**kwargs)
  File "/usr/local/autotest/frontend/afe/rpc_interface.py", line 1068, in create_job
    require_ssp=require_ssp)
  File "/usr/local/autotest/frontend/afe/rpc_utils.py", line 831, in create_job_common
    metahost_objects=metahost_objects)
  File "/usr/local/autotest/frontend/afe/rpc_utils.py", line 515, in create_new_job
    check_job_metahost_dependencies(metahost_objects, dependencies)
  File "/usr/local/autotest/frontend/afe/rpc_utils.py", line 355, in check_job_metahost_dependencies
    % (metahost.name, ', '.join(job_dependencies)))
NoEligibleHostException: No hosts within board:eve satisfy recovery_test, servo, cros-version:eve-release/R70-10932.0.0, board:eve, pool:chameleon.
)
08/02 20:20:49.172 DEBUG|             suite:1020| platform_StageAndRecover not applicable for this board/pool. Emitting TEST_NA.

The change below is creating a new test file.
https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/1162706
Project Member

Comment 9 by bugdroid1@chromium.org, Aug 5

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/040df2d8769a78b02686deb8e20e925ae9ca0182

commit 040df2d8769a78b02686deb8e20e925ae9ca0182
Author: Kalin Stoyanov <kalin@chromium.org>
Date: Sun Aug 05 22:03:30 2018

Replace suite file.

I created new suite, but left it in old suite file.

BUG= chromium:869537 
TEST=None

Change-Id: I98e2a99c2037a351e293943220d7b0a010f426e5
Reviewed-on: https://chromium-review.googlesource.com/1162706
Commit-Ready: Kalin Stoyanov <kalin@chromium.org>
Tested-by: Kalin Stoyanov <kalin@chromium.org>
Reviewed-by: Sridhar Sonti <sontis@chromium.org>
Reviewed-by: Katherine Threlkeld <kathrelkeld@chromium.org>

[rename] https://crrev.com/040df2d8769a78b02686deb8e20e925ae9ca0182/test_suites/control.platform_test_nightly

Project Member

Comment 10 by bugdroid1@chromium.org, Aug 5

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/a038a84788ef04ab63dddeac5864832323937ee7

commit a038a84788ef04ab63dddeac5864832323937ee7
Author: Kalin Stoyanov <kalin@chromium.org>
Date: Sun Aug 05 22:36:47 2018

Replace suite file.

I created new suite, but left it in old suite file.

BUG= chromium:869537 
TEST=None

Change-Id: I98e2a99c2037a351e293943220d7b0a010f426e5
Reviewed-on: https://chromium-review.googlesource.com/1162706
Commit-Ready: Kalin Stoyanov <kalin@chromium.org>
Tested-by: Kalin Stoyanov <kalin@chromium.org>
Reviewed-by: Sridhar Sonti <sontis@chromium.org>
Reviewed-by: Katherine Threlkeld <kathrelkeld@chromium.org>
(cherry picked from commit 040df2d8769a78b02686deb8e20e925ae9ca0182)
Reviewed-on: https://chromium-review.googlesource.com/1162722
Reviewed-by: Kalin Stoyanov <kalin@chromium.org>
Commit-Queue: Kalin Stoyanov <kalin@chromium.org>

[rename] https://crrev.com/a038a84788ef04ab63dddeac5864832323937ee7/test_suites/control.platform_test_nightly

Project Member

Comment 11 by sheriffbot@chromium.org, Aug 6

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
Project Member

Comment 12 by sheriffbot@chromium.org, Aug 10

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

Sign in to add a comment