kubernetes
/
kops
mirror of https://github.com/kubernetes/kops.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

AWS IAM Role listing: don't ignore "other" errors 

If the error was an AWS error, but not one of the recognized ones, we
ignored it.
This commit is contained in:
justinsb 2022-09-01 07:57:03 -04:00
parent 8b83dedf24
commit 90a484f049
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
pkg/resources/aws/aws.go
View File

@ -2012,11 +2012,10 @@ func ListIAMRoles(cloud fi.Cloud, clusterName string) ([]*resources.Resource, er
klog.Warningf("could not find instance profile %q. Resource may already have been deleted: %v", name, awserror) klog.Warningf("could not find instance profile %q. Resource may already have been deleted: %v", name, awserror)
continue continue
} }
} else { }
getRoleErr = fmt.Errorf("calling IAM GetRole on %s: %w", name, err) getRoleErr = fmt.Errorf("calling IAM GetRole on %s: %w", name, err)
return false return false
} }
}
for _, tag := range roleOutput.Role.Tags { for _, tag := range roleOutput.Role.Tags {
if fi.StringValue(tag.Key) == ownershipTag && fi.StringValue(tag.Value) == "owned" { if fi.StringValue(tag.Key) == ownershipTag && fi.StringValue(tag.Value) == "owned" {
resourceTracker := &resources.Resource{ resourceTracker := &resources.Resource{