Skip to content

Commit 2d4c849

Browse files
authored
Merge branch 'main' into agentic-search-feature-flag
2 parents e6464ed + 38ea2fc commit 2d4c849

File tree

17 files changed

+1083
-46
lines changed

17 files changed

+1083
-46
lines changed

.github/workflows/CI-workflow.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ jobs:
4242
needs: [Get-Require-Approval, Get-CI-Image-Tag, spotless]
4343
strategy:
4444
matrix:
45-
java: [21, 23]
45+
java: [21, 24]
4646

4747
name: Build and Test MLCommons Plugin on linux
4848
if: github.repository == 'opensearch-project/ml-commons'
@@ -107,7 +107,7 @@ jobs:
107107
needs: [Get-Require-Approval, Build-ml-linux, spotless]
108108
strategy:
109109
matrix:
110-
java: [21, 23]
110+
java: [21, 24]
111111

112112
name: Test MLCommons Plugin on linux docker
113113
if: github.repository == 'opensearch-project/ml-commons'
@@ -203,7 +203,7 @@ jobs:
203203
Build-ml-windows:
204204
strategy:
205205
matrix:
206-
java: [21, 23]
206+
java: [21, 24]
207207
name: Build and Test MLCommons Plugin on Windows
208208
if: github.repository == 'opensearch-project/ml-commons'
209209
needs: [Get-Require-Approval, spotless]

build.gradle

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,9 +60,9 @@ buildscript {
6060
}
6161

6262
plugins {
63-
id 'com.netflix.nebula.ospackage' version "11.5.0"
63+
id 'com.netflix.nebula.ospackage' version "12.0.0"
6464
id 'java'
65-
id "io.freefair.lombok" version "8.4"
65+
id "io.freefair.lombok" version "8.14"
6666
id 'jacoco'
6767
}
6868

common/build.gradle

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ dependencies {
4747
}
4848

4949
lombok {
50-
version = "1.18.30"
50+
version = "1.18.38"
5151
}
5252

5353
jacocoTestReport {

common/src/test/java/org/opensearch/ml/common/MLModelGroupTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ public void toXContent() throws IOException {
6363
.assertEquals(
6464
"{\"name\":\"test\",\"latest_version\":1,\"description\":\"this is test group\","
6565
+ "\"backend_roles\":[\"role1\",\"role2\"],"
66-
+ "\"owner\":{\"name\":\"\",\"backend_roles\":[],\"roles\":[],\"custom_attribute_names\":[],\"user_requested_tenant\":null},"
66+
+ "\"owner\":{\"name\":\"\",\"backend_roles\":[],\"roles\":[],\"custom_attribute_names\":[],\"user_requested_tenant\":null,\"user_requested_tenant_access\":null},"
6767
+ "\"access\":\"PUBLIC\"}",
6868
content
6969
);
@@ -166,7 +166,7 @@ public void toXContent_WithTenantId() throws IOException {
166166
.assertEquals(
167167
"{\"name\":\"test\",\"latest_version\":1,\"description\":\"this is test group\","
168168
+ "\"backend_roles\":[\"role1\",\"role2\"],"
169-
+ "\"owner\":{\"name\":\"\",\"backend_roles\":[],\"roles\":[],\"custom_attribute_names\":[],\"user_requested_tenant\":null},"
169+
+ "\"owner\":{\"name\":\"\",\"backend_roles\":[],\"roles\":[],\"custom_attribute_names\":[],\"user_requested_tenant\":null,\"user_requested_tenant_access\":null},"
170170
+ "\"access\":\"PUBLIC\",\"tenant_id\":\"test_tenant\"}",
171171
content
172172
);
@@ -176,7 +176,7 @@ public void toXContent_WithTenantId() throws IOException {
176176
public void parse_WithTenantId() throws IOException {
177177
String jsonStr = "{\"name\":\"test\",\"latest_version\":1,\"description\":\"this is test group\","
178178
+ "\"backend_roles\":[\"role1\",\"role2\"],"
179-
+ "\"owner\":{\"name\":\"\",\"backend_roles\":[],\"roles\":[],\"custom_attribute_names\":[],\"user_requested_tenant\":null},"
179+
+ "\"owner\":{\"name\":\"\",\"backend_roles\":[],\"roles\":[],\"custom_attribute_names\":[],\"user_requested_tenant\":null,\"user_requested_tenant_access\":null},"
180180
+ "\"access\":\"PUBLIC\",\"tenant_id\":\"test_tenant\"}";
181181

182182
XContentParser parser = XContentType.JSON

common/src/test/java/org/opensearch/ml/common/transport/model/MLModelGetResponseTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ public void toXContentTest() throws IOException {
6767
+ "\"algorithm\":\"KMEANS\","
6868
+ "\"model_version\":\"1.0.0\","
6969
+ "\"model_content\":\"content\","
70-
+ "\"user\":{\"name\":\"\",\"backend_roles\":[],\"roles\":[],\"custom_attribute_names\":[],\"user_requested_tenant\":null},\"model_state\":\"TRAINED\"}",
70+
+ "\"user\":{\"name\":\"\",\"backend_roles\":[],\"roles\":[],\"custom_attribute_names\":[],\"user_requested_tenant\":null,\"user_requested_tenant_access\":null},\"model_state\":\"TRAINED\"}",
7171
jsonStr
7272
);
7373
}

common/src/test/java/org/opensearch/ml/common/transport/task/MLTaskGetResponseTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ public void toXContentTest() throws IOException {
8686
+ "\"create_time\":123,"
8787
+ "\"last_update_time\":123,"
8888
+ "\"error\":\"error\","
89-
+ "\"user\":{\"name\":\"\",\"backend_roles\":[],\"roles\":[],\"custom_attribute_names\":[],\"user_requested_tenant\":null},"
89+
+ "\"user\":{\"name\":\"\",\"backend_roles\":[],\"roles\":[],\"custom_attribute_names\":[],\"user_requested_tenant\":null,\"user_requested_tenant_access\":null},"
9090
+ "\"is_async\":true}",
9191
jsonStr
9292
);

0 commit comments

Comments
 (0)