Skip to content

Commit

Permalink
add disable_check for pbs, lsf executors and add implementation logic…
Browse files Browse the repository at this point in the history
… for pbs, lsf, torque executors
  • Loading branch information
shahzebsiddiqui committed Jun 12, 2024
1 parent c0ffe44 commit e7832c5
Show file tree
Hide file tree
Showing 2 changed files with 25 additions and 5 deletions.
18 changes: 15 additions & 3 deletions buildtest/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -267,7 +267,11 @@ def _validate_lsf_executors(self):
if self.is_executor_disabled(lsf_executors[executor]):
self.disabled_executors.append(executor_name)
continue

if lsf_executors[executor].get("disable_check"):
self.valid_executors[executor_type][executor_name] = {
"setting": lsf_executors[executor]
}
continue
if not lsf.validate_queue(executor=lsf_executors[executor]):
self.invalid_executors.append(executor_name)
continue
Expand Down Expand Up @@ -388,7 +392,11 @@ def _validate_pbs_executors(self):
if self.is_executor_disabled(pbs_executor[executor]):
self.disabled_executors.append(executor_name)
continue

if pbs_executor[executor].get("disable_check"):
self.valid_executors[executor_type][executor_name] = {
"setting": pbs_executor[executor]
}
continue
queue = pbs_executor[executor].get("queue")
if not pbs.validate_queue(queue):
self.invalid_executors.append(executor_name)
Expand Down Expand Up @@ -423,7 +431,11 @@ def _validate_torque_executors(self):
if self.is_executor_disabled(torque_executor[executor]):
self.disabled_executors.append(executor_name)
continue

if torque_executor[executor].get("disable_check"):
self.valid_executors[executor_type][executor_name] = {
"setting": torque_executor[executor]
}
continue
if not torque.validate_queue(torque_executor[executor]):
self.invalid_executors.append(executor_name)
continue
Expand Down
12 changes: 10 additions & 2 deletions buildtest/schemas/settings.schema.json
Original file line number Diff line number Diff line change
Expand Up @@ -581,7 +581,11 @@
"$ref": "#/definitions/account"
},
"disable": {"$ref": "#/definitions/disable"},
"module": { "$ref": "#/definitions/module" }
"module": { "$ref": "#/definitions/module" },
"disable_check": {
"type": "boolean",
"description": "Disable check for partition or qos "
}
}
},
"pbs": {
Expand Down Expand Up @@ -613,7 +617,11 @@
"$ref": "#/definitions/account"
},
"disable": {"$ref": "#/definitions/disable"},
"module": { "$ref": "#/definitions/module" }
"module": { "$ref": "#/definitions/module" },
"disable_check": {
"type": "boolean",
"description": "Disable check for partition or qos "
}
}
},
"container": {
Expand Down

0 comments on commit e7832c5

Please sign in to comment.