Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: split release workflow so the majority happens on Linux #2092

Merged
merged 11 commits into from Jun 7, 2022

Conversation

deansheather
Copy link
Member

Splits the release workflow up so most of it is built on Linux runners, with the only exception being the darwin binaries as they must be signed on a Mac.

This should yield major performance improvements as running Docker on Macs is really inefficient due to virtualization being required.

Also makes main deployments depend on database tests passing so we have less risk of breaking our dogfood deployment

@deansheather deansheather marked this pull request as ready for review June 6, 2022 19:43
.github/workflows/coder.yaml Outdated Show resolved Hide resolved
.github/workflows/release.yaml Outdated Show resolved Hide resolved
.goreleaser-release-darwin.yaml Outdated Show resolved Hide resolved
.goreleaser-release.yaml Outdated Show resolved Hide resolved
@deansheather
Copy link
Member Author

This PR should roughly halve the release time based on my tests

.goreleaser.yaml Outdated Show resolved Hide resolved
@deansheather deansheather enabled auto-merge (squash) June 7, 2022 15:10
@deansheather deansheather merged commit b87096b into main Jun 7, 2022
@deansheather deansheather deleted the split-release-workflow branch June 7, 2022 15:24
deansheather added a commit that referenced this pull request Jun 7, 2022
deansheather added a commit that referenced this pull request Jun 7, 2022
* Revert "chore: ignore artifacts dir (#2132)"

This reverts commit 27acb98.

* Revert "chore: split release workflow so the majority happens on Linux (#2092)"

This reverts commit b87096b.
kylecarbs pushed a commit that referenced this pull request Jun 10, 2022
* Revert "chore: ignore artifacts dir (#2132)"

This reverts commit 27acb98.

* Revert "chore: split release workflow so the majority happens on Linux (#2092)"

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

Successfully merging this pull request may close these issues.

None yet

2 participants