Skip to content

Conversation

andrewnester
Copy link
Contributor

Changes

It can happen that after creating app the compute state returned from API is Starting already. Previously we incorrectly handled this and did not wait for the app to start first

Why

Fixes nightly tests

Tests

Copy link
Contributor

@denik denik left a comment

Choose a reason for hiding this comment

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

Fixes nightly tests

Could you specify which ones?

Also, can we add an acceptance tests that covers this?

@andrewnester
Copy link
Contributor Author

acceptance/bundle/run/app-with-job is the one that failed, it's an acceptance test already (CloudSlow one)

@eng-dev-ecosystem-bot
Copy link
Collaborator

eng-dev-ecosystem-bot commented Aug 25, 2025

Run: 17578777585

Env ✅​pass 🔄​flaky 🙈​skip
✅​ aws linux 308 523
✅​ aws windows 309 522
🔄​ aws-ucws linux 417 3 421
✅​ aws-ucws windows 421 420
✅​ azure linux 308 522
🔄​ azure windows 306 3 521
✅​ azure-ucws linux 420 420
✅​ azure-ucws windows 421 419
✅​ gcp linux 307 524
✅​ gcp windows 308 523
Test Name aws-ucws linux azure windows
TestAccept 🔄​flaky 🔄​flaky
TestAccept/bundle/templates/default-python/integration_classic ✅​pass 🔄​flaky
TestAccept/bundle/templates/default-python/integration_classic/DATABRICKS_CLI_DEPLOYMENT=direct-exp/UV_PYTHON=3.12 ✅​pass 🔄​flaky
TestAccept/selftest/record_cloud/workspace-file-io 🔄​flaky ✅​pass
TestAccept/selftest/record_cloud/workspace-file-io/DATABRICKS_CLI_DEPLOYMENT=direct-exp 🔄​flaky ✅​pass

Copy link
Contributor

@pietern pietern left a comment

Choose a reason for hiding this comment

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

@andrewnester This still seems relevant. Is that right?

@andrewnester
Copy link
Contributor Author

@pietern yes, it is still relevant

@andrewnester andrewnester added this pull request to the merge queue Sep 9, 2025
Merged via the queue into main with commit ec7df41 Sep 9, 2025
13 checks passed
@andrewnester andrewnester deleted the acc/app-run-job-direct branch September 9, 2025 10:35
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.

4 participants