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

Makefile: use buildx component to build images with BuildKit #3009

Merged

Conversation

End-rey
Copy link
Contributor

@End-rey End-rey commented Nov 12, 2024

The GH workflow uses the build-push-action, where images are built with buildx. So I made it like this in our Makefile.
Also, fix warnings in Dockerfiles.

Closes #3006.

WARN: FromAsCasing: 'as' and 'FROM' keywords' casing do not match (line 1)

Signed-off-by: Andrey Butusov <andrey@nspcc.io>
https://docs.docker.com/engine/deprecated/#legacy-builder-fallback
```
DEPRECATED: The legacy builder is deprecated and will be removed in a future release.
            Install the buildx component to build images with BuildKit:
            https://docs.docker.com/go/buildx/
```
Use `buildx` directly like in the GH workflow.

Closes #3006.

Signed-off-by: Andrey Butusov <andrey@nspcc.io>
Copy link

codecov bot commented Nov 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 23.06%. Comparing base (007e992) to head (9f80a9d).
Report is 28 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3009   +/-   ##
=======================================
  Coverage   23.06%   23.06%           
=======================================
  Files         790      790           
  Lines       58709    58709           
=======================================
  Hits        13542    13542           
  Misses      44285    44285           
  Partials      882      882           

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

Copy link
Member

@carpawell carpawell left a comment

Choose a reason for hiding this comment

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

Are you sure it is a solution? As I understand, the client just asks to install some additional plugin, not sure Makefile is wrong here and we are doing incorrect things.

@End-rey
Copy link
Contributor Author

End-rey commented Nov 13, 2024

Are you sure it is a solution? As I understand, the client just asks to install some additional plugin, not sure Makefile is wrong here and we are doing incorrect things.

I didn't have any mistake initially. But I set the value DOCKER_BUILDKIT=0 and then the message started to appear. What I did in the commit fixed the situation.

@roman-khimov roman-khimov merged commit b7714ae into master Nov 13, 2024
21 of 22 checks passed
@roman-khimov roman-khimov deleted the 3006-buildx-component-to-build-images-with-buildkit branch November 13, 2024 11:29
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.

legacy builder is deprecated, install the buildx component to build images with BuildKit
4 participants