Skip to content

Conversation

gcurtis
Copy link
Collaborator

@gcurtis gcurtis commented Mar 13, 2023

Hopefully this can help fix the flaky installer bugs on macOS. Also remove a really slow integration test.

Hopefully this can help fix the flaky installer bugs on macOS.
Copy link
Contributor

@mikeland73 mikeland73 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Worth trying. FWIW, I tried using this installer on Ubuntu a few weeks ago and it failed. Maybe only for mac?

Either way, we're still testing our own installer on every PR

@gcurtis
Copy link
Collaborator Author

gcurtis commented Mar 13, 2023

Yeah, I wanted to try it out in CI first to see how it goes.

@gcurtis gcurtis merged commit 5ad9c19 into main Mar 13, 2023
@gcurtis gcurtis deleted the gcurtis/ci-installer branch March 13, 2023 21:01
uses: DeterminateSystems/nix-installer-action@v1
with:
logger: pretty
nix-build-user-count: 4
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is there a reason behind putting 4 here or it's just to have a multi-user mode

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It defaults to 32, but creating 32 user accounts on macOS can be slow so I reduced it to 4. I don't think it should affect things too much since none of our jobs should be building from source.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants