elasticsearch.BadRequestError: unrecognized parameter `node_id` on tasks.list

[margaretpearce]

In 8.15.1, the list function on TaskClient accepts a parameter node_id according to the docs:

node_id (Sequence[str] | None) – Comma-separated list of node IDs or names used to limit returned information.

node_id is passed as a query parameter: https://github.com/elastic/elasticsearch-py/blob/main/elasticsearch/_sync/client/tasks.py#L200

If this parameter is set, _perform_request will fail because it expects nodes not node_id:

elasticsearch.BadRequestError: BadRequestError(400, 'illegal_argument_exception', 'request [/_tasks] contains unrecognized parameter: [node_id] -> did you mean [nodes]?')

If I change the underlying call to self.perform_request to assign the query parameter to key nodes instead of node_id the request succeeds:

es_client.tasks.perform_request("GET", "/_tasks", params={"nodes":["node_id_1", "node_id_2"]}, headers={"accept": "application/json"}, endpoint_id="tasks.list", path_parts={})

but passing nodes to task.list is not accepted.

I believe this line should be changed to:

            __query["nodes"] = node_id

Alternatively the list function could be changed from using node_id to using nodes on the query parameters as well as the function parameters.

[pquentin]

Thank you @margaretpearce for your report (and pull request)! The code you edited is generated from the Elasticsesrch specification, so I transferred the issue here. The relevant part:

elasticsearch-specification/specification/tasks/list/ListTasksRequest.ts

Lines 48 to 51 in 36dbbd9

	/**
	* Comma-separated list of node IDs or names used to limit returned information.
	*/
	node_id?: string[]

If you are still interested in fixing this, you can open a pull request against this new repo. Otherwise no worries we will fix it in the coming week and this will be in elasticsearch-py 8.16.0.

[margaretpearce]

Thank you for letting me know @pquentin! I gave the fix in the correct repo a shot: #3113