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

Remove the "xpack" namespace from the REST API #38623

Closed
jpountz opened this issue Feb 8, 2019 · 2 comments · Fixed by #38625
Closed

Remove the "xpack" namespace from the REST API #38623

jpountz opened this issue Feb 8, 2019 · 2 comments · Fixed by #38625
Labels
blocker >bug :Core/Infra/REST API REST infrastructure and utilities v7.0.0-rc1

Comments

@jpountz
Copy link
Contributor

jpountz commented Feb 8, 2019

@Mpdreamz noticed that some APIs still include the "xpack" namespace in their name. We should remove it. See https://github.com/elastic/elasticsearch/tree/7.x/x-pack/plugin/src/test/resources/rest-api-spec/api.

Relates #35958
@jpountz jpountz added >bug blocker :Core/Infra/REST API REST infrastructure and utilities v7.0.0 labels Feb 8, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@Mpdreamz
Copy link
Member

Mpdreamz commented Feb 8, 2019

Opened #38625 to address this.

Mpdreamz added a commit to Mpdreamz/elasticsearch that referenced this issue Feb 15, 2019
@Mpdreamz
Fix elastic#38623 remove xpack namespace REST API
7a9f158 
Except for xpack.usage and xpack.info API's, this moves the last remaining API's out of the xpack namespace
This was referenced Feb 18, 2019
Merged
Merged
Mpdreamz added a commit to Mpdreamz/elasticsearch that referenced this issue Feb 18, 2019
@Mpdreamz
Fix elastic#38623 remove xpack namespace REST API (elastic#38625)
e8af4e2 
* Fix elastic#38623 remove xpack namespace REST API

Except for xpack.usage and xpack.info API's, this moves the last remaining API's out of the xpack namespace

* rename xpack api's inside inside the files as well

* updated yaml tests references to xpack namespaces api's

* update callsApi calls in the IT subclasses

* make sure docs testing does not use xpack namespaced api's

* fix leftover xpack namespaced method names in docs/build.gradle

* found another leftover reference

(cherry picked from commit ccb5d93)
Mpdreamz added a commit to Mpdreamz/elasticsearch that referenced this issue Feb 18, 2019
@Mpdreamz
Fix elastic#38623 remove xpack namespace REST API (elastic#38625)
fa98cad 
* Fix elastic#38623 remove xpack namespace REST API

Except for xpack.usage and xpack.info API's, this moves the last remaining API's out of the xpack namespace

* rename xpack api's inside inside the files as well

* updated yaml tests references to xpack namespaces api's

* update callsApi calls in the IT subclasses

* make sure docs testing does not use xpack namespaced api's

* fix leftover xpack namespaced method names in docs/build.gradle

* found another leftover reference

(cherry picked from commit ccb5d93)
Mpdreamz added a commit that referenced this issue Feb 18, 2019
@Mpdreamz
Fix #38623 remove xpack namespace REST API (#38625) (#39037)
8a716ac 
* Fix #38623 remove xpack namespace REST API

Except for xpack.usage and xpack.info API's, this moves the last remaining API's out of the xpack namespace

* rename xpack api's inside inside the files as well

* updated yaml tests references to xpack namespaces api's

* update callsApi calls in the IT subclasses

* make sure docs testing does not use xpack namespaced api's

* fix leftover xpack namespaced method names in docs/build.gradle

* found another leftover reference

(cherry picked from commit ccb5d93)
Mpdreamz added a commit that referenced this issue Feb 18, 2019
@Mpdreamz
Fix #38623 remove xpack namespace REST API (#38625) (#39036)
9b4d965 
* Fix #38623 remove xpack namespace REST API

Except for xpack.usage and xpack.info API's, this moves the last remaining API's out of the xpack namespace

* rename xpack api's inside inside the files as well

* updated yaml tests references to xpack namespaces api's

* update callsApi calls in the IT subclasses

* make sure docs testing does not use xpack namespaced api's

* fix leftover xpack namespaced method names in docs/build.gradle

* found another leftover reference

(cherry picked from commit ccb5d93)
jasontedor added a commit to jasontedor/elasticsearch that referenced this issue Feb 18, 2019
@jasontedor
Merge remote-tracking branch 'elastic/master' into retention-lease-st…
8433480 
…ate-file

* elastic/master:
  Remove tests and branches that will never execute (elastic#38772)
  also check ccr stats api return empty response in ensureNoCcrTasks()
  Add overlapping, before, after filters to intervals query (elastic#38999)
  Mute test elastic#38949
  Add remote recovery to ShardFollowTaskReplicationTests (elastic#39007)
  [ML] More advanced post-test cleanup of ML indices (elastic#39049)
  wait for shard to be allocated before executing a resume follow api
  Update track-total-hits.asciidoc
  Force kill testcluster nodes (elastic#37353)
  Make pullFixture a task dependency of resolveAllDependencies (elastic#38956)
  set minimum supported version (elastic#39043)
  Enforce Completion Context Limit (elastic#38675)
  Mute test
  Don't close caches while there might still be in-flight requests. (elastic#38958)
  Fix elastic#38623 remove xpack namespace REST API (elastic#38625)
  Add data frame feature (elastic#38934)
  Test bi-directional index following during a rolling upgrade. (elastic#38962)
  Generate mvn pom for ssl-config library (elastic#39019)
  Mute testRetentionLeaseIsRenewedDuringRecovery
@Mpdreamz Mpdreamz mentioned this issue Apr 9, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
blocker >bug :Core/Infra/REST API REST infrastructure and utilities v7.0.0-rc1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #38623 remove xpack namespace REST API Mpdreamz/elasticsearch
4 participants
@Mpdreamz @jpountz @michaelbaamonde @elasticmachine