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.

Sign up for GitHub

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

Jump to bottom

Reduce container image size #5630

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Reduce container image size #5630

wants to merge 1 commit into from

Conversation

ferrarimarco
Copy link
Collaborator

@ferrarimarco ferrarimarco commented May 9, 2024 •
edited

Proposed changes

The baseline container image size for the standard flavor is 6.04GB (v6.4.1)

Reduce container image size:

  • Remove unneeded Dart SDK components. Size down to 5.7GB

Also, I did the following changes to the CI workflow:

  • Run make info in a dedicated step.
  • Move the info target after we configure the required variables

Readiness checklist

In order to have this pull request merged, complete the following tasks.

Pull request author tasks

  • I included all the needed documentation for this change.
  • I provided the necessary tests.
  • I squashed all the commits into a single commit.
  • I followed the Conventional Commit v1.0.0 spec.
  • I wrote the necessary upgrade instructions in the upgrade guide.
  • If this pull request is about and existing issue,
    I added the Fix #ISSUE_NUMBER label to the description of the pull request.

Super-linter maintainer tasks

  • Label as breaking if this change breaks compatibility with the previous released version.
  • Label as either: automation, bug, documentation, enhancement, infrastructure.

@ferrarimarco ferrarimarco added infrastructure related to project structure and maintenance O: backlog 🤖 Backlog, stale ignores this label labels May 9, 2024
@ferrarimarco ferrarimarco self-assigned this May 9, 2024
@ferrarimarco ferrarimarco changed the title build: reduce container image size Reduce container image size May 9, 2024
@ferrarimarco ferrarimarco added this to the 6.6.0 milestone May 9, 2024
@ferrarimarco ferrarimarco force-pushed the reduce-size branch 2 times, most recently from 5a386dd to 8dde368 Compare May 13, 2024 13:53
@ferrarimarco
build: reduce container image size
b2ab4e6 
Install only executables and needed libraries
@ferrarimarco ferrarimarco removed this from the 6.6.0 milestone May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zkoppert zkoppert Awaiting requested review from zkoppert zkoppert will be requested when the pull request is marked ready for review zkoppert is a code owner

@Hanse00 Hanse00 Awaiting requested review from Hanse00 Hanse00 will be requested when the pull request is marked ready for review Hanse00 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@ferrarimarco ferrarimarco

Labels
infrastructure related to project structure and maintenance O: backlog 🤖 Backlog, stale ignores this label
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@ferrarimarco