This is an automated email from the ASF dual-hosted git repository.
adoroszlai pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ozone.git
The following commit(s) were added to refs/heads/master by this push:
new d66c0888fb HDDS-11848. Serialization bug in Recon listKeys API (#7524)
d66c0888fb is described below
commit d66c0888fbf576745a3ea960702d68a9d045ca02
Author: Siddhant Sangwan <[email protected]>
AuthorDate: Wed Dec 4 20:49:52 2024 +0530
HDDS-11848. Serialization bug in Recon listKeys API (#7524)
---
hadoop-ozone/dist/src/main/smoketest/recon/recon-api.robot | 12 ++++++++++++
.../ozone/recon/api/types/KeyEntityInfoProtoWrapper.java | 2 +-
2 files changed, 13 insertions(+), 1 deletion(-)
diff --git a/hadoop-ozone/dist/src/main/smoketest/recon/recon-api.robot
b/hadoop-ozone/dist/src/main/smoketest/recon/recon-api.robot
index 43860b75d9..883e93bdc5 100644
--- a/hadoop-ozone/dist/src/main/smoketest/recon/recon-api.robot
+++ b/hadoop-ozone/dist/src/main/smoketest/recon/recon-api.robot
@@ -28,6 +28,8 @@ ${API_ENDPOINT_URL} ${ENDPOINT_URL}/api/v1
${ADMIN_API_ENDPOINT_URL} ${API_ENDPOINT_URL}/containers
${UNHEALTHY_ENDPOINT_URL} ${API_ENDPOINT_URL}/containers/unhealthy
${NON_ADMIN_API_ENDPOINT_URL} ${API_ENDPOINT_URL}/clusterState
+${VOLUME} vol1
+${BUCKET} bucket1
*** Keywords ***
Check if Recon picks up container from OM
@@ -57,6 +59,15 @@ Check http return code
Should contain ${result} 200
END
+Check if the listKeys api responds OK
+ [Arguments] ${volume} ${bucket}
+ Run Keyword if '${SECURITY_ENABLED}' == 'true' Kinit as ozone admin
+ ${result} = Execute curl --negotiate -u : -LSs
${API_ENDPOINT_URL}/keys/listKeys?startPrefix=/${volume}/${bucket}&limit=1000
+ Should contain ${result} "OK"
+ Should contain ${result} "keys"
+ Should contain ${result} "${volume}"
+ Should contain ${result} "${bucket}"
+
*** Test Cases ***
Check if Recon picks up OM data
Execute ozone sh volume create recon
@@ -67,6 +78,7 @@ Check if Recon picks up OM data
Execute ozone sh bucket create recon/api --layout=LEGACY
Freon OCKG n=10 args=-s 1025 -v recon -b api
Wait Until Keyword Succeeds 90sec 10sec Check if Recon
picks up container from OM
+ Wait Until Keyword Succeeds 90sec 10sec Check if the
listKeys api responds OK recon api
Check if Recon picks up DN heartbeats
${result} = Execute curl --negotiate
-u : -LSs ${API_ENDPOINT_URL}/datanodes
diff --git
a/hadoop-ozone/recon/src/main/java/org/apache/hadoop/ozone/recon/api/types/KeyEntityInfoProtoWrapper.java
b/hadoop-ozone/recon/src/main/java/org/apache/hadoop/ozone/recon/api/types/KeyEntityInfoProtoWrapper.java
index b61ebf9963..890f8fbd3a 100644
---
a/hadoop-ozone/recon/src/main/java/org/apache/hadoop/ozone/recon/api/types/KeyEntityInfoProtoWrapper.java
+++
b/hadoop-ozone/recon/src/main/java/org/apache/hadoop/ozone/recon/api/types/KeyEntityInfoProtoWrapper.java
@@ -122,7 +122,7 @@ public final class KeyEntityInfoProtoWrapper {
}
@JsonProperty("isKey")
- public boolean isKey() {
+ public boolean getIsKey() {
return keyInfoProto.getIsFile();
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
