PR

【5/12】docker構築処理更新 #2768

wada opened this at 2023-05-08T17:00:57; closed

Description

opened by wada at 2023-05-08T17:00:57; updated 2023-05-15T11:41:45

概要

アサイン準備 #2759

docker構築に必要なテーブル情報の更新(パイレーツパスシーズン1まで)
今後、テーブル追加や構造に調整が入る場合、各種.sqlファイルを更新してマイグレーションの代わりとする

不要な設定を削除

Comment

#27312 by wada at 2023-05-08T17:01:08; updated 2023-05-08T17:01:08

@kouho
レビューお願いします。

Comment

#27400 by kouho at 2023-05-12T10:12:05; updated 2023-05-12T10:12:05

@wada

今後、テーブル追加や構造に調整が入る場合、各種.sqlファイルを更新してマイグレーションの代わりとする

こちらは運用上どうしましょう?実装者はプルリクにsqlファイルの更新を含めるようにこれからはして、含め忘れていたらレビュー者が指摘するとかになるのでしょうか?

Comment

#27401 by wada at 2023-05-12T10:17:16; updated 2023-05-12T10:17:16

@kouho
まだ考えていないですが、以下の対応のどれかになるかと思います。

  • 更新ごとに追記
  • 更新内容のみを別ファイルで作成
  • 一定期間ごとにダンプ

Comment

#27402 by kouho at 2023-05-12T10:30:48; updated 2023-05-12T10:30:48

かしこまりました。
> まだ考えていないですが、以下の対応のどれかになるかと思います。
>
更新ごとに追記
更新内容のみを別ファイルで作成
一定期間ごとにダンプ

@wada
以下質問と確認です。

1. mysqlのlower_case_table_namesは何に設定されていますか?
こちらで作り直したローカル環境だと.my.cnfを設定していない状態だと2になり、テーブルの大文字小文字が区別されませんでした。
開発と本番と同じ0に設定するとローカルのmysqlサーバーが起動時に以下のエラーが出て起動できず、mysqlのlower_case_table_namesを1にすると起動できました。

The server option 'lower_case_table_names' is configured to use case sensitive table names but the data directory is on a case-insensitive file system which is an unsupported combination. Please consider either using a case sensitive file system for your data directory or switching to a case-insensitive table name mode.

2. card_status_dataの最新のインポートはどうしましょう?
こちらの今のローカルの運用だと基本的に足りてないデータはなかったですがビョルンなどの入っていないカードは開発からエクスポートしてインポートしていました。
card_no 1310以降が現在のmaster_card_data/ 以下に入っていないように見えます。

Comment

#27403 by wada at 2023-05-12T10:40:03; updated 2023-05-12T10:40:11

@kouho
> mysqlのlower_case_table_namesは何に設定されていますか?

my.cnfに設定していないのであればmysqlのデフォルト値の2になりますね。
my.cnfに記載しておきます。

> card_status_dataの最新のインポートはどうしましょう?

どうしましょう。また考えておきます。

wada close_comment at 2023-05-15T11:41:45: developマージ済み

Commit range

base62b86d57de011049557ba26d107894b191551be0
head28603c88e875ca25e2bfaa4625545a6c36e52954
base branchjolly/jr-web:master
head branchjolly/jr-web:prepare_db_install

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed9 filesView patch
statusfile
modifiedDocker_ds/README.md
modifiedDocker_ds/docker-compose.yml
addedDocker_ds/mysql57/conf.d/.my.cnf
modifieddatabase/install.sh
modifieddatabase/jrye.sql
modifieddatabase/jrye_cpanel.sql
modifieddatabase/jrye_log.sql
modifieddatabase/jrye_teamWar.sql
modifieddatabase/official_info.sql
Patch body is stored as a separate file to keep this page lightweight.