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

Run tests on AWS runners, separate workflows for linux and mac. #253

Conversation

martun
Copy link
Contributor

@martun martun commented Dec 11, 2023

No description provided.

@martun martun self-assigned this Dec 11, 2023
@martun martun linked an issue Dec 11, 2023 that may be closed by this pull request
@martun martun marked this pull request as draft December 11, 2023 06:54
@martun martun changed the title First try with separated linux-mac aws runners. Run tests on AWS runners, separate workflows for linux and mac. Dec 11, 2023
@martun martun force-pushed the 238-move-our-runners-to-aws_autoscaling-since-we-need-a-bit-more-ram branch 7 times, most recently from 9e0d26b to 6918b20 Compare December 12, 2023 08:33
@martun martun requested a review from x-mass December 12, 2023 11:33
@martun martun force-pushed the 238-move-our-runners-to-aws_autoscaling-since-we-need-a-bit-more-ram branch from aa71694 to 218010f Compare December 12, 2023 12:21
jobs:
call-reusable-workflow:
name: Call Reusable Testing Callback Workflow
uses: NilFoundation/ci-cd/.github/workflows/reusable-crypto3-publish-result.yml@v1
Copy link
Contributor

@x-mass x-mass Dec 12, 2023

Choose a reason for hiding this comment

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

Update this to the latest version, please

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

name: Linux Reusable Crypto3 Testing
needs:
- handle-syncwith
uses: NilFoundation/ci-cd/.github/workflows/reusable-crypto3-testing-linux.yml@f9d91d2a62f5fb8d6767e4a458cae49da29aefe9
Copy link
Contributor

Choose a reason for hiding this comment

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

Friendly reminder to replace references. Btw, you can use branch name here, so you won't need to replace SHA manually

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@martun martun marked this pull request as ready for review December 12, 2023 13:52
@martun martun merged commit d8bd4e1 into master Dec 12, 2023
7 checks passed
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.

Move our runners to aws_autoscaling, since we need a bit more RAM
2 participants