fournos-ui: add rhaiis submit form, CPT pipelines, and job detail enhancements - #113
fournos-ui: add rhaiis submit form, CPT pipelines, and job detail enhancements#113Harshith-umesh wants to merge 2 commits into
Conversation
Secure the dashboard with an OAuth proxy sidecar that authenticates users via OpenShift's built-in OAuth server. Add cert-manager integration for automatic Let's Encrypt certificate issuance. - Add kustomize overlay patches for OAuth proxy sidecar, service TLS, and ServiceAccount OAuth redirect annotation - Add Route, Certificate CR, and ClusterIssuer manifests - Update README with full deployment instructions and troubleshooting - Gitignore deployment-specific files (secrets, cluster config) - Remove unused params.env.example Co-authored-by: Cursor <cursoragent@cursor.com>
…etail enhancements - Add dynamic rhaiis project form with accelerator/engine/cluster/model/workload selection, fetched from Forge's GitHub config - Add CPT (Continuous Performance Testing) pipeline support with matrix UI for model x workload selection and multi-job submission - Support per-model config overrides and __tp (tensor-parallel-size) in CPT definitions for GPU-aware Kueue admission - Support /suffix convention for running same model preset with different TP configs - Parse tensor-parallel-size from Forge's models.yaml (vllm_args/sglang_args) to set correct hardware.gpuCount on FournosJobs - Add compare version and slack notification target fields to CPT form - Add engine version auto-population based on accelerator+engine combination - Add validation for invalid accelerator-engine combinations - Switch theme from dark to light, improve section headers and layout - Display detailed job failure info: pod exit codes, termination reasons, auto-load logs for failed pods - Enforce cancel-only for active jobs on history page (no delete for running jobs) - Add "View Live Jobs" button after CPT submission - Update dashboard route hostname and certificate for shorter URL Co-authored-by: Cursor <cursoragent@cursor.com>
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
openshift-ci
Bot
added
the
needs-rebase
|
PR needs rebase. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
1 participant
Summary
RHAIIS Submit Form: Dynamic project-specific form with accelerator/engine/cluster/model/workload selection, fetched from Forge's GitHub config. Includes engine version auto-population, validation for invalid combos, run settings with tooltips, compare version and slack notification fields.
CPT Pipelines: New Continuous Performance Testing pipeline support — define model × workload matrices in a
cpt.yamlfile, select/deselect models and workloads in a matrix UI, and submit multiple FournosJobs in one click. Supports per-model config overrides,__tpfor GPU-aware Kueue admission, and/suffixconvention for running the same model preset with different TP sizes.GPU-Aware Scheduling: Parse
tensor-parallel-sizefrom Forge'smodels.yamlto set correcthardware.gpuCounton FournosJobs, enabling Kueue to properly gate admission based on actual GPU needs.Job Detail Enhancements: Display pod exit codes, termination reasons, and status messages for failed jobs. Auto-load logs for the first failed pod.
UI/UX Improvements: Light theme, improved layout, enforce cancel-only for active jobs on history page (no delete for running jobs), "View Live Jobs" button after CPT submission.
Infrastructure: Updated dashboard route hostname and TLS certificate config.
Test plan
__tpvalues/suffixmodel variants generate separate jobs with correct presetsMade with Cursor