opensearch-php icon indicating copy to clipboard operation
opensearch-php copied to clipboard

Updated opensearch-php to reflect the latest OpenSearch API spec

Open opensearch-trigger-bot[bot] opened this issue 1 year ago • 2 comments

Updated opensearch-php to reflect the latest OpenSearch API spec Date: 2024-09-24

I don't merge all of these to gather more of the API changes together, then merge all at once before a release.

dblock avatar Aug 20 '24 13:08 dblock

Some legit errors that need to be looked at from the generated code.

1) OpenSearch\Tests\Namespaces\MlNamespaceTest::testDeployModel
OpenSearch\Common\Exceptions\RuntimeException: Missing parameter for the endpoint ml.deploy_model

/Users/runner/work/opensearch-php/opensearch-php/src/OpenSearch/Endpoints/Ml/DeployModel.php:32
/Users/runner/work/opensearch-php/opensearch-php/src/OpenSearch/Namespaces/AbstractNamespace.php:63
/Users/runner/work/opensearch-php/opensearch-php/src/OpenSearch/Namespaces/MlNamespace.php:273
/Users/runner/work/opensearch-php/opensearch-php/tests/Namespaces/MachineLearningNamespaceTest.php:[32](https://github.com/opensearch-project/opensearch-php/actions/runs/10571259471/job/29287037384?pr=223#step:5:33)6

2) OpenSearch\Tests\Namespaces\MlNamespaceTest::testUnDeployModel
OpenSearch\Common\Exceptions\RuntimeException: Missing parameter for the endpoint ml.undeploy_model

/Users/runner/work/opensearch-php/opensearch-php/src/OpenSearch/Endpoints/Ml/UndeployModel.php:32
/Users/runner/work/opensearch-php/opensearch-php/src/OpenSearch/Namespaces/AbstractNamespace.php:63
/Users/runner/work/opensearch-php/opensearch-php/src/OpenSearch/Namespaces/MlNamespace.php:420
/Users/runner/work/opensearch-php/opensearch-php/tests/Namespaces/MachineLearningNamespaceTest.php:[34](https://github.com/opensearch-project/opensearch-php/actions/runs/10571259471/job/29287037384?pr=223#step:5:35)6

dblock avatar Aug 27 '24 10:08 dblock

The ML namespace changed parameter names, fixed the templates.

dblock avatar Oct 25 '24 15:10 dblock