Add missing timeouts to rest-api-spec shutdown APIs (#118921) (#119737)

pquentin · web-flow · commit 7881ea2d5c81 · 2025-01-08T21:56:46.000+11:00
diff --git a/docs/changelog/118921.yaml b/docs/changelog/118921.yaml
@@ -0,0 +1,5 @@
+pr: 118921
+summary: Add missing timeouts to rest-api-spec shutdown APIs
+area: Infra/Node Lifecycle
+type: bug
+issues: []
diff --git a/rest-api-spec/src/main/resources/rest-api-spec/api/shutdown.delete_node.json b/rest-api-spec/src/main/resources/rest-api-spec/api/shutdown.delete_node.json
@@ -26,6 +26,15 @@
         }
       ]
     },
-    "params":{}
+    "params":{
+      "master_timeout":{
+        "type":"time",
+        "description":"Explicit operation timeout for connection to master node"
+      },
+      "timeout":{
+        "type":"time",
+        "description":"Explicit operation timeout"
+      }
+    }
   }
 }
diff --git a/rest-api-spec/src/main/resources/rest-api-spec/api/shutdown.put_node.json b/rest-api-spec/src/main/resources/rest-api-spec/api/shutdown.put_node.json
@@ -26,7 +26,16 @@
         }
       ]
     },
-    "params":{},
+    "params":{
+      "master_timeout":{
+        "type":"time",
+        "description":"Explicit operation timeout for connection to master node"
+      },
+      "timeout":{
+        "type":"time",
+        "description":"Explicit operation timeout"
+      }
+    },
     "body":{
       "description":"The shutdown type definition to register",
       "required": true

Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,15 @@`
`26`	`26`	`}`
`27`	`27`	`]`
`28`	`28`	`},`
`29`		`- "params":{}`
	`29`	`+ "params":{`
	`30`	`+ "master_timeout":{`
	`31`	`+ "type":"time",`
	`32`	`+ "description":"Explicit operation timeout for connection to master node"`
	`33`	`+ },`
	`34`	`+ "timeout":{`
	`35`	`+ "type":"time",`
	`36`	`+ "description":"Explicit operation timeout"`
	`37`	`+ }`
	`38`	`+ }`
`30`	`39`	`}`
`31`	`40`	`}`