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

Use workflow-kotlin-test-runner-ubuntu-4core to unflake #163

Merged
merged 1 commit into from
Apr 23, 2024

Conversation

rjrjr
Copy link
Contributor

@rjrjr rjrjr commented Apr 22, 2024

Updates our GHA config to take advantage of the runner that was created to unflake Workflow's integration tests, for the same reason.

@rjrjr rjrjr requested review from pyricau and a team as code owners April 22, 2024 23:29
@CLAassistant
Copy link

CLAassistant commented Apr 22, 2024

CLA assistant check
All committers have signed the CLA.

@rjrjr rjrjr force-pushed the ray/fatter-runner branch from b482c16 to 0b351b2 Compare April 22, 2024 23:52
Comment on lines -3 to +5
on:
push:
branches: [ main ]
# Build on all pull requests, regardless of target.
pull_request:
on :
pull_request :
merge_group :

Choose a reason for hiding this comment

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

Does this need to be changed as part of this?

Choose a reason for hiding this comment

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

It depends on how securely we want to lock down the custom runner. if we specify the workflows it can run on rather than just open it to any workflow then we need a stable workflow name - which can only happen if we run it on the main branch rather than the new branch that is created for each PR.

Copy link

@handstandsam handstandsam left a comment

Choose a reason for hiding this comment

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

Approving as it makes sense.

@steve-the-edwards
Copy link

We need to add the KVM group perms to enable HW acceleration for teh emulator in the action.

See https://github.com/square/workflow-kotlin/pull/1147/files?short_path=6b8d3aa#diff-6b8d3aa6fcc54e0e940045c4c73d85625e9f4ecf8da7a5841b17ca5210bd9cdb

Updates our GHA config to take advantage of the runner that was created to unflake Workflow's integration tests, for the same reason.
@rjrjr rjrjr force-pushed the ray/fatter-runner branch from 0b351b2 to 422d76f Compare April 23, 2024 20:51
@rjrjr rjrjr merged commit 344f158 into main Apr 23, 2024
6 checks passed
@rjrjr rjrjr deleted the ray/fatter-runner branch April 23, 2024 21:08
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.

4 participants