Skip to content

docs: Change Cloud Scheduler region from us-central1 to us-east1#948

Merged
camiekim merged 2 commits intoterraform-google-modules:mainfrom
camiekim:b/507114306
Apr 28, 2026
Merged

docs: Change Cloud Scheduler region from us-central1 to us-east1#948
camiekim merged 2 commits intoterraform-google-modules:mainfrom
camiekim:b/507114306

Conversation

@camiekim
Copy link
Copy Markdown
Contributor

Description

Fixes b/507114306

Note: If you are not associated with Google, open an issue for discussion before submitting a pull request.

Checklist

Readiness

  • [x ] Yes, merge this PR after it is approved
  • No, don't merge this PR after it is approved

Style

Testing

  • [ x] I have performed tests described in the Contributing guide:

    • [ x] Tests pass: terraform apply
    • [ x] Lint pass: terraform fmt check

Intended location

API enablement

  • If the sample needs an API enabled to pass testing, I have added the service to the Test setup file

Review

  • If this sample adds a new directory, I have added codeowners to the CODEOWNERS file

@camiekim camiekim requested review from a team as code owners April 28, 2026 12:43
@iennae
Copy link
Copy Markdown
Contributor

iennae commented Apr 28, 2026

/gcbrun

@iennae
Copy link
Copy Markdown
Contributor

iennae commented Apr 28, 2026

this needs some outside of the region tag changes to address timing issues since it's attempting to delete before it's ready. you could do something like this PR with the time_sleep and the null_resource to ensure that it doesn't attempt to delete to early 364192b#diff-d2d5e41ff94877d4a3388fbad8bf95dd71a3ef1e704d3f1d5d90fe4b5dd5cc1f

In a customer's environment they wouldn't be creating/deleting this quickly so it should be outside the region tag.

Added a sleep resource to wait for the Cloud Scheduler job to complete before proceeding
@camiekim
Copy link
Copy Markdown
Contributor Author

Thanks, @iennae !

@camiekim camiekim merged commit 88647d9 into terraform-google-modules:main Apr 28, 2026
4 checks passed
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