Skip to content

[Transform] Stop transforms at the end of tests #139783

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
prwhelan merged 8 commits into from
Mar 3, 2026
Merged

[Transform] Stop transforms at the end of tests #139783

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
a00d54f
[Transform] Stop transforms at the end of tests
prwhelan Dec 18, 2025
9d9545a
Merge branch 'main' into fix/122980
prwhelan Dec 19, 2025
36b58fd
Merge branch 'main' into fix/122980
prwhelan Jan 12, 2026
588ec6c
Merge branch 'main' into fix/122980
prwhelan Mar 2, 2026
e4d85c9
also delete the transforms
prwhelan Mar 2, 2026
fcd02fd
Merge branch 'main' into fix/122980
prwhelan Mar 2, 2026
7f64849
Merge branch 'main' into fix/122980
prwhelan Mar 2, 2026
590318e
Merge branch 'main' into fix/122980
prwhelan Mar 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions ...rc/javaRestTest/java/org/elasticsearch/xpack/transform/integration/TransformUpdateIT.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -364,6 +364,11 @@ private void updateTransferRightsTester(boolean useSecondaryAuthHeaders) throws
Map<?, ?> transformStatsAsMap = getTransformStateAndStats(transformIdCloned);
assertThat(XContentMapValues.extractValue("stats.documents_indexed", transformStatsAsMap), equalTo(27));
}, 15, TimeUnit.SECONDS);

stopTransform(transformId, BASIC_AUTH_VALUE_TRANSFORM_ADMIN_1, true, false);
stopTransform(transformIdCloned, BASIC_AUTH_VALUE_TRANSFORM_ADMIN_1, true, false);
Copy link
Copy Markdown
Contributor

@darius-vil darius-vil Mar 2, 2026

Choose a reason for hiding this comment

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

No need to deleteTransform?

Copy link
Copy Markdown
Member Author

@prwhelan prwhelan Mar 2, 2026

Choose a reason for hiding this comment

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

I don't think so? But let's add it just to be safe, it'll keep a consistent pattern across the testing frameworks

deleteTransform(transformId);
deleteTransform(transformIdCloned);
}

private void deleteUser(String user) throws IOException {
Expand Down