-
Notifications
You must be signed in to change notification settings - Fork 205
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
docs: Update mng-cluster-provider.md to remove typo #847
Conversation
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.
@gdoteof Thankyou for the PR. Please check our comments.
@@ -9,7 +9,6 @@ const props: MngClusterProviderProps = { | |||
minSize: 1, | |||
maxSize: 10, | |||
desiredSize: 4, | |||
version: "auto" |
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.
This line should not be removed. Rather version: KubernetesVersion.V1_25,
line should be removed if you want to update it. Applies to other part of the doc too.
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.
+1 seems more reasonable not to hardcode the 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.
@gdoteof please address the comment, will merge after that.
This PR has been automatically marked as stale because it has been open 60 days |
@gdoteof Can you fix the above comment. We are planning to release soon! |
This PR has been automatically marked as stale because it has been open 60 days |
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.
Upon further review the version is expect as KubernetesVersion so has to either be removed or type changed to eks.KubernetesVersion.
Approving,
closing in favor of #1023 |
Issue #, if available:
Description of changes:
It looks like a mass update to the readme failed here.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.