Skip to content

Add ci for erofs-snapshotter#2

Draft
aadhar-agarwal wants to merge 5 commits intomainfrom
aadagarwal/add-ci-test-erofs-snapshotter
Draft

Add ci for erofs-snapshotter#2
aadhar-agarwal wants to merge 5 commits intomainfrom
aadagarwal/add-ci-test-erofs-snapshotter

Conversation

@aadhar-agarwal
Copy link
Owner

No description provided.

@aadhar-agarwal aadhar-agarwal requested a review from Copilot June 18, 2025 23:55
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the CI workflow to support building and using EROFS tools for the erofs-snapshotter project.

  • Introduces a step to build and install erofs-utils from source on non-ARM runners
  • Adds a step to load the EROFS kernel module when supported
Comments suppressed due to low confidence (1)

.github/workflows/ci.yml:402

  • Consider adding --no-install-recommends to apt-get install to avoid pulling in unnecessary packages and reduce image size.
          sudo apt-get install -y git make gcc libz-dev liblz4-dev libfuse-dev autoconf libtool pkg-config uuid-dev

Comment on lines +409 to +411
git clone https://github.com/erofs/erofs-utils.git
cd erofs-utils
git checkout v1.8.5
Copy link

Copilot AI Jun 18, 2025

Choose a reason for hiding this comment

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

Use a shallow clone and branch flag to speed up CI, e.g., git clone --depth 1 --branch v1.8.5 https://github.com/erofs/erofs-utils.git and remove the separate git checkout step.

Suggested change
git clone https://github.com/erofs/erofs-utils.git
cd erofs-utils
git checkout v1.8.5
git clone --depth 1 --branch v1.8.5 https://github.com/erofs/erofs-utils.git
cd erofs-utils

Copilot uses AI. Check for mistakes.
aadhar-agarwal added a commit that referenced this pull request Nov 11, 2025
# This is the 1st commit message:

Add dmverity support to erofs snapshotter using veritysetup cli

Signed-off-by: Aadhar Agarwal <aadagarwal@microsoft.com>

# This is the commit message #2:

Add dmverity support to the erofs snapshotter

Signed-off-by: Aadhar Agarwal <aadagarwal@microsoft.com>

# This is the commit message #3:

Align changes
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