Fixing register agent tests

[joshpalis]

Description

Fixes register agent tests failing due to opensearch-project/ml-commons#2204

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.75%. Comparing base (aa8a454) to head (e991a40).

Additional details and impacted files

@@            Coverage Diff            @@
##               main     #595   +/-   ##
=========================================
  Coverage     72.75%   72.75%           
  Complexity      681      681           
=========================================
  Files            82       82           
  Lines          3491     3491           
  Branches        279      279           
=========================================
  Hits           2540     2540           
  Misses          832      832           
  Partials        119      119           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[joshpalis]

Seeing some failures in integration tests, seems its an issue with the ML Indices handler being unable to create the ml model index

» ERROR][o.o.m.e.i.MLIndicesHandler] [integTest-0] Failed to create index .plugins-ml-model
»  com.fasterxml.jackson.core.JsonParseException: Unexpected character ('}' (code 125)): was expecting double-quote to start field name
»   at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 54, column: 1]
»  	at com.fasterxml.jackson.core.JsonParser._constructReadException(JsonParser.java:2643) ~[jackson-core-2.17.0.jar:2.17.0]
»  	at com.fasterxml.jackson.core.base.ParserMinimalBase._reportUnexpectedChar(ParserMinimalBase.java:685) ~[jackson-core-2.17.0.jar:2.17.0]
»  	at com.fasterxml.jackson.core.json.ReaderBasedJsonParser._handleOddName(ReaderBasedJsonParser.java:1910) ~[jackson-core-2.17.0.jar:2.17.0]
»  	at com.fasterxml.jackson.core.json.ReaderBasedJsonParser.nextToken(ReaderBasedJsonParser.java:709) ~[jackson-core-2.17.0.jar:2.17.0]
»  	at org.opensearch.common.xcontent.json.JsonXContentParser.nextToken(JsonXContentParser.java:66) ~[opensearch-x-content-3.0.0-SNAPSHOT.jar:3.0.0-SNAPSHOT]

[dbwiddis]

Unexpected character ('}' (code 125)): was expecting double-quote to start field name

Are we missing a mandatory field when creating the model? CC: @dhrubo-os

[dbwiddis]

These errors are occurring on ML-Commons 2.x branch tests: see https://github.com/opensearch-project/ml-commons/actions/runs/8340647023/job/22825002139 for example.

Test run history for 2.x branch: https://github.com/opensearch-project/ml-commons/actions/workflows/CI-workflow.yml?query=branch%3A2.x

CC: @rbhavna @dhrubo-os

[dhrubo-os]

Unexpected character ('}' (code 125)): was expecting double-quote to start field name

Are we missing a mandatory field when creating the model? CC: @dhrubo-os

We fixed the bug now: opensearch-project/ml-commons#2234