Skip to content

Commit ccca489

Browse files
committed
Skip shard routing tests on OpenSearch < 2.2.0 with security
OpenSearch < 2.2.0 with the security plugin throws java.io.OptionalDataException on shard-routed requests due to non-thread-safe HashSet/HashMap in User serialization. Fixed in 2.2.0 by opensearch-project/security#1970 (50a94b47). Widen the existing 2.1.0-only skip to cover all versions below 2.2.0. Signed-off-by: Sean Chittenden <sean.chittenden@crowdstrike.com>
1 parent 83bc582 commit ccca489

1 file changed

Lines changed: 5 additions & 4 deletions

File tree

opensearchtransport/shard_routing_integration_test.go

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -232,11 +232,12 @@ func TestShardExactRouting_FullPipeline_Integration(t *testing.T) {
232232
testutil.WaitForCluster(t)
233233
testutil.SkipIfSingleNode(t, 2) // 1 replica requires at least 2 nodes for green health
234234

235-
// OpenSearch 2.1.0 with the security plugin returns HTTP 500 on search
236-
// requests routed to specific shard-hosting nodes. The insecure 2.1.0
237-
// compat job passes; this is a server-side security plugin bug.
235+
// OpenSearch < 2.2.0 with the security plugin returns HTTP 500 on
236+
// shard-routed requests due to non-thread-safe User serialization
237+
// (java.io.OptionalDataException). Fixed in 2.2.0 by
238+
// opensearch-project/security#1970 (50a94b47).
238239
if testutil.IsSecure(t) {
239-
testutil.SkipIfVersion(t, "=", "2.1.0", "shard-exact pipeline (security plugin HTTP 500)")
240+
testutil.SkipIfVersion(t, "<", "2.2.0", "shard-exact pipeline (security plugin OptionalDataException)")
240241
}
241242

242243
u := testutil.GetTestURL(t)

0 commit comments

Comments
 (0)