Skip to content

[iris] Document priority bands and warn on PRODUCTION#4749

Merged
rjpower merged 4 commits intomainfrom
agent/20260414-fix-4747
Apr 14, 2026
Merged

[iris] Document priority bands and warn on PRODUCTION#4749
rjpower merged 4 commits intomainfrom
agent/20260414-fix-4747

Conversation

@claude
Copy link
Copy Markdown
Contributor

@claude claude bot commented Apr 14, 2026

Add lib/iris/docs/priority-bands.md describing when to pick PRODUCTION, INTERACTIVE, or BATCH and the social contract for using PRODUCTION. CLI now logs a warning when --priority production is used so submitters know they are preempting other users' jobs.

Fixes #4747

Adds lib/iris/docs/priority-bands.md describing when to pick PRODUCTION,
INTERACTIVE, or BATCH and the social contract for using PRODUCTION. CLI
now logs a warning when --priority production is used so submitters know
they are preempting other users' jobs.

Fixes #4747
@claude claude bot added the agent-generated Created by automation/agent label Apr 14, 2026
@Helw150
Copy link
Copy Markdown
Member

Helw150 commented Apr 14, 2026

@claude remove all the CLI changes. Just change the docs.

@claude
Copy link
Copy Markdown
Contributor Author

claude bot commented Apr 14, 2026

Claude finished @Helw150's task in 1m 59s —— View job


Task: Remove CLI changes, keep only docs

  • Inspect current diff vs origin/main
  • Revert changes to lib/iris/src/iris/cli/job.py
  • Revert changes to lib/iris/tests/cli/test_job.py
  • Drop the docs sentence that referenced the (now-removed) CLI warning
  • Run ./infra/pre-commit.py --all-files --fix — passed
  • Commit and push

Summary

PR now contains only documentation changes:

 lib/iris/OPS.md                 |  2 +-
 lib/iris/docs/priority-bands.md | 53 +++++++++++++++++++++++++++++++++++++++++
 2 files changed, 54 insertions(+), 1 deletion(-)

The CLI warning behavior in lib/iris/src/iris/cli/job.py and its test in lib/iris/tests/cli/test_job.py were reverted to origin/main. I also removed the sentence in priority-bands.md that pointed at the CLI warning, since that behavior no longer exists.

Testing

I did not run uv run pytest because the CLI/test files are reverted to origin/main and the only remaining changes are Markdown docs, which have no tests to break. ./infra/pre-commit.py --all-files --fix passed (ruff, black, pyrefly, markdown, etc.).
• Branch: agent/20260414-fix-4747

Per review feedback, revert the CLI change to lib/iris/src/iris/cli/job.py
and its test; the priority-bands doc no longer mentions the warning.

Co-authored-by: William Held <Helw150@users.noreply.github.com>
@rjpower rjpower enabled auto-merge (squash) April 14, 2026 21:03
@rjpower rjpower merged commit f25986a into main Apr 14, 2026
37 of 38 checks passed
@rjpower rjpower deleted the agent/20260414-fix-4747 branch April 14, 2026 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent-generated Created by automation/agent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Document Intended Usage of the Priority Queues

2 participants