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[typing-server]: 接続するDBのアドレスを環境変数で指定できるように #13

Merged
merged 3 commits into from
Mar 12, 2024

Conversation

h-takeyeah
Copy link
Collaborator

チケットへのリンク

(省略)

やったこと

  • データベースのアドレスを環境変数 DB_ADDR から設定するように変更した
    • 動作確認のために Cloud Run にデプロイするが、その段階で希望の値をセットしたかったため

やらないこと

  • ドキュメントの整備
    • 一応、書いたには書いたんですがここに含めると時間がかかりそうだったので省略しています

できるようになること(ユーザ目線)

  • 無し

できなくなること(ユーザ目線)

  • 無し

動作確認

  • 異なる環境変数を渡して go のプログラムが異なる接続先に接続しに行くことを確認した
# Error response from daemon: Address already in use と出るので
# ここで api サービスは削除しておきます (docker compose up -d api すれば元に戻ります)
$ docker compose down api

🙆‍♂️ まず適当なアドレスを指定すると繋がらないことを確認

docker compose run --rm --env DB_ADDR=192.168.0.1:3306 api
[+] Creating 1/0
 ✔ Container typing-server-db-1  Running                                                                                                                                               0.0s
2024/03/10 05:04:21 INFO ent client is opened
2024/03/10 05:04:23 ERROR failed to create schema: %v !BADKEY="mysql: querying mysql version dial tcp 192.168.0.1:3306: connect: connection refused"

🙆‍♂️ 次に docker-compose.yml の db サービスに割り当てられたアドレス(db や typing-server-db-1 も同様)で繋がることを確認

$ docker compose run --rm --env DB_ADDR=172.28.1.5:3306 api
[+] Creating 1/0
 ✔ Container typing-server-db-1  Running                                                                                                                                               0.0s
2024/03/10 05:00:11 INFO ent client is opened
2024/03/10 05:00:11 INFO schema is created
2024/03/10 05:00:11 INFO server is running at Addr :8080
^C2024/03/10 05:00:12 INFO received signal: %s !BADKEY=interrupt
^C2024/03/10 05:00:13 INFO shutting down the server...
2024/03/10 05:00:13 INFO server exited

@h-takeyeah h-takeyeah changed the title 接続するDBのアドレスを環境変数で指定できるように fix[typing-server]: 接続するDBのアドレスを環境変数で指定できるように Mar 10, 2024
Copy link
Contributor

@shin0729 shin0729 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM(というかこれを否とする知識がない)

@h-takeyeah h-takeyeah merged commit b60c7ed into develop Mar 12, 2024
1 check passed
@h-takeyeah h-takeyeah deleted the fix/backend/fixed-db-addr branch March 12, 2024 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants