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

Heroku Review Apps 自動生成の ON/OFF について #1381

Open
yasulab opened this issue Sep 5, 2021 · 0 comments
Open

Heroku Review Apps 自動生成の ON/OFF について #1381

yasulab opened this issue Sep 5, 2021 · 0 comments
Labels
設計・議論 Topics to discuss for better business.

Comments

@yasulab
Copy link
Member

yasulab commented Sep 5, 2021

現在、Heroku Review Apps の自動生成は 📴 になっています。Review Apps が必要になる場面はあまり多くないので、特に OFF のままでも問題ないかなと考えていますが、自動生成の Pros/Cons を軽くあげておきます 🤔💭

Heroku Review Apps の前提条件と Pros/Cons

前提条件

  • 誰でも PR を送れるので、環境変数を Reveal する PR を送って Review Apps で確認できる状態はダメ ㊙️ 💦
  • したがって、少なくとも Review Apps へのアクセス制限を掛けるか、もしくはホワイトリスト形式で一部の環境変数のみを使うカタチにする必要がある 🔒
  • しかしほとんどの PR は簡単の情報の更新なので、Review Apps が必要な場面がそもそも多くない(権限を持つ人が、欲しい時に手動で生成しても良さそう) 👀 💭

Pros/Cons

  • Pros
    • デザインやシステムを変更する PR があったとき、ブラウザから簡単に確認できる
  • Cons
    • Review Apps が不要な PR に対しても、Review Apps が作成されるため、生成したインスタンスに応じた金額を支払う必要がある

🤔💭 @yasulab の意見

「Review Apps が 緊急で 必要だ!」という場面になってから Review Apps の準備をしても、致命的なダメージを負う可能性はほとんど無いかなと考えています。なので そういった場面になるまでは対応しなくて良い と考えています 😌

@yasulab yasulab added the 設計・議論 Topics to discuss for better business. label Sep 5, 2021
@yasulab yasulab changed the title Heroku Review Apps を自動生成する? Heroku Review Apps を自動生成の ON/OFF について Sep 5, 2021
@yasulab yasulab changed the title Heroku Review Apps を自動生成の ON/OFF について Heroku Review Apps 自動生成の ON/OFF について Sep 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
設計・議論 Topics to discuss for better business.
Projects
None yet
Development

No branches or pull requests

1 participant