Merge pull request #1529 from yue9944882/generic-api-handling-status-code

Passing actual response status code in generic api's error handler

Author: k8s-ci-robot

util/src/main/java/io/kubernetes/client/util/generic/KubernetesApiResponse.java

@@ -59,8 +59,8 @@ public boolean isSuccess() {
    */
   public KubernetesApiResponse<DataType> throwsApiException() throws ApiException {
     return onFailure(
-        errorStatus -> {
-          throw new ApiException(errorStatus.toString());
+        (code, errorStatus) -> {
+          throw new ApiException(code, errorStatus.toString());
         });
   }
 
@@ -74,12 +74,12 @@ public KubernetesApiResponse<DataType> throwsApiException() throws ApiException
   public KubernetesApiResponse<DataType> onFailure(ErrorStatusHandler errorStatusHandler)
       throws ApiException {
     if (!isSuccess()) {
-      errorStatusHandler.handle(this.status);
+      errorStatusHandler.handle(this.getHttpStatusCode(), this.status);
     }
     return this;
   }
 
   public interface ErrorStatusHandler {
-    void handle(V1Status errorStatus) throws ApiException;
+    void handle(int code, V1Status errorStatus) throws ApiException;
   }
 }

util/src/test/java/io/kubernetes/client/util/generic/KubernetesApiResponseTest.java

@@ -55,7 +55,7 @@ public void testErrorStatusHandler() throws ApiException {
         podClient
             .delete("default", "foo")
             .onFailure(
-                errStatus -> {
+                (code, errStatus) -> {
                   catched.set(true);
                 })
             .getObject());