Skip to content

Dockerfile: pin alpine to 3.22, use nonroot users#1124

Draft
makramkd wants to merge 2 commits into
mainfrom
mk/CCIP-11487
Draft

Dockerfile: pin alpine to 3.22, use nonroot users#1124
makramkd wants to merge 2 commits into
mainfrom
mk/CCIP-11487

Conversation

@makramkd
Copy link
Copy Markdown
Collaborator

Description

Testing

Checklist

  • Breaking changes documented in changelog (see changelog directory)
  • Cross link related PRs (in this or other repositories)
  • just lint fix - no new lint errors
  • just generate - mocks and protobufs are up to date

@github-actions
Copy link
Copy Markdown

Code coverage report:

Package main mk/CCIP-11487 Diff
github.com/smartcontractkit/chainlink-ccv/aggregator 49.35% 49.33% -0.02%
github.com/smartcontractkit/chainlink-ccv/bootstrap 54.14% 54.14% +0.00%
github.com/smartcontractkit/chainlink-ccv/cli 65.13% 65.13% +0.00%
github.com/smartcontractkit/chainlink-ccv/cmd 15.54% 15.54% +0.00%
github.com/smartcontractkit/chainlink-ccv/common 56.54% 56.54% +0.00%
github.com/smartcontractkit/chainlink-ccv/executor 45.97% 45.97% +0.00%
github.com/smartcontractkit/chainlink-ccv/indexer 37.68% 37.68% +0.00%
github.com/smartcontractkit/chainlink-ccv/integration 45.84% 45.93% +0.09%
github.com/smartcontractkit/chainlink-ccv/pkg 84.62% 84.62% +0.00%
github.com/smartcontractkit/chainlink-ccv/pricer 0.00% 0.00% +0.00%
github.com/smartcontractkit/chainlink-ccv/protocol 63.75% 63.75% +0.00%
github.com/smartcontractkit/chainlink-ccv/verifier 34.48% 34.48% +0.00%
Total 46.50% 46.50% +0.00%

Comment thread verifier/Dockerfile
CGO_ENABLED=0 go build -ldflags='-s -w' -o /bin/verifier main.go

FROM alpine:latest
FROM alpine:3.22
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Any reason not to use 3.23?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Oh right, didn't see it. I just saw the top tag and it was 3.22. Will change.

Copy link
Copy Markdown
Collaborator

@winder winder left a comment

Choose a reason for hiding this comment

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

Changes seem reasonable, but consider the latest version of alpine.

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