-
Notifications
You must be signed in to change notification settings - Fork 773
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
Bootstrap k8s-infra-prow GCP project #6740
base: main
Are you sure you want to change the base?
Conversation
/hold to push further commits after initial approval |
/approve |
bbc7fd4
to
51a3d79
Compare
I deployed all the infra in the PR. /hold cancel |
The infra has been deployed, this PR is ready to merged |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left some non-blocking nits, feel free to discard them or fix them later
/lgtm
/approve
/hold
feel free to unhold when ready
type = "Delete" | ||
} | ||
condition = { | ||
age = 7 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice to document what is 7
exactly (e.g. days, months, etc).
version = "~> 30.2" | ||
project_id = module.project.project_id | ||
name = "prow" | ||
region = "us-central1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should eventually be a variable because we use it in many places
enable_private_nodes = true | ||
enable_cost_allocation = true | ||
gateway_api_channel = "CHANNEL_STANDARD" | ||
master_ipv4_cidr_block = "10.254.0.16/28" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice to document how did we choose that value
enable_l4_ilb_subsetting = true | ||
enable_private_nodes = true | ||
enable_cost_allocation = true | ||
master_ipv4_cidr_block = "10.254.0.0/28" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice to document how did we choose that value
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dims, upodroid, xmudrii The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This project will host 3 key services:
gcr.io/k8s-prow
replacement atus-docker.pkg.dev/k8s-infra-prow/images
Before Aug 2024, I want to have the images being dual published to both locations and the utility cluster to be fully operational.
/cc @ameukam @dims @BenTheElder