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

Fix: Streaming用Dockerfile分離に合わせたComposeファイルの修正 #485

Merged
merged 2 commits into from
Mar 8, 2024
Merged

Conversation

yuicho
Copy link

@yuicho yuicho commented Jan 18, 2024

UpstreamでDockerに関するファイル群(Dockerfiledocker-compose.yml)のリファクタリングが実施されています。

DockerfileにおけるリファクタリングのPR(mastodon#26850)はマージされ、この中でStreamingに使用するイメージのDockerfileを別ファイルへ分離する修正が行われています。
Upstreamでは、この修正に関連したcomposeファイルの対応を含めたリファクタリングを別PR(mastodon#28148)で進めていますが、こちらがまだマージされていません。

この状況でkmyblueでもUpstreamコミットを取り込んでいるため、composeファイルが正しく無い状態となっていました。
当PRではこれを修正しています。

@yuicho
Copy link
Author

yuicho commented Jan 18, 2024

当PRがマージされる前に本家の対応PRがマージされましたら、このPRはマージせずCloseしてしまってください。
(流石にこの状態でrelease発行することないと思うので、4.2.44.3.0出すまでにはマージされると思ってるのですが…)

また、当PRがマージされた後本家でも対応が完了した場合、本家のPRには他のリファクタリングも含まれますので本家を優先してください。

@kmycode kmycode added this to the kb11.0 milestone Jan 18, 2024
@kmycode kmycode added bug Something isn't working pending labels Jan 18, 2024
@kmycode
Copy link
Owner

kmycode commented Jan 18, 2024

本家が対応しなければバージョン11.0での対応とします

Copy link

This pull request has merge conflicts that must be resolved before it can be merged.

@kmycode kmycode modified the milestones: kb11.0, kb11.next Feb 17, 2024
@kmycode
Copy link
Owner

kmycode commented Feb 26, 2024

@yuicho
このPRは3月リリース予定のバージョンに入れる予定でしたが競合が起きているようです。
本家の当該PRもレビューなど進んでいるようですが、このPRを先に入れてほしいのでしたら3月8日までに修正してもらえると嬉しいです

ただこのPRを通した後に本家PRがマージされて手動マージ発生するようでしたら、Dockerの知識などないのでdocker-compose.ymlを本家のものに置き換えてしまう形になると思います

@yuicho
Copy link
Author

yuicho commented Mar 2, 2024

ただこのPRを通した後に本家PRがマージされて手動マージ発生するようでしたら、Dockerの知識などないのでdocker-compose.ymlを本家のものに置き換えてしまう形になると思います

上記全く問題ないと思います。
むしろ、今回のPRではまだ #484 については一切触ってないので、本家PRが終わった時点で一旦本家のものに置き換えるのが一番良いかと思われます。

Copy link

github-actions bot commented Mar 2, 2024

This pull request has resolved merge conflicts and is ready for review.

@kmycode kmycode removed the pending label Mar 8, 2024
@kmycode kmycode merged commit 7e7736b into kmycode:kb_development Mar 8, 2024
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants