Skip to content

Commit 99a83ff

Browse files
committed
Remove extraneous comments
1 parent 0a8bec4 commit 99a83ff

File tree

3 files changed

+2
-10
lines changed

3 files changed

+2
-10
lines changed

pkg/monitor/sqsevent/sqs-monitor.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -353,7 +353,7 @@ func (m SQSMonitor) getNodeInfo(instanceID string) (*NodeInfo, error) {
353353
}
354354

355355
if m.CheckIfManaged {
356-
if _, ok := nodeInfo.Tags[m.ManagedTag]; !ok { // austin: wait, the presence of this should be a good sign...how did this ever work?
356+
if _, ok := nodeInfo.Tags[m.ManagedTag]; !ok {
357357
nodeInfo.IsManaged = false
358358
}
359359
}

pkg/monitor/sqsevent/sqs-monitor_internal_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ func TestGetNodeInfo_ASG(t *testing.T) {
134134
h.Equals(t, true, nodeInfo.IsManaged)
135135
}
136136

137-
func TestGetNodeInfo_ASG_ASGManaged(t *testing.T) { // austin: may be root cause of failures
137+
func TestGetNodeInfo_ASG_ASGManaged(t *testing.T) {
138138
asgName := "test-asg"
139139
managedTag := "aws-nth/managed"
140140
tags := map[string]string{managedTag: "", "aws:autoscaling:groupName": asgName}

pkg/monitor/sqsevent/sqs-monitor_test.go

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -902,11 +902,3 @@ func mockIsManagedFalse(asg *h.MockedASG) h.MockedASG {
902902
}
903903
return *asg
904904
}
905-
906-
func mockIsManagedErr(asg *h.MockedASG) h.MockedASG {
907-
if asg == nil {
908-
asg = &h.MockedASG{}
909-
}
910-
asg.DescribeAutoScalingInstancesErr = fmt.Errorf("error")
911-
return *asg
912-
}

0 commit comments

Comments
 (0)