return 400 if request validation fails

Signed-off-by: Madhu Rajanna <mrajanna@redhat.com>

glusterd2/commands/snapshot/snapshot-activate.go

@@ -167,7 +167,7 @@ func snapshotActivateHandler(w http.ResponseWriter, r *http.Request) {
 	var req api.SnapActivateReq
 	vol = &snapinfo.SnapVolinfo
 	if err := restutils.UnmarshalRequest(r, &req); err != nil {
-		restutils.SendHTTPError(ctx, w, http.StatusUnprocessableEntity, err)
+		restutils.SendHTTPError(ctx, w, http.StatusBadRequest, err)
 		return
 	}
 

glusterd2/commands/snapshot/snapshot-clone.go

@@ -286,7 +286,7 @@ func snapshotCloneHandler(w http.ResponseWriter, r *http.Request) {
 
 	snapname := mux.Vars(r)["snapname"]
 	if snapname == "" {
-		restutils.SendHTTPError(ctx, w, http.StatusUnprocessableEntity, errors.New("Snapshot name should not be empty"))
+		restutils.SendHTTPError(ctx, w, http.StatusBadRequest, errors.New("Snapshot name should not be empty"))
 		return
 	}
 
@@ -296,7 +296,7 @@ func snapshotCloneHandler(w http.ResponseWriter, r *http.Request) {
 	}
 
 	if !volume.IsValidName(req.CloneName) {
-		restutils.SendHTTPError(ctx, w, http.StatusUnprocessableEntity, gderrors.ErrInvalidVolName)
+		restutils.SendHTTPError(ctx, w, http.StatusBadRequest, gderrors.ErrInvalidVolName)
 		return
 	}
 

glusterd2/commands/snapshot/snapshot-create.go

@@ -705,7 +705,7 @@ func snapshotCreateHandler(w http.ResponseWriter, r *http.Request) {
 	}
 
 	if !volume.IsValidName(req.SnapName) {
-		restutils.SendHTTPError(ctx, w, http.StatusUnprocessableEntity, gderrors.ErrInvalidSnapName)
+		restutils.SendHTTPError(ctx, w, http.StatusBadRequest, gderrors.ErrInvalidSnapName)
 		return
 	}