Merge pull request #681 from mattfarina/fix-667

Remove code causing node roles to be deleted

Author: mattfarina

src/k8s-engine/client.ts

@@ -533,20 +533,4 @@ export class KubeClient extends events.EventEmitter {
       });
     });
   }
-
-  /**
-   * Clean up obsolete nodes that match a predicate.
-   * @param predicate A function that returns true if a node should be removed.
-   */
-  async cleanupNodes(predicate: (node: k8s.V1Node) => boolean) {
-    const { body: nodeList } = await this.coreV1API.listNode();
-    const promises = nodeList
-      .items
-      .filter(predicate)
-      .map(node => node.metadata?.name)
-      .filter(defined)
-      .map(nodeName => this.coreV1API.deleteNode(nodeName));
-
-    return await Promise.all(promises);
-  }
 }

src/k8s-engine/lima.ts

@@ -755,10 +755,6 @@ export default class LimaBackend extends events.EventEmitter implements K8s.Kube
         this.emit('service-changed', services);
       });
 
-      // We need to remove obsolete nodes because we renamed the VM; otherwise
-      // they get stuck forever.  No need to wait for the result.
-      this.client.cleanupNodes(node => node.metadata?.name === 'lima-rancher-desktop');
-
       this.activeVersion = desiredShortVersion;
       this.currentPort = this.#desiredPort;
       this.emit('current-port-changed', this.currentPort);