Skip to content

Fix filter for gke-nodepool instance_templates#4559

Merged
sharabiani merged 2 commits into
GoogleCloudPlatform:developfrom
sharabiani:fix-np-name-filter
Aug 22, 2025
Merged

Fix filter for gke-nodepool instance_templates#4559
sharabiani merged 2 commits into
GoogleCloudPlatform:developfrom
sharabiani:fix-np-name-filter

Conversation

@sharabiani

Copy link
Copy Markdown
Collaborator

Submission Checklist

NOTE: Community submissions can take up to 2 weeks to be reviewed.

Please take the following actions before submitting this pull request.

  • Fork your PR branch from the Toolkit "develop" branch (not main)
  • Test all changes with pre-commit in a local branch #
  • Confirm that "make tests" passes all tests
  • Add or modify unit tests to cover code changes
  • Ensure that unit test coverage remains above 80%
  • Update all applicable documentation
  • Follow Cluster Toolkit Contribution guidelines #

@sharabiani sharabiani requested review from a team and samskillman as code owners August 21, 2025 05:55
@sharabiani sharabiani added the release-bugfix Added to release notes under the "Bug fixes" heading. label Aug 21, 2025
@sharabiani sharabiani enabled auto-merge August 21, 2025 09:37
Comment thread modules/compute/gke-node-pool/main.tf
samskillman
samskillman previously approved these changes Aug 21, 2025
@samskillman

Copy link
Copy Markdown
Collaborator

Please run the GKE PR tests as well after adding some comments in the code above. Reach out to me if needed.

@samskillman samskillman left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. Let's see the tests pass before merging

@sharabiani sharabiani merged commit 702865c into GoogleCloudPlatform:develop Aug 22, 2025
13 of 62 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-bugfix Added to release notes under the "Bug fixes" heading.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants