Skip to content

Add cleanup steps to CI workflows and Makefile#227

Merged
rene merged 1 commit intolf-edge:eve-kernel-amd64-v6.12.49-genericfrom
europaul:cleanup/eve-kernel-amd64-v6.12.49-generic
Mar 6, 2026
Merged

Add cleanup steps to CI workflows and Makefile#227
rene merged 1 commit intolf-edge:eve-kernel-amd64-v6.12.49-genericfrom
europaul:cleanup/eve-kernel-amd64-v6.12.49-generic

Conversation

@europaul
Copy link
Contributor

@europaul europaul commented Mar 4, 2026

To prevent GH runners from filling up we clean up all artifacts created during the build process.

To prevent GH runners from filling up we clean up all artifacts created
during the build process.

Signed-off-by: Paul Gaiduk <paulg@zededa.com>
@europaul europaul requested review from rene and rucoder March 4, 2026 17:34
if: always()
run: |
make -f Makefile.eve clean
docker system prune -af
Copy link
Collaborator

Choose a reason for hiding this comment

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

@europaul experience shown this is not enough. On EVE we came-up with the following commands for a good and effective clean up: https://github.com/lf-edge/eve/blob/master/.github/workflows/publish.yml#L233

- name: Clean
        if: ${{ always() }}
        run: |
          make clean || :
          docker system prune -f -a || :
          docker rm -f $(docker ps -aq) && docker volume rm -f $(docker volume ls -q) || :
          rm -rf ~/.linuxkit || :

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@rene I tested my approach - it removes all the created artifacts

Copy link
Collaborator

Choose a reason for hiding this comment

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

ok, I see you run docker kill $(BUILD_KIT_BUILDER) in the makefile, we should be good...

Copy link
Collaborator

@rene rene left a comment

Choose a reason for hiding this comment

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

LGTM

@rene rene merged commit 62283c1 into lf-edge:eve-kernel-amd64-v6.12.49-generic Mar 6, 2026
4 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.

2 participants