Skip to content

Conversation

Divyaasm
Copy link
Collaborator

Description

Add release notes for 2.19.3

Issues Resolved

List any issues this PR will resolve, e.g. Closes [...].

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.

Signed-off-by: Divya Madala <[email protected]>
Copy link
Collaborator

@rishabh6788 rishabh6788 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!

Comment on lines 15 to 21
### Opensearch Flow Framework


* Fix ApiSpecFetcher Memory Issues and Exception Handling ([#1192](https://github.com/opensearch-project/flow-framework/pull/1192))


* Better handling of Workflow Steps with Bad Request status ([#1191](https://github.com/opensearch-project/flow-framework/pull/1191))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Patch releases should have an enhancements. Check with the owners for right category please.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which category do you recommend, will change it for the consolidated ones, component can be tracked separately.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ohltyler @owaiskazi @dbwiddis Can you guys recommend? I believe bug fix and refactoring should be the okay. Please correct if I am wrong.
Thanks!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug fix

Comment on lines 66 to 72
## INFRASTRUCTURE


### SQL


* Increment version to 2.19.3-SNAPSHOT ([#3601](https://github.com/opensearch-project/sql/pull/3601))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this required? We might need to add for all components right?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Divyaasm Lets remove it form here

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not mandatory for patch right? We only include this if it's part of component release notes

## Bug fix


* validate that index patterns are not allowed in create/update doc level monitor ([#829](https://github.com/opensearch-project/common-utils/pull/829))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* validate that index patterns are not allowed in create/update doc level monitor ([#829](https://github.com/opensearch-project/common-utils/pull/829))
* Validate that index patterns are not allowed in create/update doc level monitor ([#829](https://github.com/opensearch-project/common-utils/pull/829))


* validate that index patterns are not allowed in create/update doc level monitor ([#829](https://github.com/opensearch-project/common-utils/pull/829))
* Fix isDocLevelMonitor check to account for threat intel monitor ([#835](https://github.com/opensearch-project/common-utils/pull/835))
* updating PublishFindingsRequest to use a list of findings rather than... ([#832](https://github.com/opensearch-project/common-utils/pull/832))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* updating PublishFindingsRequest to use a list of findings rather than... ([#832](https://github.com/opensearch-project/common-utils/pull/832))
* Updating PublishFindingsRequest to use a list of findings rather than... ([#832](https://github.com/opensearch-project/common-utils/pull/832))


* Java version bump ([#828](https://github.com/opensearch-project/performance-analyzer/pull/828))
* Spotbug version bump ([#828](https://github.com/opensearch-project/performance-analyzer/pull/828))
* checkstyle version bump ([#828](https://github.com/opensearch-project/performance-analyzer/pull/828))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* checkstyle version bump ([#828](https://github.com/opensearch-project/performance-analyzer/pull/828))
* Checkstyle version bump ([#828](https://github.com/opensearch-project/performance-analyzer/pull/828))

Comment on lines 210 to 213
* Bump `com.nimbusds:nimbus-jose-jwt:9.48` from 9.48 to 10.0.2 ([#5480](https://github.com/opensearch-project/security/pull/5480))


* Bump `checkstyle` from 10.3.3 to 10.26.1 ([#5480](https://github.com/opensearch-project/security/pull/5480))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Bump `com.nimbusds:nimbus-jose-jwt:9.48` from 9.48 to 10.0.2 ([#5480](https://github.com/opensearch-project/security/pull/5480))
* Bump `checkstyle` from 10.3.3 to 10.26.1 ([#5480](https://github.com/opensearch-project/security/pull/5480))
* Bump `com.nimbusds:nimbus-jose-jwt:9.48` from 9.48 to 10.0.2 ([#5480](https://github.com/opensearch-project/security/pull/5480))
* Bump `checkstyle` from 10.3.3 to 10.26.1 ([#5480](https://github.com/opensearch-project/security/pull/5480))

Copy link

codecov bot commented Jul 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.45%. Comparing base (ab2f098) to head (159cbe3).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5655   +/-   ##
=======================================
  Coverage   96.45%   96.45%           
=======================================
  Files         398      398           
  Lines       17550    17550           
=======================================
  Hits        16928    16928           
  Misses        622      622           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Member

@peterzhuamazon peterzhuamazon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Divyaasm can you please change all Opensearch to OpenSearch and add OpenSearch in front of SQL on headings?

Thanks.

* [BUG FIX] Enable Correct Sorting for Metrics in Query Insights Dashboard + dependent PRs ([#201](https://github.com/opensearch-project/query-insights-dashboards/pull/201))
* [Bug Fix] Window size changing unexpectedly ([#203](https://github.com/opensearch-project/query-insights-dashboards/pull/203))
* [Fix] Ensure accurate time filtering for Top Queries in OpenSearch 2.19+ ([#249](https://github.com/opensearch-project/query-insights-dashboards/pull/249))
* Ensure accurate time filtering for Top Queries in OpenSearch 2.19 + revert [#219] ([#274](https://github.com/opensearch-project/query-insights-dashboards/pull/274))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Ensure accurate time filtering for Top Queries in OpenSearch 2.19 + revert [#219] ([#274](https://github.com/opensearch-project/query-insights-dashboards/pull/274))
* Ensure accurate time filtering for Top Queries in OpenSearch 2.19 + revert [#219](https://github.com/opensearch-project/query-insights-dashboards/pull/219) ([#274](https://github.com/opensearch-project/query-insights-dashboards/pull/274))

@peterzhuamazon
Copy link
Member

Hi @Divyaasm can you please change all Opensearch to OpenSearch and add OpenSearch in front of SQL on headings?

Thanks.

Divyaasm added 2 commits July 22, 2025 11:35
Signed-off-by: Divya Madala <[email protected]>
Signed-off-by: Divya Madala <[email protected]>
* Backports #1850, #1854, #1856 to 2.19 ([#1858](https://github.com/opensearch-project/common-utils/pull/1858))


### Opensearch Common Utils
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
### Opensearch Common Utils
### OpenSearch Common Utils

### OpenSearch Flow Framework


* Fix ApiSpecFetcher Memory Issues and Exception Handling ([#1192](https://github.com/opensearch-project/flow-framework/pull/1192))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move this to bug fix section



* Fix ApiSpecFetcher Memory Issues and Exception Handling ([#1192](https://github.com/opensearch-project/flow-framework/pull/1192))
* Better handling of Workflow Steps with Bad Request status ([#1191](https://github.com/opensearch-project/flow-framework/pull/1191))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Move this to refactoring section

Comment on lines 59 to 64
## DOCUMENTATION


## ENHANCEMENTS
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove

* Better handling of Workflow Steps with Bad Request status ([#1191](https://github.com/opensearch-project/flow-framework/pull/1191))


## INFRASTRUCTURE
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove if not required

* [2.19] Moved the commons-beanutils pinning to the core gradle file ([#1893](https://github.com/OpenSearch-project/common-utils/pull/1893))


### Opensearch Anomaly Detection
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
### Opensearch Anomaly Detection
### OpenSearch Anomaly Detection

* Fixed CVE: babel dependencies & elliptic dependency ([#1329](https://github.com/opensearch-project/index-management-dashboards-plugin/pull/1329))


### OpenSearch ML Common
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
### OpenSearch ML Common
### OpenSearch ML Commons


## MAINTENANCE

### Opensearch Alerting Dashboards Plugin
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
### Opensearch Alerting Dashboards Plugin
### OpenSearch Alerting Dashboards Plugin

* Bump elliptic to 6.6.1 ([#1062](https://github.com/OpenSearch-project/anomaly-detection-dashboards-plugin/pull/1062))


### Opensearch Common Utils
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
### Opensearch Common Utils
### OpenSearch Common Utils

* Migrating from commons-lang2.6 to commons-lang3.18 ([#1526](https://github.com/OpenSearch-project/anomaly-detection/pull/1526))


### Opensearch Anomaly Detection Dashboards
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
### Opensearch Anomaly Detection Dashboards
### OpenSearch Anomaly Detection Dashboards

Signed-off-by: Divya Madala <[email protected]>
Divyaasm added 2 commits July 22, 2025 12:10
Signed-off-by: Divya Madala <[email protected]>
Signed-off-by: Divya Madala <[email protected]>
Copy link
Member

@gaiksaya gaiksaya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Signed-off-by: Divya Madala <[email protected]>
@rishabh6788 rishabh6788 merged commit 54db595 into opensearch-project:main Jul 23, 2025
15 checks passed
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✅ Done in Engineering Effectiveness Board Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

5 participants