-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
fix(terraform): update CKV_AWS_339 valid EKS versions #5652
Conversation
EKS 1.23 ended support on 11 Oct 2023. Include 1.28 as valid version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thansk for the contribution 💪
@@ -24,7 +24,7 @@ def get_inspected_key(self) -> str: | |||
return "version" | |||
|
|||
def get_expected_values(self) -> list[Any]: | |||
return ["1.23", "1.24", "1.25", "1.26", "1.27"] | |||
return ["1.24", "1.25", "1.26", "1.27", "1.28"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
return ["1.24", "1.25", "1.26", "1.27", "1.28"] | |
return ["1.23", "1.24", "1.25", "1.26", "1.27", "1.28"] |
I'm fine with adding 1.28
, but not about removing 1.23
, because AWS offers extended support for an additional 12 months.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Understood!
EKS 1.23 extended support works as 11 Oct 2024. Include 1.28 as valid version. By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. fix(terraform): update CKV_AWS_339 valid EKS versions Description The change in valid versions, as now 1.24 is the lowest supported version and 1.28 the newest available. Fix Update EKS version to a valid version Checklist: My code follows the style guidelines of this project I have performed a self-review of my own code I have commented my code, particularly in hard-to-understand areas I have made corresponding changes to the documentation I have added tests that prove my feature, policy, or fix is effective and works New and existing tests pass locally with my changes Any dependent changes have been merged and published in downstream modules
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks 🏅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
* Update EKSPlatformVersion.py EKS 1.23 ended support on 11 Oct 2023. Include 1.28 as valid version. * fix(terraform): update CKV_AWS_339 valid EKS versions EKS 1.23 extended support works as 11 Oct 2024. Include 1.28 as valid version. By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. fix(terraform): update CKV_AWS_339 valid EKS versions Description The change in valid versions, as now 1.24 is the lowest supported version and 1.28 the newest available. Fix Update EKS version to a valid version Checklist: My code follows the style guidelines of this project I have performed a self-review of my own code I have commented my code, particularly in hard-to-understand areas I have made corresponding changes to the documentation I have added tests that prove my feature, policy, or fix is effective and works New and existing tests pass locally with my changes Any dependent changes have been merged and published in downstream modules * add missing space --------- Co-authored-by: Anton Grübel <[email protected]>
EKS 1.23 ended support on 11 Oct 2023.
Include 1.28 as valid version.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Description
The change in valid versions, as now 1.24 is the lowest supported version and 1.28 the newest available.
Fix
Update EKS version to a valid version
Checklist: