Skip to content

Consider adding some kind of (optional?) toolchain pinning to the CI #498

Closed
@nicholasbishop

Description

@nicholasbishop

Overall the nightly channel is remarkably stable, but every now and then there's a bug that breaks the uefi-rs CI. For example: #494

When such a bug occurs it can take a while for things to get back to working; we have to first notice the bug, then bisect to where it fails, then file an issue in the rust repo, maybe put up a fix or wait for someone else to find a fix, wait for it to get reviewed, wait for it to get merged, and wait for the following nightly build to be created. So even under the best circumstances it will take some days to be resolved.

To minimize disruption when a thing like this occurs, it would be nice to pin to a known-working nightly release so that we can continue to merge PRs. On the other hand, it's also nice to generally test against latest nightly so that we can quickly respond to "legitimate" breakages, like when an unstable feature that we rely on is changed.

I'm thinking maybe something along the lines of a NIGHTLY_TOOLCHAIN=nightly in the workflow file, and adjust all the jobs to use that variable. When a breakage occurs we can temporarily set it to e.g. nightly-2022-01-02, and then revert back to nightly once the bug has been fixed.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions