Skip to content

Commit b0fab0a

Browse files
committed
disable resource monitor
1 parent 5f45507 commit b0fab0a

File tree

1 file changed

+10
-6
lines changed

1 file changed

+10
-6
lines changed

apps/supervisor/src/index.ts

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,10 @@ class ManagedSupervisor {
9999
this.logger.warn("[ManagedWorker] Failed pod handler disabled");
100100
}
101101

102-
this.resourceMonitor = new KubernetesResourceMonitor(createK8sApi(), "");
102+
this.resourceMonitor = new KubernetesResourceMonitor(
103+
createK8sApi(),
104+
env.TRIGGER_WORKER_INSTANCE_NAME
105+
);
103106
this.workloadManager = new KubernetesWorkloadManager(workloadManagerOptions);
104107
} else {
105108
this.resourceMonitor = new DockerResourceMonitor(new Docker());
@@ -117,7 +120,7 @@ class ManagedSupervisor {
117120
runNotificationsEnabled: env.TRIGGER_WORKLOAD_API_ENABLED,
118121
preDequeue: async () => {
119122
if (this.isKubernetes) {
120-
// TODO: Test k8s resource monitor and remove this
123+
// Not used in k8s for now
121124
return {};
122125
}
123126

@@ -235,10 +238,11 @@ class ManagedSupervisor {
235238
snapshotFriendlyId: message.snapshot.friendlyId,
236239
});
237240

238-
this.resourceMonitor.blockResources({
239-
cpu: message.run.machine.cpu,
240-
memory: message.run.machine.memory,
241-
});
241+
// Disabled for now
242+
// this.resourceMonitor.blockResources({
243+
// cpu: message.run.machine.cpu,
244+
// memory: message.run.machine.memory,
245+
// });
242246
} catch (error) {
243247
this.logger.error("[ManagedWorker] Failed to create workload", { error });
244248
}

0 commit comments

Comments
 (0)