Make possible to configure own vpc and subnet for cluster image build#2633
Conversation
PR SummaryMedium Risk Overview Reviewed by Cursor Bugbot for commit 5bd605a. Bugbot is set up for automated code reviews on this repo. Configure here. |
❌ 4 Tests Failed:
View the full list of 4 ❄️ flaky test(s)
To view more test analytics, go to the Test Analytics Dashboard |
There was a problem hiding this comment.
Code Review
The packer build command in the Makefile incorrectly assigns
There was a problem hiding this comment.
💡 Codex Review
With only the new defaults in this Makefile, NETWORK_NAME is no longer defined anywhere in the repo, so make build exports TF_VAR_network_name= to Terraform. The Terraform resources in main.tf use that value directly as the GCP network and firewall name, so a default build now tries to apply resources with an empty name before Packer runs.
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit d3cf149. Configure here.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d3cf149f9a
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c53c3624a8
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| @@ -1,8 +1,10 @@ | |||
| terraform { | |||
| required_version = ">= 1.5.0, < 1.6.0" | |||
| required_version = ">= 1.7.0, < 1.8.0" | |||
There was a problem hiding this comment.
please revert this as we use required_version = ">= 1.5.0, < 1.6.0" in all other places in this repository
There was a problem hiding this comment.
@djeebus is there a block to update the infra TF? Is it planned on your end, or should I do it?
There was a problem hiding this comment.
@jakubno i reverted required version back to >= 1.5.0, < 1.6.0 so we can merge

No description provided.