Skip to content

Add tenacity retries to secrets file creation#2933

Merged
adamtheturtle merged 5 commits intomainfrom
adamtheturtle/tenacity-retries
Feb 17, 2026
Merged

Add tenacity retries to secrets file creation#2933
adamtheturtle merged 5 commits intomainfrom
adamtheturtle/tenacity-retries

Conversation

@adamtheturtle
Copy link
Member

@adamtheturtle adamtheturtle commented Feb 17, 2026

This updates admin/create_secrets_files.py to use explicit tenacity retry policies for timeout-prone Selenium operations. Database and VuMark creation/detail retrieval now raise TimeoutException and are wrapped with retries that stop after three attempts with exponential backoff. The main loop now handles exhausted retries by resetting the driver and continuing to the next attempt, preserving the existing workflow while making retry behavior bounded and clearer. The tenacity call signatures were adjusted to keyword arguments to satisfy strict mypy checks in pre-push hooks.


Note

Low Risk
Low risk: changes are limited to an admin script and mainly adjust timeout handling by retrying and failing fast after bounded attempts.

Overview
Makes secrets-file generation more resilient to flaky Selenium timeouts by introducing a shared tenacity RETRY_ON_TIMEOUT policy (3 attempts with exponential backoff) around database/VuMark detail retrieval.

Refactors helper functions to always return details (no None) and lets TimeoutException propagate after retries; the main loop now catches exhausted timeouts, logs a clearer error, resets the web driver, and continues to the next file.

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

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@adamtheturtle adamtheturtle merged commit 7a4a6f1 into main Feb 17, 2026
121 checks passed
@adamtheturtle adamtheturtle deleted the adamtheturtle/tenacity-retries branch February 17, 2026 08:04
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