Skip to content

feat: impose image size limit on cuda containers - #337

Open
wdconinc wants to merge 1 commit into
masterfrom
wdconinc-patch-3
Open

feat: impose image size limit on cuda containers#337
wdconinc wants to merge 1 commit into
masterfrom
wdconinc-patch-3

Conversation

@wdconinc

@wdconinc wdconinc commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

Briefly, what does this PR introduce? Please link to any relevant presentations or discussions.

This PR imposes an upper limit on the image size for eic_cuda and eic_dev_cuda.

Needs:

What is the urgency of this PR?

  • High (please describe reason below)
  • Medium
  • Low

What kind of change does this PR introduce?

  • Bug fix (issue #__)
  • New feature (issue #__)
  • Optimization (issue #__)
  • Updated documentation
  • other: __

Please check if any of the following apply

  • This PR introduces breaking changes. Please describe changes users need to make below.
  • This PR changes default behavior. Please describe changes below.
  • AI was used in preparing this PR. Please describe usage below.

Copilot AI review requested due to automatic review settings July 5, 2026 14:58
@wdconinc
wdconinc enabled auto-merge (squash) July 5, 2026 14:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds compressed image size caps for the CUDA-based EIC container images in GitLab CI to catch size regressions early during builds.

Changes:

  • Introduce SIZE_LIMIT_EIC_CUDA_GIB and SIZE_LIMIT_EIC_DEV_CUDA_GIB CI variables to enable the existing image-size check for eic_cuda and eic_dev_cuda.

Comment thread .gitlab-ci.yml Outdated
@github-actions

github-actions Bot commented Jul 5, 2026

Copy link
Copy Markdown

Capybara summary for PR 337

Copilot AI review requested due to automatic review settings July 10, 2026 10:30

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread .gitlab-ci.yml Outdated
@wdconinc
wdconinc requested review from Copilot and removed request for Copilot July 26, 2026 19:49
Copilot AI review requested due to automatic review settings July 31, 2026 19:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

.gitlab-ci.yml:40

  • The size-limit guidance comment still says to update limits "here and in .github/workflows/build-push.yml env", but this file now contains CUDA-specific SIZE_LIMIT_* entries that are not (and per prior discussion, need not be) mirrored in GitHub Actions. This is misleading for future updates—please clarify that only the shared images' limits are updated in both places, while CUDA image limits are GitLab-only.
  ## in .github/workflows/build-push.yml env.
  SIZE_LIMIT_DEBIAN_STABLE_BASE_GIB: "0.9"
  SIZE_LIMIT_EIC_CI_GIB: "3.1"
  SIZE_LIMIT_EIC_XL_GIB: "4.6"
  SIZE_LIMIT_EIC_CUDA_GIB: "8.1"

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