Skip to content

feat(examples/templates/gcp-windows): add GCP region module #15552

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

Merged
merged 3 commits into from
Nov 28, 2024

Conversation

phorcys420
Copy link
Member

@phorcys420 phorcys420 commented Nov 16, 2024

This PR adds the GCP region module.

  • Testing
    • Workspace gets provisioned and region selector works

@github-actions github-actions bot added the stale This issue is like stale bread. label Nov 24, 2024
@github-actions github-actions bot closed this Nov 28, 2024
@phorcys420 phorcys420 reopened this Nov 28, 2024
@phorcys420 phorcys420 marked this pull request as ready for review November 28, 2024 04:01
@phorcys420
Copy link
Member Author

Works properly, available on dogfood @ https://dev.coder.com/templates/coder/gcp-windows/versions/promote-modules-pr-6863

@phorcys420 phorcys420 enabled auto-merge (squash) November 28, 2024 04:05
@phorcys420
Copy link
Member Author

phorcys420 commented Nov 28, 2024

The template actually runs the agent as NT Authority\System, we might want to fix that in the future lol

image

@phorcys420 phorcys420 requested review from matifali, deansheather and johnstcn and removed request for deansheather November 28, 2024 04:12
@phorcys420 phorcys420 disabled auto-merge November 28, 2024 04:33
@phorcys420 phorcys420 enabled auto-merge (squash) November 28, 2024 04:34
@matifali matifali disabled auto-merge November 28, 2024 05:02
@matifali matifali merged commit 7a7db9f into main Nov 28, 2024
55 checks passed
@matifali matifali deleted the phorcys/promote-modules-gcp-windows branch November 28, 2024 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale This issue is like stale bread.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants