feat: Add GKE CustomNodeInit support#17743
Conversation
Adds support for GKE custom node initialization scripts configuration in LinuxNodeConfig. This includes: - Adding custom_node_init schema to linux_node_config block. - Implementing expander and flattener helpers for CustomNodeInit and InitScript. - Adding TestAccContainerNodePool_withCustomNodeInit integration test template. This supports both GCS URI (with specific generation) and GCP Secret Manager secret URI configurations. TAG=agy CONV=753c21e2-16d4-4cc3-9aef-bf8b374d142c
|
Googlers: For automatic test runs see go/terraform-auto-test-runs. @trodge, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look. You can help make sure that review is quick by doing a self-review and by running impacted tests locally. |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 46d0291: Diff reportYour PR generated the following diffs in downstream repositories:
Missing test reportYour PR includes resource fields which are not covered by any test. Resource: resource "google_container_cluster" "primary" {
node_config {
linux_node_config {
custom_node_init {
init_script {
gcp_secret_manager_secret_uri = # value needed
gcs_generation = # value needed
gcs_uri = # value needed
}
}
}
}
node_pool {
node_config {
linux_node_config {
custom_node_init {
init_script {
gcp_secret_manager_secret_uri = # value needed
gcs_generation = # value needed
gcs_uri = # value needed
}
}
}
}
}
}
Resource: resource "google_container_node_pool" "primary" {
node_config {
linux_node_config {
custom_node_init {
init_script {
gcp_secret_manager_secret_uri = # value needed
gcs_generation = # value needed
}
}
}
}
}
Missing doc report (experimental)The following resources have fields missing in documents.
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 3 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
Caution Issues requiring attention before PR completion 🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details. Please address these issues to complete your PR. If you believe these detections are incorrect or unrelated to your change, please raise the concern with your reviewer. View the recording VCR build log or the debug logs folder for detailed results. @zicongmei, @trodge VCR tests complete for 46d0291! |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit c22df28: Diff reportYour PR generated the following diffs in downstream repositories:
Missing doc report (experimental)The following resources have fields missing in documents.
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 6 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
Caution Issues requiring attention before PR completion 🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details. Please address these issues to complete your PR. If you believe these detections are incorrect or unrelated to your change, please raise the concern with your reviewer. View the recording VCR build log or the debug logs folder for detailed results. @zicongmei, @trodge VCR tests complete for c22df28! |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 14d8fc8: Diff reportYour PR generated the following diffs in downstream repositories:
Missing doc report (experimental)The following resources have fields missing in documents.
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 6 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
Caution Issues requiring attention before PR completion 🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details. Please address these issues to complete your PR. If you believe these detections are incorrect or unrelated to your change, please raise the concern with your reviewer. View the recording VCR build log or the debug logs folder for detailed results. @zicongmei, @trodge VCR tests complete for 14d8fc8! |
|
Current test failure: |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit a7e8cd8: Diff reportYour PR generated the following diffs in downstream repositories:
Missing doc report (experimental)The following resources have fields missing in documents.
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 7 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
Caution Issues requiring attention before PR completion 🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details. Please address these issues to complete your PR. If you believe these detections are incorrect or unrelated to your change, please raise the concern with your reviewer. View the recording VCR build log or the debug logs folder for detailed results. @zicongmei, @trodge VCR tests complete for a7e8cd8! |
Moves GKE Cluster and GKE Node Pool node_config stubs and custom_node_init nested blocks from the bottom sections (User Project Overrides / Import) up to the correct locations under the Argument Reference section. This fixes the document layout semantics for human readers while maintaining the paths expected by the static doc-validator tool. TAG=agy CONV=753c21e2-16d4-4cc3-9aef-bf8b374d142c
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit e6a9966: Diff reportYour PR generated the following diffs in downstream repositories:
Missing doc report (experimental)The following resources have fields missing in documents.
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 7 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
Caution Issues requiring attention before PR completion 🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details. Please address these issues to complete your PR. If you believe these detections are incorrect or unrelated to your change, please raise the concern with your reviewer. View the recording VCR build log or the debug logs folder for detailed results. @zicongmei, @trodge VCR tests complete for e6a9966! |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 37de94b: Diff reportYour PR generated the following diffs in downstream repositories:
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 7 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
Caution Issues requiring attention before PR completion 🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details. Please address these issues to complete your PR. If you believe these detections are incorrect or unrelated to your change, please raise the concern with your reviewer. View the recording VCR build log or the debug logs folder for detailed results. @zicongmei, @trodge VCR tests complete for 37de94b! |
…edirects Reverts GKE Cluster and GKE Node Pool path stubs and parameter link redirect updates. This leaves both documents in a clean Path A state (similar to GKE containerd_config): - GKE Custom Node Init is fully documented under GKE Cluster's linux_node_config. - GKE Node Pool document continues to rely on the clean global redirect to GKE Cluster for its node_config schema, without maintaining duplicate stubs or parser-specific helper anchors. TAG=agy CONV=753c21e2-16d4-4cc3-9aef-bf8b374d142c
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 02715d5: Diff reportYour PR generated the following diffs in downstream repositories:
Missing doc report (experimental)The following resources have fields missing in documents.
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 7 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
Caution Issues requiring attention before PR completion 🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details. Please address these issues to complete your PR. If you believe these detections are incorrect or unrelated to your change, please raise the concern with your reviewer. View the recording VCR build log or the debug logs folder for detailed results. @zicongmei, @trodge VCR tests complete for 02715d5! |
Declares the new CustomNodeInit leaf fields under GKE Cluster and GKE Node Pool resource metadata configs. This fixes the static downstream validation check (TestValidateResourceMetadata in allservices package) which ensures all Go schema definitions have matching metadata mappings. TAG=agy CONV=753c21e2-16d4-4cc3-9aef-bf8b374d142c
|
Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 48a38b4: Diff reportYour PR generated the following diffs in downstream repositories:
Missing doc report (experimental)The following resources have fields missing in documents.
Test reportAnalytics
Affected Service Packages
Step 1: Replaying Mode Action takenFound 7 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
View the replaying VCR build log Step 2: Recording Mode
Caution Issues requiring attention before PR completion 🔴 Initial Recording Failed: Some tests failed during the recording step. See the table above for details. Please address these issues to complete your PR. If you believe these detections are incorrect or unrelated to your change, please raise the concern with your reviewer. View the recording VCR build log or the debug logs folder for detailed results. @zicongmei, @trodge VCR tests complete for 48a38b4! |
Adds support for GKE custom node initialization scripts configuration in LinuxNodeConfig. This includes:
This supports both GCS URI (with specific generation) and GCP Secret Manager secret URI configurations.
TAG=agy
CONV=753c21e2-16d4-4cc3-9aef-bf8b374d142c
Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.