Skip to content

Consolidate timeout handling and add step logging#2999

Merged
adamtheturtle merged 1 commit intomainfrom
adamtheturtle/consolidate-timeout-logging
Feb 22, 2026
Merged

Consolidate timeout handling and add step logging#2999
adamtheturtle merged 1 commit intomainfrom
adamtheturtle/consolidate-timeout-logging

Conversation

@adamtheturtle
Copy link
Member

@adamtheturtle adamtheturtle commented Feb 22, 2026

Summary

  • Merged three separate try/except TimeoutException blocks into a single catch block
  • Added logging for each step of the database setup process (database details, VuMark database details, VuMark target creation)
  • Simplified error handling while improving visibility into which step times out

Test plan

  • Existing tests should pass
  • Manual testing of the script should show the new log messages during database setup

Note

Low Risk
Small refactor limited to a one-off admin script; behavior changes are mainly log output and slightly less specific timeout messaging.

Overview
Consolidates timeout handling in admin/create_secrets_files.py by replacing three step-specific TimeoutException handlers with a single catch that resets the Selenium driver and retries the loop.

Adds stdout step logging before each major setup action (database details, VuMark database details, VuMark target creation) and simplifies the timeout error message to Timed out during database setup.

Written by Cursor Bugbot for commit d2998df. This will update automatically on new commits. Configure here.

Merge three separate try/except blocks into one catch for TimeoutException, and add logging for each step of the database setup process for better visibility into which step times out.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@adamtheturtle adamtheturtle merged commit 3d30fcc into main Feb 22, 2026
11 checks passed
@adamtheturtle adamtheturtle deleted the adamtheturtle/consolidate-timeout-logging branch February 22, 2026 22:46
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