Skip to content

GTFS追加 - #22

Open
hikaru-0602 wants to merge 16 commits into
mainfrom
feature/gtfs
Open

GTFS追加#22
hikaru-0602 wants to merge 16 commits into
mainfrom
feature/gtfs

Conversation

@hikaru-0602

Copy link
Copy Markdown
Contributor

やったこと

  • SecretManagerにGTFS(ShceduleとRealtime)追加
  • Cloud Run Job,Cloud Scheduler,SA,Cloud SQLの設定を追加

@hikaru-0602
hikaru-0602 requested review from a team, Hosoda-abo, kantacky and masaya-osuga August 6, 2026 15:51

@kantacky kantacky left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

SecretManager側でサービスアカウントにシークレットアクセサーロールを付与しないとかも?
あと、Cloud SQLのdotto_serviceロールをサービスアカウントに付与しないとかも

}

variable "schedule" {
default = "0 6 * * *"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

朝の3時でも良さそう
始発って6時くらいだっけ

Suggested change
default = "0 6 * * *"
default = "0 3 * * *"

service_account_display_name = "Import GTFS Job (Dev)"
db_name = "dotto_dev"
service_name = "import-gtfs-job-dev"
schedule = "0 4 * * *"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

devで前の日の夜にやってエラー出たら次の朝までに対応できそう

Suggested change
schedule = "0 4 * * *"
schedule = "0 21 * * *"

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