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

Tune arm64 memory allocation and disable telemetry #652

Merged
merged 3 commits into from
Dec 28, 2023

Conversation

jmhbnz
Copy link
Member

@jmhbnz jmhbnz commented Dec 27, 2023

This pull request tunes arm64 actuated.dev runner memory requests down from 16gb to 8gb to better reflect what our workflows actually need based on telemetry and ensure we are being good citizens with this shared infrastructure.

In #640 we introduced the telemetry and based on recent workflow runs it looks like we stay under 4gb currently:

image

With this tuning now complete I have also removed the workflow telemetry to reduce noise.

Fixes: etcd-io/etcd#17045

Note: I also tidied up a couple of things yamllint complained about in the workflow files.

Based on memory consumption statistics for historic runs.

Signed-off-by: James Blair <[email protected]>
Copy link
Member

@fuweid fuweid left a comment

Choose a reason for hiding this comment

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

LGTM

@ahrtr ahrtr merged commit 47cc272 into etcd-io:main Dec 28, 2023
14 checks passed
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.

Tune memory allocation of arm64 github actions workflows
3 participants