FAQ | This is a LIVE service | Changelog

Skip to content
Snippets Groups Projects

allow webapp service account id to be customised and SQL instance to be blank

Merged Dr Rich Wareham requested to merge issue-6-custom-service-account-id into master
All threads resolved!

As noted in #6 (closed), we were hard-coding the service account id used for the webapp to "webapp-run". This meant it was impossible to deploy more than one webapp in a project.

As noted in #5 (closed), sometimes we didn't want to associate a SQL instance with the webapp.

Form a better default from the "name" variable. For existing deployments the service account id will be unchanged if the "name" variable is at its default value.

Allow the service account id to be specified explicitly via the optional "service_account_id" variable.

Allow sql_instance_connection_name to be blank. If so, we don't set the Cloud SQL annotation on the service or attempt to grant the service account roles.

Closes #6 (closed) Closes #5 (closed)

Edited by Dr Rich Wareham

Merge request reports

Pipeline #33071 passed

Pipeline passed for 61a5460e on issue-6-custom-service-account-id

Approved by

Merged by Dr Abraham MartinDr Abraham Martin 4 years ago (Jun 16, 2020 7:59am UTC)

Loading

Pipeline #33072 passed

Pipeline passed for 38156df5 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Dr Abraham Martin resolved all threads

    resolved all threads

  • Dr Abraham Martin approved this merge request

    approved this merge request

  • Dr Rich Wareham resolved all threads

    resolved all threads

  • added 1 commit

    Compare with previous version

  • Dr Abraham Martin approved this merge request

    approved this merge request

  • mentioned in commit 38156df5

  • Please register or sign in to reply
    Loading