Skip to content

Commit 50adf8f

Browse files
authored
fix synchronizing TGs (#146)
Signed-off-by: Egor Balakin <me@mironoff.msk.ru>
1 parent ae2ff2c commit 50adf8f

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

pkg/cloudprovider/yandex/load_balancer_tg_controller.go

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -117,6 +117,10 @@ func (ntgs *NodeTargetGroupSyncer) synchronizeNodesWithTargetGroups(ctx context.
117117
// TODO: speed up by not performing individual lookups
118118
var instances []*instanceWithNodeInfo
119119
for _, node := range nodes {
120+
if node.Spec.ProviderID == "" {
121+
return errors.Errorf("node %s ProviderID is empty", node.Name)
122+
}
123+
120124
if !(strings.Contains(node.Spec.ProviderID, "yandex")) {
121125
log.Printf("node %s ProviderID is not yandex (%s), skipping", node.Name, node.Spec.ProviderID)
122126
continue

0 commit comments

Comments
 (0)