Skip to content
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

Check for k8s version in the Cloudstack template name #7130

Conversation

panktishah26
Copy link
Member

@panktishah26 panktishah26 commented Dec 5, 2023

Description of changes:
Template name field in each cloudstack machine object should contain Cluster object's kubernetes version and/or worker machine's kubernetes version.

Simple Upgrade:

  • Control plane machine's template name should contain cluster object's kubernetes version
  • External Etcd (if present) machine's template name should contain cluster object's kubernetes version
  • Worker machine's template name should contain cluster object's kubernetes version

Modular Upgrade:

  • Control plane machine's template name should contain cluster object's kubernetes version
  • External Etcd (if present) machine's template name should contain cluster object's kubernetes version
  • Worker machine's template name should contain cluster object's kubernetes version or Worker machine's template name should contain workerNodeGroupConfigurations's kubernetes version

Testing (if applicable):

Documentation added/planned (if applicable):

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@eks-distro-bot eks-distro-bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Dec 5, 2023
@eks-distro-bot eks-distro-bot added lgtm and removed lgtm labels Dec 5, 2023
@panktishah26 panktishah26 force-pushed the cloudstack-k8s-version-template-0.17 branch from 7b8b033 to a0e1173 Compare December 5, 2023 21:10
@panktishah26 panktishah26 force-pushed the cloudstack-k8s-version-template-0.17 branch from a0e1173 to 82ec8f4 Compare December 5, 2023 21:15
Copy link

codecov bot commented Dec 5, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ac6d85b) 75.51% compared to head (82ec8f4) 75.53%.

Additional details and impacted files
@@               Coverage Diff                @@
##           release-0.17    #7130      +/-   ##
================================================
+ Coverage         75.51%   75.53%   +0.02%     
================================================
  Files               455      455              
  Lines             37922    37951      +29     
================================================
+ Hits              28635    28667      +32     
+ Misses             7676     7674       -2     
+ Partials           1611     1610       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@panktishah26
Copy link
Member Author

/test eks-anywhere-presubmit

@cxbrowne1207
Copy link
Member

/approve

@eks-distro-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cxbrowne1207

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@cxbrowne1207
Copy link
Member

/lgtm

@eks-distro-bot eks-distro-bot merged commit 5037656 into aws:release-0.17 Dec 6, 2023
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved lgtm size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants