-
Notifications
You must be signed in to change notification settings - Fork 11
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
Fixed parse eks region #92
Conversation
Signed-off-by: David Wertenteil <dwertent@armosec.io>
PR Description updated to latest commit (23cdfaa) |
PR Review
✨ Review tool usage guide:Overview:
With a configuration file, use the following template:
See the review usage page for a comprehensive guide on using this tool. |
PR Code Suggestions
✨ Improve tool usage guide:Overview:
With a configuration file, use the following template:
See the improve usage page for a more comprehensive guide on using this tool. |
Summary:
|
1 similar comment
Summary:
|
@@ -125,8 +125,10 @@ func (eksSupport *EKSSupport) GetRegion(cluster string) (string, error) { | |||
splittedClusterContext := strings.Split(cluster, "-") | |||
if len(splittedClusterContext) < 4 { | |||
return "", fmt.Errorf("failed to get region") | |||
} else { | |||
} else if len(splittedClusterContext) >= 6 { |
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.
holy ****
User description
Here is the error log:
Type
bug_fix, enhancement
Description
Changes walkthrough
ekssupport.go
Enhanced EKS Region Parsing Logic
cloudsupport/v1/ekssupport.go
determined.
ekssupport_test.go
Comprehensive Tests for EKS Region Parsing
cloudsupport/v1/ekssupport_test.go