Skip to content

Use version prefix in conjunction with release channels#3520

Merged
annuay-google merged 1 commit into
GoogleCloudPlatform:developfrom
annuay-google:annuay/fix-min-master-version
Jan 9, 2025
Merged

Use version prefix in conjunction with release channels#3520
annuay-google merged 1 commit into
GoogleCloudPlatform:developfrom
annuay-google:annuay/fix-min-master-version

Conversation

@annuay-google

@annuay-google annuay-google commented Jan 9, 2025

Copy link
Copy Markdown
Contributor

Use min master version in conjunction with Rapid release channel

  • Pick the latest version on Rapid release channel with prefix 1.31
  • Maintenance exclusion and auto upgrade stays as it is today

Testing

  • Verified that the correct version (latest from Rapid) is picked by the cluster
  • Verified nodes are running the above latest version

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 #

@annuay-google annuay-google added the release-improvements Added to release notes under the "Improvements" heading. label Jan 9, 2025
Comment thread modules/scheduler/gke-cluster/main.tf
@annuay-google annuay-google force-pushed the annuay/fix-min-master-version branch from ac1b2f6 to 443b895 Compare January 9, 2025 22:33
@annuay-google annuay-google changed the title Fix min master version Use version prefix in conjunction with release channels Jan 9, 2025
@annuay-google annuay-google merged commit a0c6937 into GoogleCloudPlatform:develop Jan 9, 2025
@annuay-google annuay-google deleted the annuay/fix-min-master-version branch January 9, 2025 22:41
variable "version_prefix" {
description = "If provided, Terraform will only return versions that match the string prefix. For example, `1.31.` will match all `1.31` series releases. Since this is just a string match, it's recommended that you append a `.` after minor versions to ensure that prefixes such as `1.3` don't match versions like `1.30.1-gke.10` accidentally."
type = string
default = "1.31."

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think we should have a default value for this, GKE is continually upgrading and it has its own default version when customer not specifying

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants