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

[ReleasePrep][2018.08.20]RI of dev into master #6333

Merged
merged 17 commits into from
Aug 20, 2018
Merged

[ReleasePrep][2018.08.20]RI of dev into master #6333

merged 17 commits into from
Aug 20, 2018

Conversation

chenriksson
Copy link
Member

No description provided.

Scott Bommarito and others added 17 commits August 8, 2018 09:38
* Revert "Don't throw when SmtpUri is not set (#6278)"

This reverts commit e60e75f.

* Revert "Fix logging exceptions from background thread (#6273)"

This reverts commit d1610bb.

* Revert "Improve email sending (#6154)"

This reverts commit 2616a99.
* Remove PagerDuty config + appveyor integration

* Updated deployment docs

* clean images

* Fix text

* PR comment
* works

* improve log

* PR comments

* Fix comments

* bug fix
* remove current text
* remove bold
* add background color
Retry sending emails on SmtpExceptions.
Record telemetry on each attempt to send an email.
Send emails in the background, to avoid making the user wait for retries.
This sets the `CacheControl` property on all packages blobs to 2 minutes. 

Fixes #6285
@chenriksson chenriksson merged commit 1b98757 into master Aug 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants