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

Add specification for post event to an analytics collection #3363

Merged

lcawl merged 7 commits into main from post-analytics-collection

Jan 4, 2025

Contributor

lcawl commented Dec 18, 2024 •

edited

Loading

Relates to #3377, #2482

This PR adds a specification so that we can retire https://www.elastic.co/guide/en/elasticsearch/reference/master/post-analytics-collection-event.html

lcawl added specification backport 8.x backport 8.17 labels

lcawl requested a review from a team

December 18, 2024 22:49

Contributor

github-actions bot commented Dec 18, 2024

Following you can find the validation results for the APIs you have changed.

API	Status	Request	Response
`search_application.post_behavioral_analytics_event`	🔴	0/18	18/18

You can validate these APIs yourself by using the make validate target.

lcawl changed the title ~~Add specification for post even to an analytics collection~~ Add specification for post event to an analytics collection

Contributor

github-actions bot commented Dec 19, 2024

Following you can find the validation results for the APIs you have changed.

API	Status	Request	Response
`search_application.post_behavioral_analytics_event`	🔴	0/18	18/18

You can validate these APIs yourself by using the make validate target.

l-trotta self-assigned this

l-trotta reviewed

View reviewed changes

...on/search_application/post_behavioral_analytics_event/BehavioralAnalyticsEventPostRequest.ts Show resolved Hide resolved

l-trotta reviewed

View reviewed changes

...n/search_application/post_behavioral_analytics_event/BehavioralAnalyticsEventPostResponse.ts Outdated Show resolved Hide resolved

lcawl mentioned this pull request

[DOCS] Add minimal specifications for all Elasticsearch API docs #3377

Closed

26 tasks

Contributor

github-actions bot commented Dec 19, 2024

Following you can find the validation results for the APIs you have changed.

API	Status	Request	Response
`search_application.post_behavioral_analytics_event`	🔴	0/18	18/18

You can validate these APIs yourself by using the make validate target.

Contributor

github-actions bot commented Dec 19, 2024

Following you can find the validation results for the APIs you have changed.

API	Status	Request	Response
`search_application.post_behavioral_analytics_event`	🟢	18/18	18/18

You can validate these APIs yourself by using the make validate target.

1 similar comment

Contributor

github-actions bot commented Jan 3, 2025

Following you can find the validation results for the APIs you have changed.

API	Status	Request	Response
`search_application.post_behavioral_analytics_event`	🟢	18/18	18/18

You can validate these APIs yourself by using the make validate target.

lcawl mentioned this pull request

[DOCS] Add examples to API endpoints #2482

Closed

65 tasks

Contributor

github-actions bot commented Jan 3, 2025

Following you can find the validation results for the APIs you have changed.

API	Status	Request	Response
`search_application.post_behavioral_analytics_event`	🟢	18/18	18/18

You can validate these APIs yourself by using the make validate target.

1 similar comment

Contributor

github-actions bot commented Jan 3, 2025

Following you can find the validation results for the APIs you have changed.

API	Status	Request	Response
`search_application.post_behavioral_analytics_event`	🟢	18/18	18/18

You can validate these APIs yourself by using the make validate target.

pquentin approved these changes

View reviewed changes

lcawl and others added 7 commits

January 4, 2025 00:04


          Add specification for post even to an analytics collection

f1b8762


          Add doc_id

a6f1926


          Update specification/search_application/post_behavioral_analytics_eve…

4ca526a

…nt/BehavioralAnalyticsEventPostResponse.ts

Co-authored-by: Laura Trotta <[email protected]>


          Update specification/search_application/post_behavioral_analytics_eve…

71ce3e6

…nt/BehavioralAnalyticsEventPostRequest.ts

Co-authored-by: Laura Trotta <[email protected]>


          Fix style

dbcbe5a


          Add post analytics collection event example

bcc563f


          Fix compilation

fc0037b

lcawl force-pushed the post-analytics-collection branch from bc083db to fc0037b Compare

January 4, 2025 08:07

Contributor

github-actions bot commented Jan 4, 2025

Following you can find the validation results for the APIs you have changed.

API	Status	Request	Response
`search_application.post_behavioral_analytics_event`	🟢	18/18	18/18

You can validate these APIs yourself by using the make validate target.

lcawl merged commit 2f57f6b into main

7 checks passed

lcawl deleted the post-analytics-collection branch

January 4, 2025 08:13

github-actions bot pushed a commit that referenced this pull request


          Add specification for post event to an analytics collection (#3363)

780b290

Co-authored-by: Laura Trotta <[email protected]>
Co-authored-by: Quentin Pradet <[email protected]>
(cherry picked from commit 2f57f6b)

Contributor

github-actions bot commented Jan 4, 2025

The backport to 8.17 failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-8.17 8.17
# Navigate to the new working tree
cd .worktrees/backport-8.17
# Create a new branch
git switch --create backport-3363-to-8.17
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 2f57f6bbe10648cf0b45a8b7ed6ea37cbecf1494
# Push it to GitHub
git push --set-upstream origin backport-3363-to-8.17
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-8.17

Then, create a pull request where the base branch is 8.17 and the compare/head branch is backport-3363-to-8.17.

github-actions bot mentioned this pull request

[Backport 8.x] Add specification for post event to an analytics collection #3431

Merged

lcawl pushed a commit that referenced this pull request


          Add specification for post event to an analytics collection (#3363) (#…

25bd629

…3431)

Co-authored-by: Laura Trotta <[email protected]>
Co-authored-by: Quentin Pradet <[email protected]>
(cherry picked from commit 2f57f6b)

lcawl added a commit that referenced this pull request


          Add specification for post event to an analytics collection (#3363)

3a0a7f3

Co-authored-by: Laura Trotta <[email protected]>
Co-authored-by: Quentin Pradet <[email protected]>
(cherry picked from commit 2f57f6b)

lcawl mentioned this pull request

[8.17] Add specification for post event to an analytics collection (#3363) #3432

Merged

Contributor Author

lcawl commented Jan 4, 2025

💚 All backports created successfully

Status	Branch	Result
✅	8.17

Questions ?

Please refer to the Backport tool documentation

pquentin pushed a commit that referenced this pull request


          Add specification for post event to an analytics collection (#3363) (#…

4fd348f

…3432)

Co-authored-by: Laura Trotta <[email protected]>
Co-authored-by: Quentin Pradet <[email protected]>
(cherry picked from commit 2f57f6b)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 8.x backport 8.17 specification