Skip to content

Catch exceptions when log not found for retried PRs and failed jobs - #59

Merged
ahus1 merged 1 commit into
keycloak:mainfrom
mabartos:KC-58
Jul 10, 2026
Merged

Catch exceptions when log not found for retried PRs and failed jobs#59
ahus1 merged 1 commit into
keycloak:mainfrom
mabartos:KC-58

Conversation

@mabartos

Copy link
Copy Markdown
Member
  • Closes Cannot update data via CI #58
  • Fix update-data CI failing when gh run view --log-failed returns a non-zero exit code (missing/expired job logs)
  • Catch exception and collect warnings instead of aborting the entire update
  • No data loss - only the raw log output is skipped

Reproducer and fixed locally:

[INFO] --- exec:3.1.0:java (default-cli) @ dashboard ---
Fetching areas: .
Fetching updated issues since 2026-07-10: .
Fetching issues: .
Fetching updated prs since 2026-07-10: .
Fetching bugs with PRs: .
Updating pull request wait times since 2026-07-10: .
Fetching failed-jobs: 
Deleting expired failed-jobs: 
Fetching retried-prs: ...
[WARN] Failed to download log for run 29024314313: Returned with 1, see ghcli-err.log for details
Deleting expired retried-prs: 
Fetching workflow-status: ...
Loading cve stats: .......
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------

@ahus1 @stianst Can you check it? Thanks!

Closes keycloak#58

Signed-off-by: Martin Bartoš <mabartos@redhat.com>
@mabartos
mabartos requested review from ahus1 and stianst July 10, 2026 07:24
@ahus1
ahus1 merged commit 3644adc into keycloak:main Jul 10, 2026
2 checks passed
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.

Cannot update data via CI

2 participants