Merge pull request #552 from wansir/fix-api

update job rerun api

pkg/apis/operations/v1alpha2/register.go

@@ -49,11 +49,11 @@ func addWebService(c *restful.Container) error {
 
 	webservice.Route(webservice.POST("/namespaces/{namespace}/jobs/{job}").
 		To(operations.RerunJob).
-		Doc("Job rerun").
+		Doc("Rerun job whether the job is complete or not").
 		Deprecate().
 		Param(webservice.PathParameter("job", "job name")).
-		Param(webservice.PathParameter("namespace", "job's namespace")).
-		Param(webservice.QueryParameter("a", "action")).
+		Param(webservice.PathParameter("namespace", "the name of the namespace where the job runs in")).
+		Param(webservice.QueryParameter("action", "action must be \"rerun\"")).
 		Returns(http.StatusOK, ok, errors.Error{}))
 
 	c.Add(webservice)

pkg/apis/resources/v1alpha2/register.go

@@ -82,8 +82,8 @@ func addWebService(c *restful.Container) error {
 		Metadata(restfulspec.KeyOpenAPITags, []string{constants.NamespaceResourcesTag}).
 		Doc("Rerun job whether the job is complete or not").
 		Param(webservice.PathParameter("job", "job name")).
-		Param(webservice.PathParameter("namespace", "the namespace where the job runs in")).
-		Param(webservice.QueryParameter("a", "action")).
+		Param(webservice.PathParameter("namespace", "the name of the namespace where the job runs in")).
+		Param(webservice.QueryParameter("action", "action must be \"rerun\"")).
 		Returns(http.StatusOK, ok, errors.Error{}))
 
 	webservice.Route(webservice.GET("/{resources}").

pkg/apiserver/operations/job.go

@@ -33,7 +33,7 @@ func RerunJob(req *restful.Request, resp *restful.Response) {
 
 	job := req.PathParameter("job")
 	namespace := req.PathParameter("namespace")
-	action := req.QueryParameter("a")
+	action := req.QueryParameter("action")
 
 	switch action {
 	case "rerun":