Skip to content

adding sharing=locked into /var/cache/spack mounts in containers/eic/Dockerfile - #413

Merged
wdconinc merged 1 commit into
masterfrom
fix/concurrent-var-spack-cache
Sep 9, 2026
Merged

wdconinc merged 1 commit into
masterfrom
fix/concurrent-var-spack-cache

Conversation

@SebastianPaucar

@SebastianPaucar SebastianPaucar commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

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

This PR sets sharing=locked on the /var/cache/spack cache mounts in containers/eic/Dockerfile, so it prevents concurrent builds from interfering with each other's cache operations.

This is intended to fix FileNotFoundError failures during eicweb/eic_container CI builds. Check this #388 (comment)

Draft for now. cc @wdconinc

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.

@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

Capybara summary for PR 413

@SebastianPaucar

Copy link
Copy Markdown
Contributor Author

Hi @wdconinc, is there an EICweb infrastructure issue? This PR’s push to EICweb fails with HTTP 502, and #388 had repeated GHCR connection timeouts during the EICweb CI, even though the concretization and build steps succeeded. Could these be related? Is there something I’m missing? Happy to dig into it from my side if possible

@wdconinc

wdconinc commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

is there an EICweb infrastructure issue?

Yes, we've had some backend instability yesterday, and today the whole frontend went down as well.

@SebastianPaucar

Copy link
Copy Markdown
Contributor Author

is there an EICweb infrastructure issue?

Yes, we've had some backend instability yesterday, and today the whole frontend went down as well.

Could the heavy autopush workload from #388 have contributed to the EICweb backend instability, or are the GHCR timeouts likely unrelated? Just trying to understand what happened

@wdconinc

wdconinc commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Could the heavy autopush workload from #388 have contributed to the EICweb backend instability, or are the GHCR timeouts likely unrelated? Just trying to understand what happened

This is likely unrelated. The autopush functionality hardly contributes to bandwidth to the registry on eicweb at all. Even though we have probably O(10k) tags in the OCI buildcache, most of them are a few 10 MB at most. What drives the eicweb registry load is the publishing of docker layer caches, primarily those of the CUDA-enabled images which are O(20 GB) large.

@wdconinc wdconinc 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.

I'm willing to just merge this in master, since the only way we will be able to assess if it works is not in a PR but in production by keeping an eye on these intermittent failures.

If you are ok with it, please just mark as ready for review, and I will merge it.

@SebastianPaucar

Copy link
Copy Markdown
Contributor Author

I'm willing to just merge this in master, since the only way we will be able to assess if it works is not in a PR but in production by keeping an eye on these intermittent failures.

If you are ok with it, please just mark as ready for review, and I will merge it.

Sounds good, I’ll mark it ready for review. Let me know how the production testing goes.

@SebastianPaucar
SebastianPaucar marked this pull request as ready for review September 9, 2026 21:07
@wdconinc
wdconinc merged commit 18c33fc into master Sep 9, 2026
83 of 86 checks passed
@wdconinc
wdconinc deleted the fix/concurrent-var-spack-cache branch September 9, 2026 21:13
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