Skip to content

Commit 6d15ab8

Browse files
authored
remove pods from checking status, if they are no longer found, update default chart version (#570)
1 parent 7d26358 commit 6d15ab8

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

clients/js/packages/client/src/client.ts

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ export interface PodPorts {
4949
}
5050

5151
const defaultName: string = 'starship';
52-
const defaultVersion: string = 'v0.2.14';
52+
const defaultVersion: string = 'v0.2.19';
5353

5454
// TODO talk to Anmol about moving these into yaml, if not already possible?
5555
const defaultPorts: PodPorts = {
@@ -558,6 +558,13 @@ export class StarshipClient implements StarshipClientI {
558558
public async waitForPods(): Promise<void> {
559559
const podNames = this.getPodNames();
560560

561+
// Remove pods that are no longer active from the podStatuses map
562+
this.podStatuses.forEach((_value, podName) => {
563+
if (!podNames.includes(podName)) {
564+
this.podStatuses.delete(podName);
565+
}
566+
});
567+
561568
// Check the status of each pod retrieved
562569
podNames.forEach((podName) => {
563570
this.checkPodStatus(podName);
@@ -658,7 +665,6 @@ export class StarshipClient implements StarshipClientI {
658665
const eventLines = eventsSection
659666
.split('\n')
660667
.filter((line) => line.trim() !== '');
661-
this.log(`event lints: ${eventLines.join('\n')}`);
662668

663669
return eventLines;
664670
}

0 commit comments

Comments
 (0)