Skip to content

[ci] Set provenance: false in docker/build-push-action#3222

Closed
joshlf wants to merge 4 commits intoG3xo5e4llm76omanefzdnc2lopucwvv3bfrom
Gag74v3izqtosz7f5k5mqsawpiskp3yrm
Closed

[ci] Set provenance: false in docker/build-push-action#3222
joshlf wants to merge 4 commits intoG3xo5e4llm76omanefzdnc2lopucwvv3bfrom
Gag74v3izqtosz7f5k5mqsawpiskp3yrm

Conversation

@joshlf
Copy link
Copy Markdown
Member

@joshlf joshlf commented Apr 8, 2026

This works around a buggy interaction between this action and the GitHub
Actions cache which is caused by uploading provenance information for
the SLSA framework. Since we don't publish these Docker images for
external consumption, we don't need provenance information.


Latest Update: v2 — Compare vs v1

📚 Full Patch History

Links show the diff between the row version and the column version.

Version v1 Base
v2 vs v1 vs Base
v1 vs Base
⬇️ Download this PR

Branch

git fetch origin refs/heads/Gag74v3izqtosz7f5k5mqsawpiskp3yrm && git checkout -b pr-Gag74v3izqtosz7f5k5mqsawpiskp3yrm FETCH_HEAD

Checkout

git fetch origin refs/heads/Gag74v3izqtosz7f5k5mqsawpiskp3yrm && git checkout FETCH_HEAD

Cherry Pick

git fetch origin refs/heads/Gag74v3izqtosz7f5k5mqsawpiskp3yrm && git cherry-pick FETCH_HEAD

Pull

git pull origin refs/heads/Gag74v3izqtosz7f5k5mqsawpiskp3yrm

Stacked PRs enabled by GHerrit.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.85%. Comparing base (cee467e) to head (e1df673).

Additional details and impacted files
@@                        Coverage Diff                         @@
##           G3xo5e4llm76omanefzdnc2lopucwvv3b    #3222   +/-   ##
==================================================================
  Coverage                              91.85%   91.85%           
==================================================================
  Files                                     20       20           
  Lines                                   6067     6067           
==================================================================
  Hits                                    5573     5573           
  Misses                                   494      494           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@joshlf joshlf force-pushed the G5iiniran72qxcjixnuxghrjo5paoe57n branch from b16c1a8 to 23171d1 Compare April 8, 2026 06:38
joshlf added 4 commits April 8, 2026 06:40
In `autoParam` automatic proofs, insert `unfold` and `grind` tactics in
various places. `unfold` is crucial – without it, *only* statements
which are universally true for all possible implementations of the
function signature can be automatically proven. `grind` is an SMT-like
tactic which is able to automatically discharge a wide range of proofs.

Closes #3210
Closes #3213

gherrit-pr-id: G5iiniran72qxcjixnuxghrjo5paoe57n
gherrit-pr-id: Gjw64g3i4n2om5ehu57es67buaxfywnqf
It seems as though `Diagnostics.lean` had been causing the Lean compiler
to read all of Mathlib into RAM. The default behavior is more optimized,
and so this change allows us to lower the RAM footprint estimate for the
Lean compiler from 16GB to 2GB (actually from 8GB to 1GB, but we double
as a safety margin) in our integration tests. In theory this should let
us run ~8x as many tests in parallel as before.

gherrit-pr-id: G3xo5e4llm76omanefzdnc2lopucwvv3b
This works around a buggy interaction between this action and the GitHub
Actions cache which is caused by uploading provenance information for
the SLSA framework. Since we don't publish these Docker images for
external consumption, we don't need provenance information.

gherrit-pr-id: Gag74v3izqtosz7f5k5mqsawpiskp3yrm
@joshlf joshlf force-pushed the Gag74v3izqtosz7f5k5mqsawpiskp3yrm branch from df7827b to e1df673 Compare April 8, 2026 06:40
@joshlf joshlf force-pushed the G5iiniran72qxcjixnuxghrjo5paoe57n branch from 23171d1 to 5b0cb64 Compare April 8, 2026 06:40
@joshlf joshlf changed the base branch from G5iiniran72qxcjixnuxghrjo5paoe57n to G3xo5e4llm76omanefzdnc2lopucwvv3b April 8, 2026 06:40
@joshlf joshlf force-pushed the G3xo5e4llm76omanefzdnc2lopucwvv3b branch 7 times, most recently from 14c6ff1 to b64ab4d Compare April 8, 2026 08:09
@joshlf joshlf closed this Apr 8, 2026
@joshlf joshlf deleted the Gag74v3izqtosz7f5k5mqsawpiskp3yrm branch April 8, 2026 14:28
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