Conversation
- Create .envrc for environment variable management - Implement bootstrap.sh for VM setup and OpenClaw installation - Define main.tf for Azure resources including VM, networking, and security groups - Add openclaw.json.tpl for OpenClaw configuration - Set up outputs.tf for VM details and health check URL - Introduce poll_b1s_availability.sh for SKU availability checks - Configure provider.tf and variables.tf for Azure authentication and resource parameters
…2 VM - Deleted old configuration files related to B1s VM including openclaw.json.tpl, outputs.tf, poll_b1s_availability.sh, provider.tf, and variables.tf. - Added new configuration files for B2pts_v2 VM including main.tf, outputs.tf, provider.tf, and variables.tf. - Updated bootstrap script to accommodate new VM specifications and configurations. - Introduced .envrc for environment variable management. - Adjusted OpenClaw configuration to reflect changes in VM type and resource allocation.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 7 changed files in this pull request and generated 9 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| variable "subscription_id" { | ||
| sensitive = true | ||
| } | ||
|
|
||
| variable "client_id" { | ||
| sensitive = true | ||
| } | ||
|
|
||
| variable "client_secret" { | ||
| sensitive = true | ||
| } | ||
|
|
||
| variable "tenant_id" { | ||
| sensitive = true | ||
| } |
There was a problem hiding this comment.
Several variables omit an explicit type (and some omit description). For Azure auth values, omitting type = string makes them any, which can allow unintended types and leads to less helpful validation/errors. Also telegram_owner_id has a default but no type, which makes the interface less clear (string vs number). Add explicit type (and preferably descriptions) for these variables to make the module contract clearer and error messages more actionable.
| variable "telegram_owner_id" { | ||
| description = "Your Telegram numeric user ID (get it from @userinfobot). Grants /model and other privileged commands." | ||
| default = "" | ||
| } |
There was a problem hiding this comment.
Several variables omit an explicit type (and some omit description). For Azure auth values, omitting type = string makes them any, which can allow unintended types and leads to less helpful validation/errors. Also telegram_owner_id has a default but no type, which makes the interface less clear (string vs number). Add explicit type (and preferably descriptions) for these variables to make the module contract clearer and error messages more actionable.
| # Auto-load secrets from .env into Terraform variables | ||
| # Requires: brew install direnv && eval "$(direnv hook zsh)" >> ~/.zshrc | ||
|
|
||
| dotenv ../../.env |
There was a problem hiding this comment.
Using dotenv ../../.env will error when the .env file is absent, which can break direnv allow for new setups/CI checkouts. Use a guarded load (e.g., dotenv_if_exists) or a file-existence check so the environment can still be entered without local secrets.
…Claw - Removed deprecated inline Python code for approving operator approvals and replaced it with a separate script. - Updated the bootstrap script to create and execute the new approval script. - Adjusted cron job timings to reflect Pacific Time. - Cleaned up main.tf by removing old Azure resource configurations, streamlining the file for future updates.
- Added Azure as a deployment option alongside DigitalOcean. - Updated deployment instructions to reflect changes in supported platforms. - Included specific configuration details for Azure VM setup.
No description provided.