-
Notifications
You must be signed in to change notification settings - Fork 7
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
workflowにBuildjetを追加 #2008
workflowにBuildjetを追加 #2008
Conversation
id: meta | ||
uses: docker/metadata-action@v4 | ||
with: | ||
images: ${{ steps.login-ecr.outputs.registry }}/dreamkast-ecs |
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.
[actionlint] reported by reviewdog 🐶
property "login-ecr" is not defined in object type {} [expression]
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.
id: login-ecr
がここより下に存在するため、上記メッセージの通りDocker meta
でコケるように見えます
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.
Docker meta
の実行結果より ${{ steps.login-ecr.outputs.registry }}
が空欄となっており機能していませんでしたので修正しました。
通知に関しては、statusページで使われているBetterStackがFeedを提供しているので、これをSlackなりなんなりで通知させることで受け取れそう |
merge-images内のcheckoutが不要であるため削除 merge-images内のlogin-ecrの実行箇所を変更
fix: image_tag_refを修正
Graviton対応に向けて、ARM対応イメージをビルドするBuildjetを追加する。
懸念事項