Skip to content

Upgrading slack#143

Merged
marcosfrenkel merged 4 commits intomasterfrom
upgrading_slack
Apr 20, 2026
Merged

Upgrading slack#143
marcosfrenkel merged 4 commits intomasterfrom
upgrading_slack

Conversation

@marcosfrenkel
Copy link
Copy Markdown
Collaborator

@marcosfrenkel marcosfrenkel commented Apr 17, 2026

Description

Replaces the single-game chsh_daily_report.py script with a full daily_report module that:

  • Probes hardware health (/health/) before running any games
  • Checks game availability (/games/availability) and skips disabled games gracefully
  • Runs CHSH and Quantum Fortune; notes SSM as pending
  • Posts a consolidated Block Kit Slack digest with per-game status, elapsed time, and hardware component lines
  • Arms a SIGALRM watchdog for the whole run (overall_timeout_s) and a per-game HTTP timeout (per_game_timeout_s), both configurable in [daily_report]

The CLI gains a daily-report subcommand group with run, status, schedule, and unschedule commands, replacing the standalone script entry point.

Motivation and Context

The old script only reported CHSH and gave no visibility into hardware or other games. Operators had no way to tell from the Slack digest whether a failure was a hardware issue, a game-specific problem, or a hung process. This PR consolidates all that into a single daily digest and adds the watchdog so a stuck API call can't silently block the cron job.

How have these changes been tested?

  • Ran pqnstack daily-report run against a local dev stack (API up, hardware mocked)
  • Verified Slack digest format for all-ok, hardware-failure, and game-failure states
  • Tested schedule, status, and unschedule CLI commands
  • Confirmed watchdog fires and posts Slack alert when overall_timeout_s is artificially small

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project.
  • My changes require changes to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests pass.

@marcosfrenkel marcosfrenkel merged commit 9ead008 into master Apr 20, 2026
6 checks passed
@marcosfrenkel marcosfrenkel deleted the upgrading_slack branch April 20, 2026 14:17
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.

1 participant