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.

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

beater: create listener synchronously in Reload #6054

Merged
merged 1 commit into from
Aug 26, 2021

Conversation

axw
Copy link
Member

@axw axw commented Aug 26, 2021

Motivation/summary

Create socket listener synchronously in the beater Reload method, to ensure there is a new active listener before the existing listener is closed.

Checklist

- [ ] Update CHANGELOG.asciidoc
- [ ] Documentation has been updated

How to test these changes

See #6050

Related issues

Fixes #6050

@axw axw added v7.15.0 backport-7.x backport-7.15 Automated backport with mergify to the 7.15 branch labels Aug 26, 2021
@apmmachine
Copy link
Contributor

apmmachine commented Aug 26, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-08-26T06:16:46.773+0000

  • Duration: 39 min 36 sec

  • Commit: bd2ec9c

Test stats 🧪

Test Results
Failed 0
Passed 5940
Skipped 14
Total 5954

Trends 🧪

Image of Build Times

Image of Tests

@axw axw force-pushed the beater-listen-sync branch from c3622fa to bd2ec9c Compare August 26, 2021 06:16
@axw axw marked this pull request as ready for review August 26, 2021 06:58
@axw axw requested a review from a team August 26, 2021 06:59
Copy link
Contributor

@stuartnelson3 stuartnelson3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks great!

@axw axw merged commit 5453de4 into elastic:master Aug 26, 2021
@axw axw deleted the beater-listen-sync branch August 26, 2021 07:58
mergify bot pushed a commit that referenced this pull request Aug 26, 2021
mergify bot pushed a commit that referenced this pull request Aug 26, 2021
axw added a commit that referenced this pull request Aug 26, 2021
(cherry picked from commit 5453de4)

Co-authored-by: Andrew Wilkins <[email protected]>
axw added a commit that referenced this pull request Aug 27, 2021
(cherry picked from commit 5453de4)

Co-authored-by: Andrew Wilkins <[email protected]>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-7.15 Automated backport with mergify to the 7.15 branch v7.15.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

beater: when reloading config, new listener may not be started by the time the old one is stopped
3 participants