vvakame / docker-review

Re:VIEW build container by docker
MIT License
146 stars 38 forks source link

Re:VIEW 5.3, Debian bullseye更新 #65

Closed kmuto closed 2 years ago

kmuto commented 2 years ago

けっこうドラスティックで、5.2以前のはこれやると困る人もいそうなので、update.shからの対象外にしたほうがよさそうですね。→update.shを変更し、5.3.0からの対象とするようにしました。

kmuto commented 2 years ago

CI環境だとなぜかdebconf設定でおかしくなる…Debian側のバグっぽいけど、なんぞ。

kmuto commented 2 years ago

debconfではなくて

Setting up libpaper1:amd64 (1.1.28+b1) ...

Creating config file /etc/papersize with new version
stat: cannot statx '/etc/papersize.dpkg-inst': Operation not permitted
dpkg: error processing package libpaper1:amd64 (--configure):
 installed libpaper1:amd64 package post-installation script subprocess returned error exit status 1

なのだが、statxエラーはビルド側カーネルレベルの話で対処のしようが…。 Dockerのバージョンを上げるという方法があるようだが、クライアント側を上げるのではダメそう。

kmuto commented 2 years ago

うーむ、詰み状態です。なんか解決のアイデアありますか… @vvakame

kmuto commented 2 years ago

TechBooster/ReVIEW-Template側修正 https://github.com/TechBooster/ReVIEW-Template/pull/78

kmuto commented 2 years ago

action側のほうは通るようにしました (kmuto/ReVIEW-Template review530ブランチを利用)。 Circle CIのほうを引き続き考えますが… CircleCIの親環境のcontainerdが古いという問題なので、どうにかなるのだろうかこれ。

kmuto commented 2 years ago

@vvakame うーん、Circle CIのほうがどうにも動かないです。 jobs/buildでdockerでなくmachineにするといい みたいなものを見かけてはいるんですが、そうしたときに課金などが発生したりしないか心配しています。

vvakame commented 2 years ago

うぉーめちゃめちゃ頑張っていただいてる ありがとうございます Circle CIをまだ維持している理由を忘れつつあります…

https://circleci.com/docs/2.0/oss/

Note: There is a concurrency limit of 4 containers for Docker and Machine executors whereas macOS executors are limited to 1 container. Additional containers will be queued.

なので、Public repositoryの僕らかつビルド頻度は低いので machine を使ってもよさそうです。 ですがまぁCircle CIを切ってGitHub Actionsだけの運用にしてもいい気もします。

vvakame commented 2 years ago

Docker HubのAutomated Buildがアレになったので ghcr.io/vvakame/review にみんな移行していってほしい気がしなくもないですね…

vvakame commented 2 years ago

automated buildに頼るのやめて自分でpushするようにしようかな… そうするとtrusted buildじゃなくなるんだけど誰も気にしなさそうではある

kmuto commented 2 years ago

通ったよぉぉぉ :joy:

vvakame commented 2 years ago

やったー!大勝利!

kmuto commented 2 years ago

マージご判断お任せしますー。Hubのほうの設定などもあるのかな。

vvakame commented 2 years ago

descriptionにWIPって書いてありますが5.3ってリリースされた感じですか?

kmuto commented 2 years ago

あ、review 5.3 gemリリース済みです(一昨日)。description変えました

vvakame commented 2 years ago

お、承知しましたー