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

Evictperms #1

Closed
wants to merge 13 commits into from
Closed

Evictperms #1

wants to merge 13 commits into from

Conversation

finnegancarroll
Copy link
Owner

Description

[Describe what this change achieves]

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

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.

chishui and others added 13 commits June 14, 2024 09:57
* Fix flaky TSC stats tests

Signed-off-by: Peter Alfonsi <[email protected]>

* Addressed andrross's comment

Signed-off-by: Peter Alfonsi <[email protected]>

* fix forbidden API

Signed-off-by: Peter Alfonsi <[email protected]>

---------

Signed-off-by: Peter Alfonsi <[email protected]>
Co-authored-by: Peter Alfonsi <[email protected]>
* Add fingerprint ingest processor

Signed-off-by: Gao Binlong <[email protected]>

* Ignore metadata fields

Signed-off-by: Gao Binlong <[email protected]>

* Add sha3-256 hash method

Signed-off-by: Gao Binlong <[email protected]>

* Remove unused code

Signed-off-by: Gao Binlong <[email protected]>

* Add exclude_fields and remove include_all_fields

Signed-off-by: Gao Binlong <[email protected]>

* Modify processor description

Signed-off-by: Gao Binlong <[email protected]>

* Make FingerprintProcessor being final

Signed-off-by: Gao Binlong <[email protected]>

* Optimize error message and check if field name is empty string

Signed-off-by: Gao Binlong <[email protected]>

* Fix yaml test failure

Signed-off-by: Gao Binlong <[email protected]>

* Prepend string length to the field value

Signed-off-by: Gao Binlong <[email protected]>

* Append hash method with version number

Signed-off-by: Gao Binlong <[email protected]>

* Update supported version in yml test file

Signed-off-by: Gao Binlong <[email protected]>

* Add more comment

Signed-off-by: Gao Binlong <[email protected]>

* Prepend hash method to the hash value and add more test cases

Signed-off-by: Gao Binlong <[email protected]>

---------

Signed-off-by: Gao Binlong <[email protected]>
…h-project#14345)

* COMPAT locale provider will be removed in a future release

Description:
From JDK21 onwards a new warning has started to come, "WARNING: COMPAT locale provider will be removed in a future release". So, we have to avoid usage of COMPAT provider. We were setting exlpicitly to COMPAT locale provider in couple of places, this change is to convert COMPAT to CLDR locale provider. After this change, couple of tests started to fail becasue some locale data has minor changes in CLDR compared to COMPAT. For example, day and month short names of GERMAN "de" locale are different in CLDR and COMPAT, just need to add a . in the end for CLDR.

Resolves opensearch-project#11550

Signed-off-by: Abdul Muneer Kolarkunnu <[email protected]>

* COMPAT locale provider will be removed in a future release

Description:
From JDK21 onwards a new warning has started to come, "WARNING: COMPAT locale provider will be removed in a future release". So, we have to avoid usage of COMPAT provider. We were setting exlpicitly to COMPAT locale provider in couple of places, this change is to convert COMPAT to CLDR locale provider. After this change, couple of tests started to fail becasue some locale data has minor changes in CLDR compared to COMPAT. For example, day and month short names of GERMAN "de" locale are different in CLDR and COMPAT, just need to add a . in the end for CLDR.

Resolves opensearch-project#11550

Signed-off-by: Abdul Muneer Kolarkunnu <[email protected]>

---------

Signed-off-by: Abdul Muneer Kolarkunnu <[email protected]>
…opensearch-project#14372)

The problem is that this test would decommission one of six nodes. The
tear down logic of the test would attempt to assert on the health of the
cluster by randomly selecting a node and requesting the cluster health.
If this random check happened to select the node that was
decommissioned, then the test would fail. The fix is to recommission
the node at the end of the test.

Also, the "recommission node and assert cluster health" logic was used
in multiple places and could be refactored out to a helper method.

Resolves opensearch-project#14290
Resolves opensearch-project#12197

Signed-off-by: Andrew Ross <[email protected]>
…pensearch-project#14338)

* Simplify updated customs (ClusterState.Custom & Metadata.Custom) persistence logic to remote store

Signed-off-by: Sooraj Sinha <[email protected]>
…hdfs-fixture (opensearch-project#14396)

* Bump commons-net:commons-net in /test/fixtures/hdfs-fixture

Bumps commons-net:commons-net from 3.10.0 to 3.11.1.

---
updated-dependencies:
- dependency-name: commons-net:commons-net
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* Update changelog

Signed-off-by: dependabot[bot] <[email protected]>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
…in /plugins/repository-hdfs (opensearch-project#14399)

* Bump org.apache.commons:commons-configuration2

Bumps org.apache.commons:commons-configuration2 from 2.10.1 to 2.11.0.

---
updated-dependencies:
- dependency-name: org.apache.commons:commons-configuration2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* Updating SHAs

Signed-off-by: dependabot[bot] <[email protected]>

* Update changelog

Signed-off-by: dependabot[bot] <[email protected]>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants