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.

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

Pin docker/dockerfile:1.6.0 with multi-platform sha256 #4766

Merged
merged 2 commits into from
Jan 16, 2024
Merged

Pin docker/dockerfile:1.6.0 with multi-platform sha256 #4766

merged 2 commits into from
Jan 16, 2024

Conversation

jkreileder
Copy link
Contributor

Pull Request check list

  • Commit conforms to CONTRIBUTING.md?
  • Proper tests/regressions included?
  • Documentation updated?

Affected functionality
Docker build on platforms other than linux/amd64

Description of change
Set docker/dockerfile:1.6.0 sha256 to the multi-platform manifest instead of referencing the linux/amd64 version.
Cf. "crane digest docker/dockerfile:1.6.0" vs "crane digest --platform linux/amd64 docker/dockerfile:1.6.0"

Which issue this PR fixes
#4765

Cf. "crane digest docker/dockerfile:1.6.0" vs "crane digest --platform linux/amd64 docker/dockerfile:1.6.0"

Signed-off-by: Jürgen Kreileder <jk@blackdown.de>
@jkreileder
Copy link
Contributor Author

The "PR Build / unit-test (windows) (pull_request)" fail seems to be unrelated. It's failing for other PRs as well.

@evan2645 evan2645 merged commit ce7efce into spiffe:main Jan 16, 2024
32 checks passed
@azdagron azdagron added this to the 1.9.0 milestone Jan 21, 2024
faisal-memon pushed a commit to faisal-memon/spire that referenced this pull request Feb 7, 2024
Cf. "crane digest docker/dockerfile:1.6.0" vs "crane digest --platform linux/amd64 docker/dockerfile:1.6.0"

Signed-off-by: Jürgen Kreileder <jk@blackdown.de>
Signed-off-by: Faisal Memon <fymemon@yahoo.com>
sriyer pushed a commit to spire-vault/spire that referenced this pull request Feb 23, 2024
Cf. "crane digest docker/dockerfile:1.6.0" vs "crane digest --platform linux/amd64 docker/dockerfile:1.6.0"

Signed-off-by: Jürgen Kreileder <jk@blackdown.de>
rushi47 pushed a commit to rushi47/spire that referenced this pull request Apr 11, 2024
Cf. "crane digest docker/dockerfile:1.6.0" vs "crane digest --platform linux/amd64 docker/dockerfile:1.6.0"

Signed-off-by: Jürgen Kreileder <jk@blackdown.de>
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.

4 participants