Skip to content

Commit

Permalink
fix(demand): removed default value for runtime version
Browse files Browse the repository at this point in the history
  • Loading branch information
mgordel committed Sep 4, 2024
1 parent 1e2c9de commit b59f0db
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ export class WorkloadDemandDirectorConfig extends BaseConfig {
readonly engine: string = "vm";
readonly runtime = {
name: "vm",
version: "0.3.0",
version: undefined,
};
readonly minMemGib: number = 0.5;
readonly minStorageGib: number = 2;
Expand All @@ -43,9 +43,6 @@ export class WorkloadDemandDirectorConfig extends BaseConfig {
if (!this.runtime.name) {
this.runtime.name = this.engine;
}
if (!this.runtime.version) {
this.runtime.version = "0.3.0";
}

this.expirationSec = options.expirationSec;

Expand Down
7 changes: 5 additions & 2 deletions src/market/demand/directors/workload-demand-director.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,11 @@ export class WorkloadDemandDirector implements IDemandDirector {

builder
.addProperty("golem.srv.comp.vm.package_format", this.config.packageFormat)
.addConstraint("golem.runtime.name", this.config.runtime.name)
.addConstraint("golem.runtime.version", this.config.runtime.version);
.addConstraint("golem.runtime.name", this.config.runtime.name);

if (this.config.runtime.version) {
builder.addConstraint("golem.runtime.version", this.config.runtime.version);
}

if (this.config.capabilities.length)
this.config.capabilities.forEach((cap) => builder.addConstraint("golem.runtime.capabilities", cap));
Expand Down

0 comments on commit b59f0db

Please sign in to comment.