Daniel Widdis

[Feature/multi_tenancy] Tenant-aware integration tests for Connector, Model, Agent, Model Groups

12

### Description Tests the entire Create-Get-Update-Search-Delete cycle with multi-tenancy (both enabled and not). Validates expected results when tenant aware and when not (current status quo). Looking for review on the...

[BUG] Race Condition when running multiple integ tests invoking EncryptorImpl.encrypt early

1

**What is the bug?** > What we have is a race condition: > 1. When the first test starts it creates a (local) ML Config index (which isn't needed) >...

### Description Removes the ability to change the value of the mutiTenancy setting: it must be initialized in `opensearch.yml`. Two other small changes: - When multiTenancy was false (which is...

[BUG] MLResourceNotFoundException leads to 500 error

1

**What is the bug?** The [`MLResourceNotFoundException`](https://github.com/opensearch-project/ml-commons/blob/main/common/src/main/java/org/opensearch/ml/common/exception/MLResourceNotFoundException.java) is just a decorated `RuntimeException` with special handling for ML Stats. However, it's used in cases where a document is not found which should...

**Is your feature request related to a problem?** OpenSearch indexes have eventual read consistency. In general, indexed documents might not be available in search until the next refresh. On an...

[BUG] Created Time and Last Updated Time are not implemented on Connector

1

**What is the bug?** The `.plugin-ml-connector` index contains field mappings for `created_time` and `last_updated_time` but these are never populated. **How can one reproduce the bug?** Steps to reproduce the behavior:...

[PROPOSAL] Automate merging of branch version bump PRs

12

## What/Why ### What are you proposing? We should enable repo/admin-level automation to merging branch version bump PRs because maintainers aren't doing it. ### What users have asked for this...

Replace String concatenation with Log4j ParameterizedMessage for readability

2

### Is your feature request related to a problem? Log4j includes parameter substitution which improves readability of log messages and has improved performance and stability over similar `String.format()`: ``` logger.info("Updating...

Add optional config field to tool step

1

### Description Adds the optional `config` map to the `ToolStep` ### Related Issues Resolves #878 Draft, pending merge: - https://github.com/opensearch-project/ml-commons/pull/2977 ### Check List - [x] New functionality includes testing. -...

[BUG] Reprovision Workflow IT is flaky

5

### What is the bug? `FlowFrameworkRestApiIT.testReprovisionWorkflow()` is flaky. During 2.17.0 release it required multiple retries to pass on Windows x86. There are occasional GitHub CI failures on other platforms, see...