elastic / apm-server

https://www.elastic.co/guide/en/apm/guide/current/index.html
Other
1.22k stars 525 forks source link

fix: clear scroll after completing scroll requests (backport #14551) #14675

Closed mergify[bot] closed 6 days ago

mergify[bot] commented 1 week ago

Motivation/summary

clear the search context using a clear scroll request for sourcemap and agentcfg search scroll requests

Checklist

For functional changes, consider:

How to test these changes

Related issues

Related to https://github.com/elastic/apm-server/issues/14619


This is an automatic backport of pull request #14551 done by Mergify.

kruskall commented 1 week ago

@mergifyio queue

mergify[bot] commented 1 week ago

queue

🛑 Branch protection settings are not validated anymore

[Branch protection](https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/managing-protected-branches/about-protected-branches) is enabled and is preventing Mergify to merge the pull request. Mergify will merge when branch protection settings validate the pull request once again. (detail: Required status check "CLA" is pending.)
kruskall commented 1 week ago

@mergifyio refresh

mergify[bot] commented 1 week ago

refresh

✅ Pull request refreshed

v1v commented 1 week ago

@Mergifyio queue

mergify[bot] commented 1 week ago

queue

🛑 The pull request has been removed from the queue default

The merge conditions cannot be satisfied due to failing checks. You can take a look at `Queue: Embarked in merge queue` check runs for more details. In case of a failure due to a flaky test, you should first retrigger the CI. Then, re-embark the pull request into the merge queue by posting the comment `@mergifyio refresh` on the pull request.
v1v commented 1 week ago

@mergifyio refresh

mergify[bot] commented 1 week ago

refresh

✅ Pull request refreshed

v1v commented 1 week ago

@mergifyio refresh

mergify[bot] commented 1 week ago

refresh

✅ Pull request refreshed

v1v commented 1 week ago

@Mergifyio refresh

mergify[bot] commented 1 week ago

refresh

✅ Pull request refreshed

v1v commented 1 week ago

@Mergifyio queue

mergify[bot] commented 1 week ago

queue

🛑 The pull request has been removed from the queue default

The merge conditions cannot be satisfied due to failing checks. You can take a look at `Queue: Embarked in merge queue` check runs for more details. In case of a failure due to a flaky test, you should first retrigger the CI. Then, re-embark the pull request into the merge queue by posting the comment `@mergifyio refresh` on the pull request.
mergify[bot] commented 6 days ago

This pull request has not been merged yet. Could you please review and merge it @? 🙏

kruskall commented 6 days ago

@carsonip friendly ping for review

carsonip commented 6 days ago

@Mergifyio queue

mergify[bot] commented 6 days ago

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at *55e7503492787d507ce57363287ed9bbf5e83a95*