Skip to content

acc: Make CloudSlow equivalent of Cloud#3224

Closed
denik wants to merge 5 commits intomainfrom
denik/disable-cloud-slow
Closed

acc: Make CloudSlow equivalent of Cloud#3224
denik wants to merge 5 commits intomainfrom
denik/disable-cloud-slow

Conversation

@denik
Copy link
Copy Markdown
Contributor

@denik denik commented Jul 10, 2025

Changes

CloudSlow is now alias for Cloud.

Why

Not running this tests on PRs makes it easy to miss test failures (and real bugs).

@denik denik temporarily deployed to test-trigger-is July 10, 2025 10:00 — with GitHub Actions Inactive
@eng-dev-ecosystem-bot
Copy link
Copy Markdown
Collaborator

eng-dev-ecosystem-bot commented Jul 10, 2025 •
edited
Loading

Run: 16195776424

Env ✅‌pass ❌‌FAIL 🔄‌flaky 🙈‌skip 🪲‌BUG
❌‌ aws linux 311 3 392 13
❌‌ aws windows 312 3 391 13
🪲‌ aws-ucws linux 429 289 13
🪲‌ aws-ucws windows 430 288 13
🪲‌ azure linux 314 391 13
🪲‌ azure windows 315 390 13
🪲‌ azure-ucws linux 431 286 13
🪲‌ azure-ucws windows 431 1 285 13
❌‌ gcp linux 308 3 395 13
❌‌ gcp windows 309 3 394 13
17 failing tests:
Test Name aws linux aws windows aws-ucws linux aws-ucws windows azure linux azure windows azure-ucws linux azure-ucws windows gcp linux gcp windows
TestAccept 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG
TestAccept/bundle/integration_whl/base 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG
TestAccept/bundle/integration_whl/base/DATABRICKS_CLI_DEPLOYMENT=direct-exp 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG
TestAccept/bundle/integration_whl/base/DATABRICKS_CLI_DEPLOYMENT=terraform 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG
TestAccept/bundle/integration_whl/custom_params 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG
TestAccept/bundle/integration_whl/custom_params/DATABRICKS_CLI_DEPLOYMENT=direct-exp 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG
TestAccept/bundle/integration_whl/custom_params/DATABRICKS_CLI_DEPLOYMENT=terraform 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG
TestAccept/bundle/integration_whl/interactive_cluster 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG
TestAccept/bundle/integration_whl/interactive_cluster/DATABRICKS_CLI_DEPLOYMENT=direct-exp 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG
TestAccept/bundle/integration_whl/interactive_cluster/DATABRICKS_CLI_DEPLOYMENT=terraform 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG
TestAccept/bundle/integration_whl/interactive_cluster_dynamic_version ❌‌FAIL ❌‌FAIL ✅‌pass ✅‌pass ✅‌pass ✅‌pass ✅‌pass ✅‌pass ❌‌FAIL ❌‌FAIL
TestAccept/bundle/integration_whl/interactive_cluster_dynamic_version/DATABRICKS_CLI_DEPLOYMENT=terraform/DATA_SECURITY_MODE=SINGLE_USER ❌‌FAIL ❌‌FAIL ✅‌pass ✅‌pass ✅‌pass ✅‌pass ✅‌pass ✅‌pass ❌‌FAIL ❌‌FAIL
TestAccept/bundle/integration_whl/interactive_cluster_dynamic_version/DATABRICKS_CLI_DEPLOYMENT=terraform/DATA_SECURITY_MODE=USER_ISOLATION ❌‌FAIL ❌‌FAIL ✅‌pass ✅‌pass ✅‌pass ✅‌pass ✅‌pass ✅‌pass ❌‌FAIL ❌‌FAIL
TestAccept/bundle/integration_whl/interactive_single_user 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG
TestAccept/bundle/integration_whl/interactive_single_user/DATABRICKS_CLI_DEPLOYMENT=direct-exp 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG
TestAccept/bundle/integration_whl/interactive_single_user/DATABRICKS_CLI_DEPLOYMENT=terraform 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG 🪲‌BUG
TestAccept/bundle/resources/clusters/run/spark_python_task ✅‌pass ✅‌pass ✅‌pass ✅‌pass ✅‌pass ✅‌pass ✅‌pass 🔄‌flaky ✅‌pass ✅‌pass

@denik denik temporarily deployed to test-trigger-is July 10, 2025 10:52 — with GitHub Actions Inactive
@denik denik temporarily deployed to test-trigger-is July 10, 2025 11:29 — with GitHub Actions Inactive
@denik denik temporarily deployed to test-trigger-is July 10, 2025 12:19 — with GitHub Actions Inactive
@denik denik temporarily deployed to test-trigger-is July 10, 2025 12:57 — with GitHub Actions Inactive
@denik denik force-pushed the denik/disable-cloud-slow branch from 3303b54 to 584c8ca Compare July 10, 2025 13:25
@denik denik temporarily deployed to test-trigger-is July 10, 2025 13:25 — with GitHub Actions Inactive
@denik denik temporarily deployed to test-trigger-is July 10, 2025 14:08 — with GitHub Actions Inactive
@denik denik mentioned this pull request Jul 10, 2025
Merged
@denik
Copy link
Copy Markdown
Contributor Author

denik commented Jul 10, 2025

Not sure about landing this.

  • CloudSlow tests take 1.5h, regular 10-20min, big difference.
  • We don't need to mess with options, we can just disable -short flag in Makefile.

This PR has a bunch of fixes to test.toml files, I've extracted them here #3231 Please stamp that one.

@denik denik closed this Jul 10, 2025
denik added a commit that referenced this pull request Jul 10, 2025
@denik
acc: Fix CloudSlow tests; auto-enable Cloud on CloudSlow (#3231)
a7815f1 
## Changes
Extract from #3224 that
- Automatically enables Cloud if CloudSlow is enabled. Note, this only
affects out.test.toml but behaviour is the same.
- Disables broken tests.

## Tests
Commit and run with non-short tests enabled:
8a9e80b
https://go/deco-tests/16198082279
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pietern pietern pietern approved these changes

@andrewnester andrewnester andrewnester approved these changes

@shreyas-goenka shreyas-goenka Awaiting requested review from shreyas-goenka

@anton-107 anton-107 Awaiting requested review from anton-107

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@denik @eng-dev-ecosystem-bot @pietern @andrewnester