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

Fix race condition in telemetry (#18734) #18777

Merged
merged 2 commits into from Oct 20, 2021

Conversation

larkox
Copy link
Contributor

@larkox larkox commented Oct 20, 2021

Cherry Pick 5928e4f

* Fix race condition in telemetry

* Fix tests
@larkox larkox requested a review from amyblais October 20, 2021 17:31
@mm-cloud-bot
Copy link

@larkox: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

I understand the commands that are listed here

@amyblais
Copy link
Member

/release-note-none

@mm-cloud-bot mm-cloud-bot added release-note-none Denotes a PR that doesn't merit a release note. and removed do-not-merge/release-note-label-needed labels Oct 20, 2021
@amyblais amyblais added 4: Reviews Complete All reviewers have approved the pull request AutoMerge Used by Mattermod to merge PR automatically labels Oct 20, 2021
@mattermod
Copy link
Contributor

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

@amyblais amyblais added Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation labels Oct 20, 2021
@mattermod
Copy link
Contributor

Trying to auto merge this PR.

@mattermod mattermod merged commit 1c430f7 into mattermost:release-5.37 Oct 20, 2021
@mattermod
Copy link
Contributor

Pull Request successfully merged
SHA: 1c430f7

@mattermod mattermod removed the AutoMerge Used by Mattermod to merge PR automatically label Oct 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request Changelog/Not Needed Does not require a changelog entry Docs/Not Needed Does not require documentation release-note-none Denotes a PR that doesn't merit a release note.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants