Skip to content

Conversation

sanak
Copy link
Member

@sanak sanak commented Oct 30, 2022

@ishikawa999, @matobaa (CC: @tohosaku, @ananas3)
先日のパッチ会中、DB内のテーブルの値を確認する必要があり、取り急ぎ、 docker-compose.yml にPostgreSQLのポート番号をホスト側に公開し、ホスト側のpgAdmin4から接続することで対応したのですが、今後、同様の機会が頻繁にありそうな気がしましたので、パッチ会では普段使わないMySQL含めて、DB設定周りを少し変更してみました。

お手すきの際に確認頂けると助かります。

変更・修正内容

  1. PostgreSQLとMySQLのポート番号をホスト側に公開できるよう変更
    • いずれも、ホスト側で同じポート番号と衝突する可能性を考え、+1 した値を .env 内の初期値に設定しています。
  2. MySQLの設定変更
  3. VSCode SQLTools拡張の設定追加・変更
    • SQLToolsのMySQLドライバーもデフォルトで含めるようにし、接続情報も追加しました。
      • (名称は同じ app_development にしていますが、右側の接続情報からPostgreSQLかMySQLのどちらかが分かると考えてます。)
      • sql-tools-connections-list

@github-actions
Copy link

Patch can be downloaded here

@matobaa matobaa merged commit 9d03e53 into develop Feb 23, 2023
@matobaa matobaa deleted the fix/devcontainer-db-settings branch February 23, 2023 10:32
redmine-mirror-manager bot pushed a commit that referenced this pull request Feb 23, 2023
* Adjust docker-compose.yml MySQL settings and expose PostgreSQL/MySQL ports

* Adjust SQLTools MySQL driver and connections
@matobaa matobaa restored the fix/devcontainer-db-settings branch February 23, 2023 11:16
@sanak
Copy link
Member Author

sanak commented Feb 23, 2023

@matobaa (CC: @ishikawa999)
こちらもマージ頂き、ありがとうございます!🙇‍♂️

@ishikawa999
前回、Redmine4.x環境でプラグイン動作を確認する機会があり、以下の @ishikawa999 さんオリジナルの方を利用させていただきましたので、こちらの方にも後ほど、同様のPRを送らせて頂くかもしれません。🙇‍♂️
https://github.com/ishikawa999/redmine_development_docker

redmine-mirror-manager bot pushed a commit that referenced this pull request Mar 24, 2023
* Adjust docker-compose.yml MySQL settings and expose PostgreSQL/MySQL ports

* Adjust SQLTools MySQL driver and connections
redmine-mirror-manager bot pushed a commit that referenced this pull request Apr 22, 2023
* Adjust docker-compose.yml MySQL settings and expose PostgreSQL/MySQL ports

* Adjust SQLTools MySQL driver and connections
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants