Skip to content
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

[Backport 2.x] enabled mockTelemetryPlugin for IT and fixed OOM #13268

Closed

Conversation

atharvasharma61
Copy link
Contributor

@atharvasharma61 atharvasharma61 commented Apr 17, 2024

Description

This PR backports #13054

Related Issues

Resolves #13054

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)
  • Public documentation issue/PR created

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.

gbbafna and others added 30 commits February 16, 2024 15:21
…ion (opensearch-project#11986)

* Introducing mixed mode support for remote store migration

Signed-off-by: Gaurav Bafna <gbbafna@amazon.com>

* Adding changelog

Signed-off-by: Gaurav Bafna <gbbafna@amazon.com>

* Refactoring IT and naming changes

Signed-off-by: Gaurav Bafna <gbbafna@amazon.com>

* Missing java doc

Signed-off-by: Gaurav Bafna <gbbafna@amazon.com>

---------

Signed-off-by: Gaurav Bafna <gbbafna@amazon.com>
Signed-off-by: RS146BIJAY <rishavsagar4b1@gmail.com>
…tart() will always be followed by onPhaseEnd() within AbstractSearchAsyncAction (opensearch-project#12293)

* Add support of special WrappingSearchAsyncActionPhase so the onPhaseStart() will always be followed by onPhaseEnd() within AbstractSearchAsyncAction

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

* Address code review comments

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

---------

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
…ch-project#12307)

* Enable test suites to run with segrep

Signed-off-by: Poojita Raj <poojiraj@amazon.com>

* Refactor static settings into ParameterizedStaticSettingsOpenSearchIntegTestCase

Signed-off-by: Poojita Raj <poojiraj@amazon.com>

---------

Signed-off-by: Poojita Raj <poojiraj@amazon.com>
…ct#12337)

* Avoid string out of bounds error in snapshot delete

Test failure opensearch-project#8771 shows cases where certain random seeds trigger this
case. The bug is clear: the substring() call should happen after the
startsWith() check in case the blob name is shorter than the prefix
length being used as the start index of the substring call. I don't yet
know if/how this manifests in real deployments.

Signed-off-by: Andrew Ross <andrross@amazon.com>

* Extract common UUID parsing method

Signed-off-by: Andrew Ross <andrross@amazon.com>

---------

Signed-off-by: Andrew Ross <andrross@amazon.com>
…#12367)

* Bump gradle/wrapper-validation-action from 1 to 2

Bumps [gradle/wrapper-validation-action](https://github.com/gradle/wrapper-validation-action) from 1 to 2.
- [Release notes](https://github.com/gradle/wrapper-validation-action/releases)
- [Commits](gradle/wrapper-validation-action@v1...v2)

---
updated-dependencies:
- dependency-name: gradle/wrapper-validation-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

* Update changelog

Signed-off-by: dependabot[bot] <support@github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
Signed-off-by: RS146BIJAY <rishavsagar4b1@gmail.com>
)

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
…v1.1 (opensearch-project#12387)

Signed-off-by: Peter Nied <petern@amazon.com>
Signed-off-by: Peter Nied <peternied@hotmail.com>
Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
When this test fails, it takes [15 minutes to timeout][1]. There are
many examples like this. While it generally succeeds after a retry or
two, I think it is better to mute this test rather than observe regular
failures that extend build times.

[1]: https://build.ci.opensearch.org/job/gradle-check/33937/testReport/

Signed-off-by: Andrew Ross <andrross@amazon.com>
)

Signed-off-by: Daniel Widdis <widdis@gmail.com>
Views, simplify data access and manipulation by providing a virtual layer over one or more indices

Signed-off-by: Peter Nied <petern@amazon.com>
Signed-off-by: Peter Nied <peternied@hotmail.com>
…ch-project#11193)

When migrating from older versions of OpenSearch index.mapper.dynamic
could be present, use the deprecation system to warn about this state and
ignore this legacy setting so the index is usable.

Signed-off-by: Peter Nied <petern@amazon.com>
…s flaky (opensearch-project#12415)

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
…12077)

* Fix flaky test testScrollCreatedOnReplica

Signed-off-by: Marc Handalian <marc.handalian@gmail.com>

* Disable scheduled refresh

Signed-off-by: Marc Handalian <marc.handalian@gmail.com>

* Clean up segment collection assertions

Signed-off-by: Marc Handalian <marc.handalian@gmail.com>

* Fix spotless

Signed-off-by: Marc Handalian <marc.handalian@gmail.com>

---------

Signed-off-by: Marc Handalian <marc.handalian@gmail.com>
* Add bwc version 2.12.1

Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: Kunal Kotwani <kkotwani@amazon.com>

* Fix version identifier for 2.12.1

Signed-off-by: Kunal Kotwani <kkotwani@amazon.com>

---------

Signed-off-by: GitHub <noreply@github.com>
Signed-off-by: Kunal Kotwani <kkotwani@amazon.com>
Co-authored-by: opensearch-ci-bot <opensearch-ci-bot@users.noreply.github.com>
Co-authored-by: Kunal Kotwani <kkotwani@amazon.com>
…ject#12297)

* Add new method to pick a random replication strategy.

Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com>

* replace usage of refresh with refreshandWaitForReplication()

Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com>

* Add replication strategy logging.

Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com>

---------

Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com>
This test was muted due to an [intermediate state][1] during type
removal. It now passes per my local testing.

[1]: opensearch-project#2440 (comment)

Signed-off-by: Andrew Ross <andrross@amazon.com>
This PR was committed on main but not backported to 2.x in time for the
2.12 release.

Signed-off-by: Andrew Ross <andrross@amazon.com>
* Shard id awareness of SearchLookup

Signed-off-by: Alexander Koval <kovalidis@gmail.com>

* Add unit test for deprecated constructor

Signed-off-by: Andrew Ross <andrross@amazon.com>

---------

Signed-off-by: Alexander Koval <kovalidis@gmail.com>
Signed-off-by: Andrew Ross <andrross@amazon.com>
Co-authored-by: Andrew Ross <andrross@amazon.com>
…ildSrc/src/testKit/thirdPartyAudit (opensearch-project#12464)

* Bump org.apache.logging.log4j:log4j-core

Bumps org.apache.logging.log4j:log4j-core from 2.22.1 to 2.23.0.

---
updated-dependencies:
- dependency-name: org.apache.logging.log4j:log4j-core
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* Update changelog

Signed-off-by: dependabot[bot] <support@github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
…ribution/packages (opensearch-project#12461)

* Bump com.netflix.nebula.ospackage-base in /distribution/packages

Bumps com.netflix.nebula.ospackage-base from 11.8.0 to 11.8.1.

---
updated-dependencies:
- dependency-name: com.netflix.nebula.ospackage-base
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* Update changelog

Signed-off-by: dependabot[bot] <support@github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
Copy link
Contributor

❌ Gradle check result for dc25246: 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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Backport to 2.x branch backport-failed bug Something isn't working flaky-test Random test failure that succeeds on second run Other
Projects
None yet
Development

Successfully merging this pull request may close these issues.