Remove manual cluster core count configuration #113

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

NessieCanCode merged 1 commit into main from remove-total-cluster-cores-field

Aug 7, 2025

Owner

NessieCanCode commented Aug 7, 2025

Summary

Drop "Total Cluster Cores" field from settings UI
Clarify rate schema and config to rely on cores parsed from slurm.conf
Update billing summary test to use slurm.conf data instead of manual core count

Testing

PYTHONPATH=src python test/unit/accounts_listing.test.py
PYTHONPATH=src python test/unit/auth_boundaries.test.py
PYTHONPATH=src python test/unit/billing_summary.test.py
PYTHONPATH=src python test/unit/invoice_retrieval.test.py
PYTHONPATH=src python test/unit/slurm_schema_dump.test.py
PYTHONPATH=src python test/unit/slurmdb_validation.test.py
node test/unit/calculator.test.js
node test/unit/date_boundaries.test.js

https://chatgpt.com/codex/tasks/task_e_6895268667bc8324883dd43a0ec6db2d


          Clarify rate schema excludes cluster core count

02c1d51

NessieCanCode added the codex label

— with

ChatGPT Codex Connector

NessieCanCode merged commit 55c0ab1 into main

0 of 4 checks passed

NessieCanCode deleted the remove-total-cluster-cores-field branch

August 7, 2025 22:32

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

Labels