Skip to content

Conversation

@markstory
Copy link
Member

Sales is trying to move a larger customer account and hitting timeouts.

Sales is trying to move a larger customer account and hitting timeouts.
@markstory markstory requested a review from a team as a code owner January 23, 2026 17:06
@markstory markstory requested a review from a team January 23, 2026 17:06
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jan 23, 2026
cursor[bot]

This comment was marked as outdated.

Copy link
Contributor

@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 2 potential issues.

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

DEFAULT_VALIDATION_TIMEOUT = timedelta(minutes=60)
CROSS_REGION_EXPORT_TIMEOUT = timedelta(minutes=60)
DEFAULT_VALIDATION_TIMEOUT = timedelta(minutes=80)
CROSS_REGION_EXPORT_TIMEOUT = timedelta(minutes=80)
Copy link
Contributor

Choose a reason for hiding this comment

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

Polling limit inconsistent with extended validation timeout

High Severity

The DEFAULT_VALIDATION_TIMEOUT was increased from 60 to 80 minutes, but MAX_VALIDATION_POLLS remains at 60. Since each poll runs with a 60-second countdown and polls are limited to MAX_VALIDATION_POLL_ATTEMPTS (61), the system can only poll for approximately 61 minutes. If CloudBuild takes between 61-80 minutes (which the extended timeout now allows), the relocation will fail prematurely due to exhausted poll attempts, defeating the purpose of extending the timeout.

Additional Locations (1)

Fix in Cursor Fix in Web

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants