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.

Sign up for GitHub

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

Jump to bottom

[PURIFY] Remove all trace of XPack code from High Level Rest Client #2

Closed
18 tasks done
nknize opened this issue Jan 30, 2021 · 1 comment
Closed
18 tasks done

[PURIFY] Remove all trace of XPack code from High Level Rest Client #2

nknize opened this issue Jan 30, 2021 · 1 comment
Labels
>FORK Related to the fork process Meta Meta issue, not directly linked to a PR :sanitize Removing elastic specific artifacts :xpack-removal Related to removal of x-pack

Comments

@nknize
Copy link
Collaborator

nknize commented Jan 30, 2021 •
edited
Loading

This meta issue tracks removal of references of Elastic licensed classes from the client/high-level-rest module.
@nknize nknize added :sanitize Removing elastic specific artifacts :xpack-removal Related to removal of x-pack >FORK Related to the fork process labels Jan 30, 2021
This was referenced Jan 30, 2021
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
Merged
This was referenced Feb 1, 2021
Closed
Merged
Merged
Closed
@nknize nknize mentioned this issue Feb 2, 2021
Merged
@adnapibar adnapibar mentioned this issue Feb 8, 2021
Closed
@adnapibar adnapibar added the Meta Meta issue, not directly linked to a PR label Feb 8, 2021
This was referenced Feb 8, 2021
Merged
Merged
Merged
adnapibar added a commit that referenced this issue Feb 9, 2021
@adnapibar
Remove x-pack aggregations. (#59)
ae1727d 
This PR removes the x-pack aggregations: string_stats, top_metrics and inference.

Resolves #51
Relates #2
adnapibar added a commit that referenced this issue Feb 9, 2021
@adnapibar
Remove x-pack data-frame analytics hlrc. (#62)
1bbc9b4 
This PR removes the hlrc for x-pack data-frame analytics.

Relates #2
@nknize nknize mentioned this issue Feb 11, 2021
Merged
@nknize
Copy link
Collaborator Author

nknize commented Feb 12, 2021

Closing meta issue since all child issues are complete. 🎉

@nknize nknize closed this as completed Feb 12, 2021
peternied pushed a commit that referenced this issue Mar 13, 2021
@adnapibar @peternied
Remove x-pack aggregations. (#59)
15638e8 
This PR removes the x-pack aggregations: string_stats, top_metrics and inference.

Resolves #51
Relates #2

Signed-off-by: Peter Nied <[email protected]>
peternied pushed a commit that referenced this issue Mar 13, 2021
@adnapibar @peternied
Remove x-pack data-frame analytics hlrc. (#62)
d625d84 
This PR removes the hlrc for x-pack data-frame analytics.

Relates #2

Signed-off-by: Peter Nied <[email protected]>
setiah added a commit that referenced this issue Mar 18, 2021
@nknize @seraphjiang
@adnapibar @saratvemulapalli @harold-wang @setiah @dnhatn @jdconrad @abbashus @peternied @CEHENKLE @bbarani @peterzhuamazon @stockholmux
[DOCS] Merge 'main' into 'oss-docs' - after DCO check was added (#349)
1c4b922 
* [PURIFY] Remove x-pack directory

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

* [PURIFY] Remove docs directory (#3)

This commit removes the doc directory

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

* Cleanup build-scan, remove publish scan to elastic server (#1) (#4)

Co-authored-by: Huan Jiang <[email protected]>
Signed-off-by: Peter Nied <[email protected]>

* [PURIFY] remove all trace of x-pack eql (#5)

This commit removes all trace of EQL from the sanitized fork.

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

* [PURIFY] remove issue, pr tempalte to avoid confusion, we could add olater (#10) (#6)

Co-authored-by: Huan Jiang <[email protected]>
Signed-off-by: Peter Nied <[email protected]>

* [PURIFY] update build.gradle files to ensure build completes; gradle check fails (#7)

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

* Cleanup build script to exclude security-authorization-engine  (#8) (#8)

* Cleanup build-scan, remove publish scan to elastic server

* Cleanup build script to exclude security-authorization-engine which test has dependency on xpack

* Cleanup build script to exclude security-authorization-engine which test has dependency on xpack

Co-authored-by: Huan Jiang <[email protected]>
Signed-off-by: Peter Nied <[email protected]>

* [PURIFY] remove all trace of x-pack enrichment processor (#9)

This commit removes all trace of the Elastic licensed enrichment processor.

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

* [PURIFY] remove all trace of x-pack async-search (#10)

This commit removes all trace of Elastic licensed asyc-search

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

* [PURIFY] remove all trace of x-pack ccr (#11)

This committ removes all trace of Elastic licensed CCR.

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

* Remove the Elastic license file, all checks for this license and the license REST APIs. (#12)

Co-authored-by: Rabi Panda <[email protected]>
Signed-off-by: Peter Nied <[email protected]>

* [PURIFY] remove all trace of x-pack graph (#13)

This commit removes all trace of Elastic licensed graph feature

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

* [PURIFY] remove all trace of x-pack ml (#14)

This commit removes all trace of x-pack ml.

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

* [PURIFY] Add InferenceConfig to org.elasticsearch.client.analytics (#15)

This commits adds InferenceConfig back to org.elasticsearch.client.analytics for use in InferencePipelineAggregationBuilder.

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

* [PURIFY] remove all trace of x-pack security (#16)

This commit removes all trace of the security high level rest client and other reference to x-pack security

Co-authored-by: Rabi Panda <[email protected]>
Signed-off-by: Peter Nied <[email protected]>

* [PURIFY] remove all trace of x-pack rollups (#17)

This commit removes all trace of Elastic licensed rollups

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

* [PURIFY] remove all trace of x-pack sql (#18)

This commit removes all trace of Elastic licensed SQL

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

* [PURIFY] remove all trace of x-pack migration (#19)

This commit removes all trace of Elastic licensed Migration

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

* [PURIFY] remove all trace of x-pack index lifecycle management (#20)

This commit removes all trace of Elastic licensed ILM.

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

* [PURIFY] remove all trace of x-pack watcher (#21)

This commit removes all trace of Elastic licensed watcher

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

* [PURIFY] remove all trace of x-pack monitoring (#22)

This commit removes all trace of Elastic licensed monitoring

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

* [PURIFY] Remove remaining x-pack license. (#25)

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

* [PURIFY] Revert "Move data stream transport and rest action to xpack (#59593)" (#28)

This commit reverts commit 2a89e13. Relicensing data streams from OSS to XPack.

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

* [PURIFY] remove all trace of x-pack transforms (#31)

This commit removes all trace of Elastic licensed transforms.

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

* [PURIFY] fix GetDataStreamsRequestTests build failure

This fixes the constructor for IndexNameExpressionResolver to pass in
Settings.EMPTY to a ThreadContext used by the resolver.

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

* [PURIFY] Remove the AuthorizationEnginePlugin from examples. (#26)

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

* Fix compilation issues for tests. (#29)

Some of the tests that use the x-pack code are failing compilation. This PR cleans up the references to fix the issue.

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

* [PURIFY] Cleanup build and unblock HLRC integration tests (#33)

This commit cleans up the following:

* Remove unused imports
* Remove ILM settings in hlrc testCluster formation
* Comment out security users settings in ElasticsearchNode creation for build-tools tests

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

* Adding initial CI workflow for search (#35)

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

* [TEST] Fix unit test failure in RestHighLevelClientTests (#36)

* fix testProvidedNamedXContents by modifying the assertion

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

* [TEST] fix DeleteDataStreamRequestTests failure (#37)

This commit fixes DeleteDataStreamRequestTests.testDeleteSnapshottingDataStream unit test failure by passing SnapshotsInProgress.Entry.SUCCESS in the createEntrymethod.

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

* Remove license option in gradlew command (#41)

* Remove license option in gradlew command

* Remove "This username and password is part of trial license. Let's remove this too" from TESTING.asciidoc

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

* Remove x-pack from build, distribution and packaging. (#43)

This PR removes references to x-pack from buildSrc, distribution and qa modules.

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

* Removing _reload_search_analyzers related changes since the related x-pack support is removed (#48)

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

* Fixing Rest Converters Tests after x-pack removal (#54)

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

* Remove license statement from CONTRIBUTING.md (#58)

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

* Revert back refresh policy in RequestConverters. (#55)

This PR reverts back the deleted code (#16, #54) related to refresh policies.

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

* Remove unused imports in RemoteClustersIT and InternalTestCluster

This commit removes unused imports that are causing checkStyle failures.

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

* [DOCS] temporarily comment verifyDocsLuceneVersion in qa:verify-version-constants

Docs have temporarily been removed. This commit can be reverted if the OSS
docs are restored.

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

* Mute AnalyticsAggsIT test failure

AnalyticsAggsIT needs to be removed. This mutes the test until removal is complete.

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

* [MUTE] AwaitsFix failing tests

This commit mutes failing tests in:
    * IndicesClientIT
    * SearchIT (Freeze/UnFreeze)
    * IndicesClientDocumentationIT

Fixes are identified and will be merged in a followup PR.

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

* Remove packaging tests for the x-pack command line tools. (#56)

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

* Remove x-pack aggregations. (#59)

This PR removes the x-pack aggregations: string_stats, top_metrics and inference.

Resolves #51
Relates #2

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

* Remove x-pack data-frame analytics hlrc. (#62)

This PR removes the hlrc for x-pack data-frame analytics.

Relates #2

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

* Remove ILM policy from GetDataStreamAction Response. (#63)

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

* Ensure ReplicationOperation notify listener once (#68256)

ReplicationOperation can notify the listener twice if the primary shard
is demoted after it has completed the primary operation.

Closes #68049

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

* Fix search template request (#43509)

A seed was hit in (#43157) that caused mutateInstance to generate an identical
instance. This change prevents that.

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

* Lower skip version for token_cound yaml test (#68583)

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

* Revert previous change to fix import issue.

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

* Remove unused imports in ArchiveTests

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

* Fix unit test for removal of x-pack aggregations. (#65)

This PR fixes the unit test which failed after removal of the x-pack aggregations #59.

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

* [PURIFY] Remove ProtocolUtils, TimeUtils, and XContentSource from HLRC (#64)

This commit removes the ProtocolUtils, TimeUtils, and XContentSource utility classes which is only used for xpack HLRC.

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

* [PURIFY] Remove x-pack feature flag from yaml test (#68)

This commit removes the xpack no_xpack feature flag from the yaml test suite.

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

* Remove testcase testSearchWithBasicLicensedQuery since basic license is no longer applicable (#74)

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

* Remove UnusedImports (#76)

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

* Bring back the REST specs for data streams. (#78)

Add back the REST specs for data streams which were moved to x-pack as part of the commit fe12217

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

* Remove unused imports after x-pack feature flag removed from yaml test (#81)

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

* [TEST] Fix Feature Flags in Test Framework and SortTemplates yaml failure (#82)

This commit adds parse logic to correctly parse feature flags in the test framework. It also fixes a test failure in cat.templates/Sort Templates yaml test.

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

* Run precommit and unit tests as part of github actions. (#84)

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

* Removing FreezeIndex related code since its x-pack counterpart is removed (#85)

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

* Only run pre-commit checks in GitHub actions. (#94)

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

* Fixing Bwc checks for 7.10.3 (#93)

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

* Temporary fix for license check path for debian packaging. (#97)

This currently unblocks the gradle check and subsequently will be removed by the meta issue #50

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

* Disable plugincli feature (#101)

Plugins CLI - disable installing official plugins by name.

Currently the plugin cli allows installation of a plugin by name in which case it downloads the plugin artifacts from the official elastic artifacts repository.

We will enable it once we have the new official artifacts download URL (Tracking Issue: #100)

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

* Support for continious integration with Jenkins (#96)

Adding jenkinsfile to describe the build / test / deployment process for this repository

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

* Remove any non oss from build, package, and distribution (#102)

This commit changes the building, packaging, and testing framework to only support OSS on different distributions.

Next steps:

completely remove -oss flag dependencies in package and build tests
move 6.x bwc testing to be an explicit option
remove any references to elastic.co download site (or replace with downloads from the OSS website)

Co-authored-by: Himanshu Setia <[email protected]>
Co-authored-by: Rabi Panda <[email protected]>
Co-authored-by: Himanshu Setia <[email protected]>
Co-authored-by: Sarat Vemulapalli <[email protected]>
Signed-off-by: Peter Nied <[email protected]>

* Remove x-pact from RESI API username and password (#117)

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

* Update signoff message (#121)

Signed-off-by: Harold Wang <[email protected]>
Signed-off-by: Peter Nied <[email protected]>

* Update CI workflow to work on new infra (#123)

* Update CI workflow to work on new infra

- Backward compatability tests are disabled during CI by default #113
- Added property to allow for disabling bwc tests
- Added agent label to use specific hardware https://www.jenkins.io/doc/book/pipeline/syntax/#agent

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

* Disable BWC checks. (#130)

As part of this PR, we are disabling the BWC checks. Once we have finalized the versions for the fork, we can re-enable it with right configurations.

Relates #105

Signed-off-by: Rabi Panda <[email protected]>
Signed-off-by: Peter Nied <[email protected]>

* Create CODE_OF_CONDUCT.md (#124)

Explicitly adding code of conduct from https://opendistro.github.io/for-elasticsearch/codeofconduct.html

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

* Add script to perform signoff check between commits (#152)

* Add script to perform signoff check between commits

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

* [Rename] server/src/main/java/org/apache (#162)

This commit refactors all instances of elasticsearch in
server/src/main/java/org/apache to opensearch.

Signed-off-by: Nicholas Knize <[email protected]>
Signed-off-by: Peter Nied <[email protected]>

* Revert "[Rename] server/src/main/java/org/apache (#162)"

This reverts commit c50e8c8
which went should have merged to the rename branch instead of
the main branch.

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

* Update CODE_OF_CONDUCT.md

replaced renameme

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

* Dummy commit to test the CI/CD workflow

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

* Update .gitignore

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

* Fixing transport deserialization with oss distribution (#218)

Signed-off-by: Sarat Vemulapalli <[email protected]>
Signed-off-by: Peter Nied <[email protected]>

* Update LICENSE.txt (#227)

updating to Apache 2.0 License from https://www.apache.org/licenses/LICENSE-2.0.txt

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

* Update NOTICE.TXT with OpenSearch copyright (#232)

This commit updates the NOTICE.txt file to include the OpenSearch copyright
notice.

Signed-off-by: Nicholas Knize <[email protected]>
Signed-off-by: Peter Nied <[email protected]>

* fixed reference to old repo (#333)

This commit fixes the url reference to bwc disabled message.

Signed-off-by: Kyle Davis <[email protected]>

Co-authored-by: nknize <[email protected]>
Co-authored-by: Huan Jiang <[email protected]>
Co-authored-by: Rabi Panda <[email protected]>
Co-authored-by: Rabi Panda <[email protected]>
Co-authored-by: nknize <[email protected]>
Co-authored-by: Sarat Vemulapalli <[email protected]>
Co-authored-by: Harold Wang <[email protected]>
Co-authored-by: Himanshu Setia <[email protected]>
Co-authored-by: Nhat Nguyen <[email protected]>
Co-authored-by: Jack Conradson <[email protected]>
Co-authored-by: Christoph Büscher <[email protected]>
Co-authored-by: Abbas Hussain <[email protected]>
Co-authored-by: Peter Nied <[email protected]>
Co-authored-by: Himanshu Setia <[email protected]>
Co-authored-by: Rabi Panda <[email protected]>
Co-authored-by: Peter Nied <[email protected]>
Co-authored-by: CEHENKLE <[email protected]>
Co-authored-by: Barani <[email protected]>
Co-authored-by: Peter Zhu <[email protected]>
Co-authored-by: Kyle J. Davis <[email protected]>
Poojita-Raj referenced this issue in Poojita-Raj/OpenSearch Nov 3, 2021
@Poojita-Raj
# This is a combination of 7 commits.
12b0cd0 
# This is the 1st commit message:

changes to shardsearchreq tests

Signed-off-by: Poojita Raj <[email protected]>

# The commit message #2 will be skipped:

# try to print
#
# Signed-off-by: Poojita Raj <[email protected]>

# The commit message #3 will be skipped:

# try
#
# Signed-off-by: Poojita Raj <[email protected]>

# The commit message #4 will be skipped:

# moved the class
#
# Signed-off-by: Poojita Raj <[email protected]>

# The commit message #5 will be skipped:

# try2
#
# Signed-off-by: Poojita Raj <[email protected]>

# The commit message #6 will be skipped:

# only update
#
# Signed-off-by: Poojita Raj <[email protected]>

# The commit message #7 will be skipped:

# add fix
#
# Signed-off-by: Poojita Raj <[email protected]>
@reta reta mentioned this issue Nov 9, 2021
Merged
14 tasks
@reta reta mentioned this issue Dec 7, 2021
Closed
@dreamer-89 dreamer-89 mentioned this issue Jan 17, 2022
Closed
@xinlamzn xinlamzn mentioned this issue Apr 22, 2022
Closed
@dreamer-89 dreamer-89 mentioned this issue Jun 8, 2022
Closed
@minalsha minalsha mentioned this issue Aug 30, 2022
Closed
@macohen macohen mentioned this issue Sep 28, 2022
Closed
@ankitkala ankitkala mentioned this issue Nov 24, 2022
Open
@pfirmstone pfirmstone mentioned this issue Feb 9, 2023
Open
1 task
@neetikasinghal neetikasinghal mentioned this issue Jul 17, 2023
Closed
@ashking94 ashking94 mentioned this issue Sep 6, 2023
Closed
@linuxpi linuxpi mentioned this issue Sep 8, 2023
Merged
@dennisoelkers dennisoelkers mentioned this issue Sep 15, 2023
Closed
@jed326 jed326 mentioned this issue Nov 10, 2023
Merged
7 tasks
@Arpit-Bandejiya Arpit-Bandejiya mentioned this issue Jan 10, 2024
Open
@chishui chishui mentioned this issue Feb 29, 2024
Closed
@linuxpi linuxpi mentioned this issue Mar 21, 2024
Open
@peteralfonsi peteralfonsi mentioned this issue May 3, 2024
Merged
7 tasks
mwilso3 added a commit to mwilso3/OpenSearch-fork that referenced this issue May 3, 2024
@mwilso3
PR#13484 Re-work opensearch-project#2
8c43d5d 
Rename licenses from jdk18on to jdk15to18 and 1.78 to 1.78.1.
ritty27 pushed a commit to ritty27/OpenSearch that referenced this issue May 12, 2024
@mitalawachat @imRishN
Remove xpack transform and watcher package (opensearch-project#2)
b4c9bdd 
* Remove transform package

* Remove watcher package

Signed-off-by: Mital Awachat <[email protected]>
@reta reta mentioned this issue Aug 21, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
>FORK Related to the fork process Meta Meta issue, not directly linked to a PR :sanitize Removing elastic specific artifacts :xpack-removal Related to removal of x-pack
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nknize @adnapibar