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

[fix][ci] Fix docker image building by releasing more disk space before building #21365

Merged
merged 4 commits into from
Oct 14, 2023

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Oct 13, 2023

Motivation

The docker image building in Pulsar CI started failing recently with an error that can be seen in the summary view
Unhandled exception. System.IO.IOException: No space left on device : '/home/runner/runners/2.309.0/_diag/Worker_20231013-122508-utc.log'

The reason seems to be that there's less disk space available for the build.
There used to be 32717 MB of free disk space before starting the build. In that case, the docker image building would complete. There's now 32001 MB of free disk space and in that case, the disk space runs out.

Modifications

Pass mode: full parameter to the clean-disk action so that it also deletes /usr/share/dotnet directory before running the build to have more disk space available.

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

@lhotari lhotari added this to the 3.2.0 milestone Oct 13, 2023
@lhotari lhotari self-assigned this Oct 13, 2023
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Oct 13, 2023
Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

+1

@codecov-commenter
Copy link

Codecov Report

Merging #21365 (49b79b6) into master (1a352f1) will increase coverage by 39.73%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             master   #21365       +/-   ##
=============================================
+ Coverage     33.49%   73.22%   +39.73%     
- Complexity    12257    32515    +20258     
=============================================
  Files          1636     1888      +252     
  Lines        127789   140280    +12491     
  Branches      13963    15447     +1484     
=============================================
+ Hits          42798   102726    +59928     
+ Misses        79372    29463    -49909     
- Partials       5619     8091     +2472     
Flag Coverage Δ
inttests 24.22% <ø> (+0.08%) ⬆️
systests 24.70% <ø> (?)
unittests 72.50% <ø> (+40.50%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1533 files with indirect coverage changes

@lhotari lhotari merged commit 421c98a into apache:master Oct 14, 2023
49 checks passed
lhotari added a commit that referenced this pull request Oct 14, 2023
lhotari added a commit that referenced this pull request Oct 14, 2023
lhotari added a commit that referenced this pull request Oct 14, 2023
vraulji567 pushed a commit to vraulji567/pulsar that referenced this pull request Oct 16, 2023
mukesh-ctds pushed a commit to datastax/pulsar that referenced this pull request Dec 6, 2023
nikhil-ctds pushed a commit to datastax/pulsar that referenced this pull request Dec 20, 2023
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request Dec 20, 2023
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.

None yet

5 participants