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

DEPLOYMENT.mdにNEW_RELIC_LICENSE_KEYなどの環境変数の情報を追加する #193

Closed
takahashim opened this issue Mar 21, 2021 · 3 comments
Labels
enhancement New feature or request

Comments

@takahashim
Copy link
Collaborator

改善詳細 / Details of Improvement

#186 でnew relicのgemが追加されましたが、これを実際に動かすには環境変数 NEW_RELIC_LICENSE_KEYNEW_RELIC_APP_NAME の設定が必要なはずです(config/newrelic.ymlで参照してるみたいです)。

これはDecidim本家のサイトのデプロイ設定・手順等を見ても分からないことなので、設定が必要な情報はどこかに記しておきたいです。現状では DEPLOYMENT.md に書くのが良いかと思うのですが、どうでしょうか(他にも必要な項目がありそうなのでまとめて書いておきたい)。

対象インスタンス / Instances

  • 全て
@takahashim takahashim added the enhancement New feature or request label Mar 21, 2021
@halsk
Copy link
Member

halsk commented Mar 21, 2021

そうですね。各種設定値がどこにどう設定されているかは、DEPLOYMENT.md に記載するようにしましょうか。

@komtaki
Copy link
Contributor

komtaki commented May 15, 2021

基本的には、ソースコードに落とした方がいいと思います。:eyes:

ただeb configの環境変数は、秘密鍵等がパラメーターストアで隠蔽できなさそうなんですよね。パスワードとか丸見えなのはまずいので、一旦DEPLOYMENT.mdですかね。 💦

https://docs.aws.amazon.com/ja_jp/elasticbeanstalk/latest/dg/command-options-general.html#command-options-general-elasticbeanstalkapplicationenvironment

ちょうどコンテナ化で手を入れるので、ざっと私が追記しておきます。

@komtaki
Copy link
Contributor

komtaki commented Aug 12, 2021

こちらドキュメントではないですが、コードに落として可視化されたのでクローズします。

@komtaki komtaki closed this as completed Aug 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants