Skip to content

Add job_submit.lua error codes and explanations - #562

Open
jreiland wants to merge 8 commits into
ResearchComputing:mainfrom
jreiland:jsl-errors
Open

Add job_submit.lua error codes and explanations#562
jreiland wants to merge 8 commits into
ResearchComputing:mainfrom
jreiland:jsl-errors

Conversation

@jreiland

Copy link
Copy Markdown
Contributor

No description provided.

@b-reyes b-reyes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is all looking fantastic. I just had some very minor comments. Mostly just on linking to appropriate sections in the Alpine hardware page.

Comment thread docs/running-jobs/error-status-codes.md Outdated
Comment thread docs/running-jobs/error-status-codes.md Outdated
Comment thread docs/running-jobs/error-status-codes.md Outdated
Comment thread docs/running-jobs/error-status-codes.md Outdated
Comment thread docs/running-jobs/error-status-codes.md Outdated
Comment thread docs/running-jobs/error-status-codes.md Outdated
Comment thread docs/running-jobs/error-status-codes.md Outdated
Comment thread docs/running-jobs/error-status-codes.md Outdated
Comment thread docs/running-jobs/error-status-codes.md Outdated
Comment thread docs/running-jobs/error-status-codes.md Outdated
Enhanced error messages with additional resources and clarifications for users regarding QoS, GRES, and job specifications.

@b-reyes b-reyes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I have just one minor change, take it or leave it. Otherwise, this looks great!

Comment thread docs/running-jobs/error-status-codes.md Outdated
Co-authored-by: Brandon  <53541061+b-reyes@users.noreply.github.com>
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