Skip to content

Fix workload_policy varible definition and usage.#3963

Merged
mohitchaurasia91 merged 1 commit into
GoogleCloudPlatform:developfrom
mohitchaurasia91:fix-resource-policy-module
Apr 17, 2025
Merged

Fix workload_policy varible definition and usage.#3963
mohitchaurasia91 merged 1 commit into
GoogleCloudPlatform:developfrom
mohitchaurasia91:fix-resource-policy-module

Conversation

@mohitchaurasia91

@mohitchaurasia91 mohitchaurasia91 commented Apr 17, 2025

Copy link
Copy Markdown
Contributor

Added a check on var.workload_policy.type as its a required field

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 #

@mohitchaurasia91 mohitchaurasia91 added the release-module-improvements Added to release notes under the "Module Improvements" heading. label Apr 17, 2025
@mohitchaurasia91 mohitchaurasia91 self-assigned this Apr 17, 2025
@mohitchaurasia91 mohitchaurasia91 requested review from a team and samskillman as code owners April 17, 2025 17:07
@mohitchaurasia91 mohitchaurasia91 merged commit 219c0e2 into GoogleCloudPlatform:develop Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-module-improvements Added to release notes under the "Module Improvements" heading.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants