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(github): free disk space for github workflows #1577

Merged
merged 11 commits into from
Aug 8, 2023

Conversation

empiredan
Copy link
Contributor

@empiredan empiredan commented Aug 1, 2023

#1576

Free more disk space for github workflows in the following ways:

  • remove unnecessary packages and directories with reduced disk space up
    to 11GB;
  • support updating options such as disk_min_available_space_ratio by env
    TEST_OPTS for some tests;
  • reduce disk_min_available_space_ratio from 10 to 5 to some failed tests
    due to SPACE_INSUFFICIENT.

@github-actions github-actions bot added the github label Aug 1, 2023
@github-actions github-actions bot added the cpp label Aug 2, 2023
@@ -31,6 +31,25 @@ exit_if_fail() {
fi
}

if [ -n ${TEST_OPTS} ]; then
Copy link
Member

Choose a reason for hiding this comment

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

I‘m planning to remove the run.sh scripts for unit tests in a long term, this change may be a small block for that.
How about define the default value as

DSN_DEFINE_int32(replication,
                 disk_min_available_space_ratio,
#ifdef PEGASUS_UNIT_TEST
                 5,
#else
                 10,
#endif
                 "if disk available space ratio "

@empiredan empiredan merged commit 8cad63e into apache:master Aug 8, 2023
empiredan added a commit that referenced this pull request Aug 28, 2023
#1585

Free more disk space for daily building cpp by referring to
#1577.

Test has been run manually on [Lint and build regularly](https://github.com/
apache/incubator-pegasus/actions/workflows/regular-build.yml)
for the branch [free-regular-disk-space](https://github.com/apache/
incubator-pegasus/tree/free-regular-disk-space).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants