Skip to content

Releasing new charts version for v1.18.1#1104

Merged
jaydeokar merged 1 commit intoaws:masterfrom
jaydeokar:new_charts_v1.18
May 24, 2024
Merged

Releasing new charts version for v1.18.1#1104
jaydeokar merged 1 commit intoaws:masterfrom
jaydeokar:new_charts_v1.18

Conversation

@jaydeokar
Copy link
Contributor

Issue

N/A

Description of changes

This update the chart version for v1.18.1 and adds the FileOrCreate type for xtables-lock volume

Checklist

  • Added/modified documentation as required (such as the README.md for modified charts)
  • Incremented the chart version in Chart.yaml for the modified chart(s)
  • Manually tested. Describe what testing was done in the testing section below
  • Make sure the title of the PR is a good description that can go into the release notes

Testing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

apiVersion: v1
name: aws-vpc-cni
version: 1.18.1
version: 1.3.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this 1.3.0? https://github.com/aws/amazon-vpc-cni-k8s/blob/master/charts/aws-vpc-cni/Chart.yaml#L3 seems like we are matching the release version.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right, but it would fail if I keep it 1.18.1.. We have kept it different in the past

eks/aws-vpc-cni	1.13.0       	v1.13.0    	A Helm chart for the AWS VPC CNI
eks/aws-vpc-cni	1.2.8        	v1.12.6    	A Helm chart for the AWS VPC CNI
eks/aws-vpc-cni	1.2.7        	v1.12.5    	A Helm chart for the AWS VPC CNI
eks/aws-vpc-cni	1.2.6        	v1.12.2    	A Helm chart for the AWS VPC CNI
eks/aws-vpc-cni	1.2.2        	v1.12.1    	A Helm chart for the AWS VPC CNI

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah it looks odd, sure we can have it 1.3.0 for now given that we will have another release in few weeks

@Kyslik
Copy link

Kyslik commented May 23, 2024

👋🏻 ping.

(Can this be merged?)

@jaydeokar jaydeokar merged commit 9b36a39 into aws:master May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants