Skip to content

Add Open Cluster Scheduler / Gridware Cluster Scheduler to the grid#6

Open
dgruber wants to merge 2 commits into
converged-computing:mainfrom
dgruber:dg-add-ocs-gcs-workload-manager
Open

Add Open Cluster Scheduler / Gridware Cluster Scheduler to the grid#6
dgruber wants to merge 2 commits into
converged-computing:mainfrom
dgruber:dg-add-ocs-gcs-workload-manager

Conversation

@dgruber

@dgruber dgruber commented May 27, 2026

Copy link
Copy Markdown

No description provided.

@dgruber

dgruber commented May 27, 2026

Copy link
Copy Markdown
Author

@vsoch

vsoch commented May 28, 2026

Copy link
Copy Markdown
Member

Awesome! I will test this soon.

@vsoch

vsoch commented Jun 2, 2026

Copy link
Copy Markdown
Member

Ran into an issue with testing: hpc-gridware/quickinstall#10 (posting cross-reference for easy access / provenance)!

@vsoch vsoch left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great! Can confirm I can detect it in the docker compose environment.

Image

If you are able to find Handles discovery and status for the SGE-family schedulers: and replace the : with a . we can consider that a bug fix. Does this have any commands to dispatch (for example, submit, job info, cancel, etc.)?

"""
if not self.bin_path:
return {"installed": False}
return {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any chance this might change, so it would be advisable to set as instance class properties instead of discovering dynamically?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you answer this question about the environment variables? Any reason to not set on probe?

str: Output of 'qhost'. Use this to see physical capacity and
current load before proposing a job.
"""
tool = shutil.which("qhost")

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We might want to set this on probe too.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would have it set as an attribute so in the subsequent functions you can just reference self.qconf

@dgruber

dgruber commented Jun 10, 2026

Copy link
Copy Markdown
Author

@vsoch sorry for the late response and thanks for your comments. Added caching, removed the colon, and added the dispatch tool.

"""
if not self.bin_path:
return {"installed": False}
return {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you answer this question about the environment variables? Any reason to not set on probe?

return self.run_workload_cmd([self.qconf, "-spl"]) if self.qconf else "qconf not found"

@dispatch_tool
def submit_job(self, arguments: str) -> str:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any kind of validation checks you want to do before submission?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants