-
Notifications
You must be signed in to change notification settings - Fork 205
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
DockerビルドCI: イメージタグの別名を別々にビルドしないようにする #1498
DockerビルドCI: イメージタグの別名を別々にビルドしないようにする #1498
Conversation
リリーステストを通過したのでReady for Reviewにしましたが、 latestの場合の動作確認をしていなかったので確認中です。引数見る限りは期待通りに動いていそうです。 |
やっぱりlatestは動かなかったのでDraftに戻して再ビルド中です。 |
もともとコメントは過剰ぎみに書いていたので、削るのに賛成です! 反映したので再レビュー依頼します。 プログラムの引数の説明・関数の引数の説明を具体例にするか、REPO/VERにするかは悩んでいます。 もう一度変えてほしいなどでも指摘歓迎です。 |
…onize-hash-of-same-feature-image
masterブランチへの追従だけしておきました |
Dockerfileのビルド変数BASE_IMAGEのBASE_BUILD_IMAGEへのリネームについて考えていませんでした。 そこは未変更です。 |
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.
ほぼLGTMです!!
コメントに関してのコメントが主になっていますが、もうほとんど完成だと思います!
Co-authored-by: Hiroshiba <[email protected]>
Co-authored-by: Hiroshiba <[email protected]>
Co-authored-by: Hiroshiba <[email protected]>
Co-authored-by: Hiroshiba <[email protected]>
Co-authored-by: Hiroshiba <[email protected]>
Co-authored-by: Hiroshiba <[email protected]>
Co-authored-by: Hiroshiba <[email protected]>
特に問題ない項目でCIが落ちてしまっているので、修正するプルリクを作りました! 🙇 同じ変更をするか、↑を取り込むか、↑がマージされてから取り込んでいただければテストが通ると思います 🙇 |
取り込みます。 |
…feature/docker-commonize-hash-of-same-feature-image
取り込みました。 コメントとは関係ないんですが、tools/generate_docker_image_names.pyで |
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.
LGTM!!!
コメントのブラッシュアップありがとうございました!!
いくつかdocker周りでご提案いただいているのもぜひプルリクいただければと!
そして今回のような感じで進められればと…!🙏
一旦だけsuggestとりこんでマージします!
内容
DockerビルドCIで、イメージタグの別名を別々にビルドしないようにします。
の実装です。
matrix.tags
にカンマ区切りで設定します。matrix.os
があることによって、matrix.tag
のパターンを記述する必要が生じていた(書かないと実行されなかったからだった気がする)ので、matrix.os
を削除しました。Dockerイメージのビルドなので、Ubuntu以外で実行することはないかなと思います。latest-buildcache
,cpu-latest-buildcache
,cpu-ubuntu20.04-latest-buildcache
のように別名ごとにビルドキャッシュが作られていたと思いますが、cpu-ubuntu20.04-latest-buildcache
(長い形式)に統一するようにしました。統一する名前は、matrix.buildcache_tag
で設定します。matrix.tags
の一番最後とかでもいいかもですが、こちらの方が確実かなと思いました。TODO
関連 Issue
その他