Skip to content

Presubmit builds may check out the wrong commit #149851

@Piinks

Description

@Piinks

Type of Request

bug

Infrastructure Environment

LUCI, Github

What is happening?

I've seen this happen several times now.

In a given PR, newly added unit tests will pass in presubmit. After the PR is approved and lands, some of those newly added unit tests fail, and the change is reverted.

I've seen this happen on:

It sounds like @christopherfujino has a lead on why this might be happening: #149245 (comment)

There were also some golden file failures in postsubmit earlier today that we weren't sure how they were getting in. If the golden file tests did not execute in presubmit, like is suspected above, then this is also the likely cause of the unexpected golden files at head.

Steps to reproduce

See linked builds above

Expected results

Tests run in both presubmit and postsubmit, reflecting the same result.

Metadata

Metadata

Labels

P0Critical issues such as a build break or regressionteam-infraOwned by Infrastructure teamtriaged-infraTriaged by Infrastructure team

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions