Skip to content

Commit

Permalink
Merge pull request #153 from fahlmant/OSD-13430-sg-error
Browse files Browse the repository at this point in the history
Add specific error message to suggest cleanup of leftover SG
  • Loading branch information
openshift-merge-robot authored Oct 24, 2022
2 parents 8f1b16d + f741596 commit 612be8c
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion pkg/verifier/aws/entry_point.go
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,9 @@ func (a *AwsVerifier) ValidateEgress(vei verifier.ValidateEgressInput) *output.O
if cleanupSecurityGroup {
_, err := a.AwsClient.DeleteSecurityGroup(vei.Ctx, &ec2.DeleteSecurityGroupInput{GroupId: awsTools.String(vei.AWS.SecurityGroupId)})
if err != nil {
a.Output.AddError(err)
a.Output.AddError(handledErrors.NewGenericError(err))
a.Output.AddException(handledErrors.NewGenericError(fmt.Errorf("unable to cleanup security group %s, please manually clean up", vei.AWS.SecurityGroupId)))

}
}

Expand Down

0 comments on commit 612be8c

Please sign in to comment.