mrva/mrvacommander
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

add status fix code; to be tested

Browse Source
This commit is contained in:
Michael Hohn
2025-08-28 15:11:13 -07:00
committed by =Michael Hohn
parent fb5adf1b5f
commit d2c7b98d1c
2 changed files with 62 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
pkg/common/types.go
View File

@@ -41,3 +41,8 @@ type JobSpec struct {
SessionID int SessionID int
NameWithOwner NameWithOwner
} }
type StatusSummary struct {
Overall Status
Counts map[Status]int
}

66
pkg/state/state_postgres.go
View File

@@ -279,20 +279,68 @@ func (s *PGState) SetStatus(js common.JobSpec, status common.Status) {
} }
} }
func (s *PGState) GetStatus(js common.JobSpec) (common.Status, error) { func (s *PGState) GetSessionStatus(sessionID int) (common.StatusSummary, error) {
ctx := context.Background() ctx := context.Background()
var status int rows, err := s.pool.Query(ctx, `
err := s.pool.QueryRow(ctx, ` SELECT status
SELECT status FROM job_status
FROM job_status WHERE session_id = $1
WHERE session_id = $1 AND owner = $2 AND repo = $3 `, sessionID)
`, js.SessionID, js.Owner, js.Repo).Scan(&status)
if err != nil { if err != nil {
return 0, err // caller must interpret not-found vs. real error return common.StatusSummary{}, err
}
defer rows.Close()
counts := map[common.Status]int{
common.StatusPending: 0,
common.StatusInProgress: 0,
common.StatusSucceeded: 0,
common.StatusFailed: 0,
common.StatusCanceled: 0,
common.StatusTimedOut: 0,
}
total := 0
for rows.Next() {
var st int
if err := rows.Scan(&st); err != nil {
return common.StatusSummary{}, err
}
counts[common.Status(st)]++
total++
} }
return common.Status(status), nil // apply deterministic rules
var overall common.Status
switch {
case counts[common.StatusSucceeded] == total:
overall = common.StatusSucceeded
case counts[common.StatusFailed] == total:
overall = common.StatusFailed
case counts[common.StatusCanceled] == total:
overall = common.StatusCanceled
case counts[common.StatusTimedOut] == total:
overall = common.StatusFailed
case counts[common.StatusInProgress] > 0:
overall = common.StatusPending
case counts[common.StatusPending] > 0 && counts[common.StatusInProgress] == 0:
overall = common.StatusPending
case counts[common.StatusPending] == 0 && counts[common.StatusInProgress] == 0:
overall = common.StatusSucceeded // covers mixed complete
default:
overall = common.StatusPending
}
return common.StatusSummary{Overall: overall, Counts: counts}, nil
}
func (s *PGState) GetStatus(js common.JobSpec) (common.Status, error) {
summary, err := s.GetSessionStatus(js.SessionID)
if err != nil {
return 0, err
}
return summary.Overall, nil
} }
// GetRepoId returns a stable unique ID for a given (owner, repo). // GetRepoId returns a stable unique ID for a given (owner, repo).