Skip to content

fix: parse WaitTimeout and use it as poll timeout in ExecuteAndWait#1730

Open
gourabsingha1 wants to merge 1 commit into
databricks:mainfrom
gourabsingha1:fix/execute-and-wait-timeout
Open

fix: parse WaitTimeout and use it as poll timeout in ExecuteAndWait#1730
gourabsingha1 wants to merge 1 commit into
databricks:mainfrom
gourabsingha1:fix/execute-and-wait-timeout

Conversation

@gourabsingha1

Copy link
Copy Markdown

Closes #1448. Parses request.WaitTimeout duration and uses it for the retries.Poll loop inside ExecuteAndWait, falling back to 20m if unspecified or invalid.

@github-actions

Copy link
Copy Markdown

Please ensure that the NEXT_CHANGELOG.md file is updated with any relevant changes.
If this is not necessary for your PR, please include the following in your PR description:
NO_CHANGELOG=true
and rerun the job.

@github-actions

Copy link
Copy Markdown

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-go

Inputs:

  • PR number: 1730
  • Commit SHA: 13f01893d355ef16fe12170e4a40725b06c91c29

Checks will be approved automatically on success.

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.

[ISSUE] ExecuteAndWait uses a hard-coded 20 minute timeout

1 participant