Deprecate Behavioral Analytics CRUD apis#122960
Conversation
kderusso
added
auto-backport
|
Hi @kderusso, I've created a changelog YAML for you. Note that since this PR is labelled |
|
Warning It looks like this PR modifies one or more |
| ---- | ||
| DELETE _application/analytics/my_analytics_collection/ | ||
| ---- | ||
| // TEST[skip:Behavioral Analytics APIs emit deprecation warnings and will not be updated] |
There was a problem hiding this comment.
There appears to be a bug in our test framework as emitting warnings during test setup can't be handled. Since we're not going to be updating these APIs my solution was to skip the tests.
There was a problem hiding this comment.
Whitespace/auto-formatting
|
I have created a followup request to update Markdown docs: elastic/docs-content#541 |
|
Warning It looks like this PR modifies one or more |
|
Pinging @elastic/search-eng (Team:SearchOrg) |
|
Pinging @elastic/search-relevance (Team:Search - Relevance) |
|
Warning It looks like this PR modifies one or more |
|
Warning It looks like this PR modifies one or more |
...eference/search/search-your-data/behavioral-analytics/behavioral-analytics-overview.asciidoc
Outdated
Show resolved
Hide resolved
Co-authored-by: Liam Thompson <32779855+leemthompo@users.noreply.github.com>
|
Warning It looks like this PR modifies one or more |
…/deprecate-behavioral-analytics-crud-apis
|
Warning It looks like this PR modifies one or more |
…/deprecate-behavioral-analytics-crud-apis
|
Warning It looks like this PR modifies one or more |
ioanatia
left a comment
ioanatia
left a comment
There was a problem hiding this comment.
Looks good - I am not sure what to do about the docs migration warning.
@leemthompo we should be good to merge, can you please check the PR?
there is another thing we should add - the UpdateForV10 annotation.
See example:
but I am okay at adding this in a follow up, there might be other things we need to annotate in the ent-search plugin.
💚 Backport successful
|
* Deprecate Behavioral Analytics CRUD APIs * Add allowed warning for REST Compatibility tests * Update docs/changelog/122960.yaml * Update changelog * Update docs to add deprecation flags and fix failing tests * Update changelog * Update changelog again * Update docs formatting Co-authored-by: Liam Thompson <32779855+leemthompo@users.noreply.github.com> * Skip asciidoc test --------- Co-authored-by: Efe Gürkan YALAMAN <efeyalaman@gmail.com> Co-authored-by: Liam Thompson <32779855+leemthompo@users.noreply.github.com> Co-authored-by: Efe Gürkan YALAMAN <efeguerkan.yalaman@elastic.co>
* Deprecate Behavioral Analytics CRUD APIs * Add allowed warning for REST Compatibility tests * Update docs/changelog/122960.yaml * Update changelog * Update docs to add deprecation flags and fix failing tests * Update changelog * Update changelog again * Update docs formatting * Skip asciidoc test --------- Co-authored-by: Efe Gürkan YALAMAN <efeyalaman@gmail.com> Co-authored-by: Liam Thompson <32779855+leemthompo@users.noreply.github.com> Co-authored-by: Efe Gürkan YALAMAN <efeguerkan.yalaman@elastic.co>
5 participants
Deprecates the Behavioral Analytics CRUD APIs and issues the following warning headers when the APIs are used: "Behavioral Analytics is deprecated and will be removed in a future release."