Updating the sdk patch for model metadata service

Author: SachinVarghese

README.md

@@ -69,5 +69,8 @@ These extra files and customisation can be found in the
 1. Update the [swagger file](./swagger-v1alpha1.yml) with latest specification.
 2. Update [config file](./config/python.json) to bump the package version.
 3. Run `make python` to re-generate the sdk api files from new specifications.
+
+    **Note** : For Seldon Deploy model metadata service, we are having to patch two additional dict type parameters namely `tags` and `metrics` to the generated docs and sdk code. We do this manually, using the same make command, by applying the [patch file](./templates/python/metadata_tags_metrics.patch) file to the code, post sdk generation task. In some cases the patch apply cmd may fail, and there might be a requirement to re-generate the patch manually by changing the generated docs/sdk files as required.
+
 4. Run `make -C build push` to build & push latest release to [PyPi](https://pypi.org/project/seldon-deploy-sdk/)
 5. Create a new Github Tag and Release with latest version and notes.

templates/python/metadata_tags_metrics.patch

@@ -1,5 +1,5 @@
 diff --git a/python/docs/ModelMetadataServiceApi.md b/python/docs/ModelMetadataServiceApi.md
-index 4ca2775..481780a 100644
+index b257b08..3da9999 100644
 --- a/python/docs/ModelMetadataServiceApi.md
 +++ b/python/docs/ModelMetadataServiceApi.md
 @@ -140,6 +140,8 @@ version = 'version_example' # str |  (optional)
@@ -11,16 +11,16 @@ index 4ca2775..481780a 100644
  query = 'query_example' # str | For more complex queries where other logical operators like OR, NOT, etc. (optional)
  page_size = 56 # int | Optional. The maximum number of Folders to return in the response. (optional)
  page_token = 'page_token_example' # str | Optional. A pagination token returned from a previous call to `List` that indicates where this listing should continue from. (optional)
-@@ -149,7 +151,7 @@ order_by = 'order_by_example' # str | The order in which to return the model met
+@@ -150,7 +152,7 @@ default_protocol = 'PROTOCOL_UNKNOWN' # str |  (optional) (default to PROTOCOL_U
 
  try:
      # List Model Metadata entries.
--    api_response = api_instance.model_metadata_service_list_model_metadata(uri=uri, name=name, version=version, artifact_type=artifact_type, task_type=task_type, model_type=model_type, query=query, page_size=page_size, page_token=page_token, list_mask=list_mask, project=project, order_by=order_by)
-+    api_response = api_instance.model_metadata_service_list_model_metadata(uri=uri, name=name, version=version, artifact_type=artifact_type, task_type=task_type, model_type=model_type, tags=tags, metrics=metrics, query=query, page_size=page_size, page_token=page_token, list_mask=list_mask, project=project, order_by=order_by)
+-    api_response = api_instance.model_metadata_service_list_model_metadata(uri=uri, name=name, version=version, artifact_type=artifact_type, task_type=task_type, model_type=model_type, query=query, page_size=page_size, page_token=page_token, list_mask=list_mask, project=project, order_by=order_by, default_protocol=default_protocol)
++    api_response = api_instance.model_metadata_service_list_model_metadata(uri=uri, name=name, version=version, artifact_type=artifact_type, task_type=task_type, model_type=model_type, tags=tags, metrics=metrics, query=query, page_size=page_size, page_token=page_token, list_mask=list_mask, project=project, order_by=order_by, default_protocol=default_protocol)
      pprint(api_response)
  except ApiException as e:
      print("Exception when calling ModelMetadataServiceApi->model_metadata_service_list_model_metadata: %s\n" % e)
-@@ -165,6 +167,8 @@ Name | Type | Description  | Notes
+@@ -166,6 +168,8 @@ Name | Type | Description  | Notes
   **artifact_type** | **str**|  | [optional] [default to UNKNOWN]
   **task_type** | **str**|  | [optional] 
   **model_type** | **str**|  | [optional] 
@@ -30,7 +30,7 @@ index 4ca2775..481780a 100644
   **page_size** | **int**| Optional. The maximum number of Folders to return in the response. | [optional] 
   **page_token** | **str**| Optional. A pagination token returned from a previous call to `List` that indicates where this listing should continue from. | [optional] 
 diff --git a/python/seldon_deploy_sdk/api/model_metadata_service_api.py b/python/seldon_deploy_sdk/api/model_metadata_service_api.py
-index 8b5d077..b667f5c 100644
+index 312e067..47adb43 100644
 --- a/python/seldon_deploy_sdk/api/model_metadata_service_api.py
 +++ b/python/seldon_deploy_sdk/api/model_metadata_service_api.py
 @@ -247,6 +247,8 @@ class ModelMetadataServiceApi(object):
@@ -42,7 +42,7 @@ index 8b5d077..b667f5c 100644
          :param str query: For more complex queries where other logical operators like OR, NOT, etc.
          :param int page_size: Optional. The maximum number of Folders to return in the response.
          :param str page_token: Optional. A pagination token returned from a previous call to `List` that indicates where this listing should continue from.
-@@ -280,6 +282,8 @@ class ModelMetadataServiceApi(object):
+@@ -281,6 +283,8 @@ class ModelMetadataServiceApi(object):
          :param str artifact_type:
          :param str task_type:
          :param str model_type:
@@ -51,16 +51,16 @@ index 8b5d077..b667f5c 100644
          :param str query: For more complex queries where other logical operators like OR, NOT, etc.
          :param int page_size: Optional. The maximum number of Folders to return in the response.
          :param str page_token: Optional. A pagination token returned from a previous call to `List` that indicates where this listing should continue from.
-@@ -291,7 +295,7 @@ class ModelMetadataServiceApi(object):
+@@ -293,7 +297,7 @@ class ModelMetadataServiceApi(object):
                   returns the request thread.
          """
 
--        all_params = ['uri', 'name', 'version', 'artifact_type', 'task_type', 'model_type', 'query', 'page_size', 'page_token', 'list_mask', 'project', 'order_by']  # noqa: E501
-+        all_params = ['uri', 'name', 'version', 'artifact_type', 'task_type', 'model_type', 'tags', 'metrics', 'query', 'page_size', 'page_token', 'list_mask', 'project', 'order_by']  # noqa: E501
+-        all_params = ['uri', 'name', 'version', 'artifact_type', 'task_type', 'model_type', 'query', 'page_size', 'page_token', 'list_mask', 'project', 'order_by', 'default_protocol']  # noqa: E501
++        all_params = ['uri', 'name', 'version', 'artifact_type', 'task_type', 'model_type', 'tags', 'metrics', 'query', 'page_size', 'page_token', 'list_mask', 'project', 'order_by', 'default_protocol']  # noqa: E501
          all_params.append('async_req')
          all_params.append('_return_http_data_only')
          all_params.append('_preload_content')
-@@ -324,6 +328,12 @@ class ModelMetadataServiceApi(object):
+@@ -326,6 +330,12 @@ class ModelMetadataServiceApi(object):
              query_params.append(('taskType', params['task_type']))  # noqa: E501
          if 'model_type' in params:
              query_params.append(('modelType', params['model_type']))  # noqa: E501