Commit 7e69426b authored by Justin Santa Barbara's avatar Justin Santa Barbara

AWS: Only warn about nameless instances if they are running

Otherwise this is just confusing logspam. Fix #20912
parent d089fa80
...@@ -2344,7 +2344,9 @@ func (a *AWSCloud) getInstancesByNodeNames(nodeNames []string) ([]*ec2.Instance, ...@@ -2344,7 +2344,9 @@ func (a *AWSCloud) getInstancesByNodeNames(nodeNames []string) ([]*ec2.Instance,
for _, instance := range allInstances { for _, instance := range allInstances {
nodeName := aws.StringValue(instance.PrivateDnsName) nodeName := aws.StringValue(instance.PrivateDnsName)
if nodeName == "" { if nodeName == "" {
glog.V(2).Infof("ignoring ec2 instance with no PrivateDnsName: %q", aws.StringValue(instance.InstanceId)) if isAlive(instance) {
glog.V(2).Infof("ignoring ec2 instance with no PrivateDnsName: %q", aws.StringValue(instance.InstanceId))
}
continue continue
} }
i, found := nodeNamesMap[nodeName] i, found := nodeNamesMap[nodeName]
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment