Skip to content

Reduce integration test cluster capacity usage - #1844

Merged
hectorcast-db merged 1 commit into
mainfrom
integration-test-capacity-cleanup
Sep 16, 2026
Merged

hectorcast-db merged 1 commit into
mainfrom
integration-test-capacity-cleanup

Conversation

@hectorcast-db

Copy link
Copy Markdown
Contributor

Summary

Reduces integration-test cluster capacity by replacing SDK waiter coverage with deterministic HTTP fixtures and removing redundant live tests.

Why

The cluster timeout test created a real cluster solely to exercise client-side polling and cleanup, while the libraries test used a live cluster to validate server-side package installation. The direct command execution test also duplicated the higher-level command integration coverage. These tests consumed cluster capacity without adding corresponding live SDK coverage value.

What changed

Interface changes

None.

Behavioral changes

None. SDK runtime behavior is unchanged.

Internal changes

  • Converted the cluster create timeout test to HTTP fixtures covering create, repeated PENDING polls, timeout, delete, and terminated cleanup.
  • Removed TestAccCommandsDirectUsage; TestAccCommands retains live command coverage.
  • Replaced TestAccLibraries with HTTP fixture coverage for install polling from PENDING to INSTALLED, followed by uninstall.
  • Removed the now-unused live-cluster helper and documented the test-only change in NEXT_CHANGELOG.md.

How is this tested?

  • make fmt
  • go test ./internal -run '^(TestClustersCreateTimesOutAndCleansUp|TestLibrariesUpdateAndWait)$' -count=1

Signed-off-by: Hector Castejon Diaz <hector.castejon@databricks.com>
@hectorcast-db
hectorcast-db deployed to test-trigger-is September 15, 2026 08:49 — with GitHub Actions Active
@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: 1844
  • Commit SHA: 33aa7d13019ee7b1763370136d0098752cfd3584

Checks will be approved automatically on success.

@hectorcast-db hectorcast-db added the CI:run-diffviewer Run DiffViewer CI label Sep 15, 2026
@hectorcast-db
hectorcast-db deployed to test-trigger-is September 15, 2026 08:51 — with GitHub Actions Active
@hectorcast-db
hectorcast-db deployed to test-trigger-is September 15, 2026 08:51 — with GitHub Actions Active
@hectorcast-db
hectorcast-db added this pull request to the merge queue Sep 16, 2026
Merged via the queue into main with commit 3166530 Sep 16, 2026
13 checks passed
@hectorcast-db
hectorcast-db deleted the integration-test-capacity-cleanup branch September 16, 2026 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI:run-diffviewer Run DiffViewer CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants