Skip to content

CI test should use PR commit instead of pulling the latest master #31788

@mrshenli

Description

@mrshenli

IIUC, currently we alway pull the latest master in CI tests, meaning that if we rerun tests on a master commit X, it could pull in more recent commits after X. This would make it harder to find out which commit breaks master.

For example, https://app.circleci.com/jobs/github/pytorch/pytorch/4101228 is a test for #31763, but the its git log contains #31526.

cc @ezyang

Metadata

Metadata

Assignees

No one assigned

    Labels

    module: ciRelated to continuous integrationtriagedThis issue has been looked at a team member, and triaged and prioritized into an appropriate module

    Type

    No type

    Projects

    Status

    Done

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions