Skip to content

feat: write infra file after gcp project creation#330

Merged
joka134 merged 11 commits intomainfrom
feat/bootstrap-write-checkpoint-infra-file
Apr 20, 2026
Merged

feat: write infra file after gcp project creation#330
joka134 merged 11 commits intomainfrom
feat/bootstrap-write-checkpoint-infra-file

Conversation

@joka134
Copy link
Copy Markdown
Member

@joka134 joka134 commented Apr 16, 2026

  • Move infra file functions into gcp package
  • Write infra file for the first time just after project creation

This PR saves some GCP infra details in the gcp-infra.json after creating the GCP project, so that bootstrap-gcp cleanup can be used in case of an unexpected termination.

https://app.clickup.com/t/869cn7e4k

joka134 and others added 2 commits April 16, 2026 14:50
Signed-off-by: joka134 <27293650+joka134@users.noreply.github.com>
@joka134 joka134 marked this pull request as ready for review April 16, 2026 12:55
Copy link
Copy Markdown
Member

@OliverTrautvetter OliverTrautvetter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice 👍 some comments

Comment thread internal/bootstrap/gcp/infrafile.go Outdated
Comment thread internal/bootstrap/gcp/infrafile.go Outdated
Comment thread internal/bootstrap/gcp/gcp.go
Comment thread cli/cmd/bootstrap_gcp.go Outdated
Comment thread internal/bootstrap/gcp/infrafile.go
Copy link
Copy Markdown
Member

@OliverTrautvetter OliverTrautvetter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good now, just one comment, still approved 🚀

Comment thread cli/cmd/bootstrap_gcp.go
joka134 and others added 2 commits April 20, 2026 13:27
@joka134 joka134 merged commit d2250b5 into main Apr 20, 2026
6 checks passed
@joka134 joka134 deleted the feat/bootstrap-write-checkpoint-infra-file branch April 20, 2026 11:59
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