New issue
Advanced search Search tips

Issue 917479 link

Starred by 2 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

luci-py: stop running luci-py CI through presubmit

Project Member Reported by mar...@chromium.org, Dec 21

Issue description

This is purely an historical reason. We should instead have a proper script to run the tests.

https://cs.chromium.org/chromium/infra/recipes/recipes/luci_py.py

I suspect problems like  issue 917474  would be averted if we were using a normal way to run the tests.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 21

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/36048348ca6faa9a10080c73ad5726f262f35b2c

commit 36048348ca6faa9a10080c73ad5726f262f35b2c
Author: Marc-Antoine Ruel <maruel@chromium.org>
Date: Fri Dec 21 20:53:38 2018

luci_py recipe: add bug to TODO

The luci-py CI recipe simply runs presubmit. While this works, we'd like
to move to properly running all the tests.

No code change.

TBR=tandrii@chromium.org

Bug: 917479
Change-Id: Iacb3d18a07ea726b3b76da41059a83fc9cc02fe4
Reviewed-on: https://chromium-review.googlesource.com/c/1388451
Reviewed-by: Marc-Antoine Ruel <maruel@chromium.org>
Commit-Queue: Marc-Antoine Ruel <maruel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#19711}
[modify] https://crrev.com/36048348ca6faa9a10080c73ad5726f262f35b2c/recipes/recipes/luci_py.py

Sign in to add a comment