Skip to content

feat: Wrap components as prep for container upgrade#5604

Merged
Bownairo merged 2 commits into
masterfrom
eero/container-env
Jun 23, 2025
Merged

feat: Wrap components as prep for container upgrade#5604
Bownairo merged 2 commits into
masterfrom
eero/container-env

Conversation

@Bownairo

@Bownairo Bownairo commented Jun 17, 2025

Copy link
Copy Markdown
Contributor
  1. We depend on the system strip when compiling or linking. Updating this binary in [NODE-1606] Upgrade build container to 24.04 #5615 has an effect on outputs that was not seen in the build graph. This adds a dependency on the system strip binary to the hermetic_cc linker and compiler.
  2. podman export behaves differently after [NODE-1606] Upgrade build container to 24.04 #5615. This adds an additional layer of sanitizing so that builds before and after the upgrade are consistent.

@github-actions github-actions Bot added the feat label Jun 17, 2025
@Bownairo Bownairo force-pushed the eero/container-env branch 5 times, most recently from 12e1cc9 to b033a1a Compare June 19, 2025 15:13
@Bownairo Bownairo force-pushed the eero/container-env branch from b033a1a to fd0ef9f Compare June 19, 2025 21:44
@Bownairo Bownairo marked this pull request as ready for review June 19, 2025 23:12
@Bownairo Bownairo requested review from a team as code owners June 19, 2025 23:12
Comment thread bazel/hermetic_cc_toolchain.patch
@Bownairo Bownairo added this pull request to the merge queue Jun 23, 2025
Merged via the queue into master with commit d184511 Jun 23, 2025
32 checks passed
@Bownairo Bownairo deleted the eero/container-env branch June 23, 2025 21:53
github-merge-queue Bot pushed a commit that referenced this pull request Jun 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants