-
Notifications
You must be signed in to change notification settings - Fork 2.3k
[Forwardport main] Fix @timestamp upgrade issue by adding a version check on skip_list param #19677
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…aram (bwc) (#19671) Fixes #19660 Signed-off-by: Asim Mahmood <[email protected]> Signed-off-by: Sandesh Kumar <[email protected]> Co-authored-by: Sandesh Kumar <[email protected]> (cherry picked from commit 0b6a2c9) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Signed-off-by: Sandesh Kumar <[email protected]>
|
#19671 and #19652 are part of 3.3 branch (upcoming 3.3.1 patch release) so removed it from changelog of cc: @jainankitk |
|
❌ Gradle check result for 049e93a: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
|
❌ Gradle check result for 049e93a: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
Probably one of the version related PR needs to merge first for resolving gradle check |
|
❌ Gradle check result for 049e93a: null Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
|
Rebased on #19703 |
|
❌ Gradle check result for 4fe914b: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
|
❌ Gradle check result for 4fe914b: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #19677 +/- ##
============================================
+ Coverage 73.07% 73.12% +0.05%
- Complexity 70741 70831 +90
============================================
Files 5732 5732
Lines 324140 324141 +1
Branches 46922 46923 +1
============================================
+ Hits 236853 237030 +177
+ Misses 68117 67991 -126
+ Partials 19170 19120 -50 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…heck on skip_list param (opensearch-project#19677) * Fix @timestamp upgrade issue by adding a version check on skip_list param (bwc) (opensearch-project#19671) Fixes opensearch-project#19660 Signed-off-by: Asim Mahmood <[email protected]> Signed-off-by: Sandesh Kumar <[email protected]> Co-authored-by: Sandesh Kumar <[email protected]> (cherry picked from commit 0b6a2c9) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> * Removed changelog of items part of 3.3.1 Signed-off-by: Sandesh Kumar <[email protected]> --------- Signed-off-by: Asim Mahmood <[email protected]> Signed-off-by: Sandesh Kumar <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Sandesh Kumar <[email protected]> Co-authored-by: Peter Zhu <[email protected]>
Backport 0b6a2c9 from #19671.