-
Notifications
You must be signed in to change notification settings - Fork 10
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
alpine: add support for PostgreSQL 17 #61
Conversation
c3c384d
to
5d5b310
Compare
89ecdb6
to
1595d2c
Compare
.github/workflows/build.yml
Outdated
if ! docker manifest inspect ${tag} > /dev/null; then | ||
platforms="linux/amd64" | ||
need_load=yes | ||
fi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
リポジトリにイメージがないときはローカルのイメージで動くようにしてみましたがどうだろうか。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
動いていそう。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
これ、常にローカルのやつを使うようにできる?
常にビルドしたやつでテストしたい。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pushしないときはローカルのイメージでテストするようにしました。
platformsがlinux/amd64,linux/arm64
だとローカルにloadするときにエラーになるので、linux/amd64
のみにしています。
ERROR: docker exporter does not currently support exporting manifest lists
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
うっす。
alpine-17-slim
は別PRでやる?ここでやる?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
別でやります!
No description provided.