2.3 new API's feature parity

This commit adds support for the new 2.3 API's:

* tasks_list
* tasks_cancel
* update_by_query
* reindex

The spec changes have also been updated in core upstream.

The Reindex API is called `ReindexOnServer()` to offset it against our
current `Reindex()` implementation which is still very useful.

Both the reindex and update_by_query steal the status code from one of
the failures and return a valid json body that we need to deserialize.
To support this this AllowedStatusCodes now accepts a -1.

.IsValid for these API's inspects whether any failures have been
returned or not.

Author: Mpdreamz

src/CodeGeneration/CodeGeneration.LowLevelClient/ApiEndpoints/tasks.cancel.json

@@ -7,7 +7,7 @@
       "paths": ["/_tasks/_cancel", "/_tasks/{task_id}/_cancel"],
       "parts": {
         "task_id": {
-          "type": "number",
+          "type": "string",
           "description": "Cancel the task with specified id"
         }
       },
@@ -25,7 +25,7 @@
           "description": "Cancel tasks with specified parent node."
         },
         "parent_task": {
-          "type" : "number",
+          "type" : "string",
           "description" : "Cancel tasks with specified parent task id. Set to -1 to cancel all."
         }
       }

src/CodeGeneration/CodeGeneration.LowLevelClient/ApiEndpoints/tasks.list.json

@@ -7,7 +7,7 @@
       "paths": ["/_tasks", "/_tasks/{task_id}"],
       "parts": {
         "task_id": {
-          "type": "number",
+          "type": "string",
           "description": "Return the task with specified id"
         }
       },
@@ -29,7 +29,7 @@
           "description": "Return tasks with specified parent node."
         },
         "parent_task": {
-          "type" : "number",
+          "type" : "string",
           "description" : "Return tasks with specified parent task id. Set to -1 to return all."
         },
         "wait_for_completion": {

src/CodeGeneration/CodeGeneration.LowLevelClient/ApiEndpoints/update_by_query.json

@@ -8,6 +8,7 @@
       "comment": "most things below this are just copied from search.json",
       "parts": {
         "index": {
+         "required" : true,
          "type" : "list",
          "description" : "A comma-separated list of index names to search; use `_all` or empty string to perform the operation on all indices"
         },

src/CodeGeneration/CodeGeneration.LowLevelClient/Overrides/Descriptors/ReindexOnServerDescriptorOverrides.cs

@@ -0,0 +1,16 @@
+using System.Collections.Generic;
+
+namespace CodeGeneration.LowLevelClient.Overrides.Descriptors
+{
+	// ReSharper disable once UnusedMember.Global
+	public class ReindexOnServerDescriptorOverrides : IDescriptorOverrides
+	{
+		public IEnumerable<string> SkipQueryStringParams => new []
+		{
+			"source"
+		};
+
+		public IDictionary<string, string> RenameQueryStringParams => null;
+	}
+
+}

src/Tests/tests.yaml

@@ -1,12 +1,10 @@
 # mode either u (unit test), i (integration test) or m (mixed mode)
-mode: i
+mode: m
 # the elasticsearch version that should be started
 # Can be a snapshot version of sonatype or "latest" to get the latest snapshot of sonatype
-elasticsearch_version: 5.2.0
-# cluster filter allows you to only run the integration tests of a particular cluster (cluster suffix not needed)
-# cluster_filter:
+elasticsearch_version: latest
 # whether we want to forcefully reseed on the node, if you are starting the tests with a node already running
 force_reseed: true
 # do not spawn nodes as part of the test setup if we find a node is already running
 # this is opt in during development in CI we never want to see our tests running against an already running node
-test_against_already_running_elasticsearch: false
+test_against_already_running_elasticsearch: true