Skip to content

FileTarget - Failing to delete old archive files, should not stop logging#1867

Merged
304NotModified merged 1 commit into
NLog:masterfrom
snakefoot:FileTargetMaxArchiveFiles
Dec 21, 2016
Merged

FileTarget - Failing to delete old archive files, should not stop logging#1867
304NotModified merged 1 commit into
NLog:masterfrom
snakefoot:FileTargetMaxArchiveFiles

Conversation

@snakefoot

@snakefoot snakefoot commented Dec 18, 2016

Copy link
Copy Markdown
Contributor

See also #1866

Have also changed the compression logic code from waiting forever to "only" 12 secs.

This change is Reviewable

@snakefoot snakefoot force-pushed the FileTargetMaxArchiveFiles branch from ba99fb8 to 6472e7f Compare December 18, 2016 23:51
@codecov-io

codecov-io commented Dec 18, 2016

Copy link
Copy Markdown

Current coverage is 82% (diff: 53%)

Merging #1867 into master will decrease coverage by <1%

@@             master      #1867   diff @@
==========================================
  Files           276        276          
  Lines         18503      18519    +16   
  Methods        2859       2860     +1   
  Messages          0          0          
  Branches       2137       2143     +6   
==========================================
  Hits          15094      15094          
- Misses         2917       2928    +11   
- Partials        492        497     +5   

Sunburst

Powered by Codecov. Last update cf507d5...0b4775e

@snakefoot snakefoot force-pushed the FileTargetMaxArchiveFiles branch from 6472e7f to 98ce8e4 Compare December 19, 2016 08:41
@snakefoot snakefoot force-pushed the FileTargetMaxArchiveFiles branch from 98ce8e4 to 0b4775e Compare December 19, 2016 22:37
@304NotModified

Copy link
Copy Markdown
Member

Thanks!

@304NotModified 304NotModified added file-target bug Bug report / Bug fix labels Dec 21, 2016
@304NotModified 304NotModified added this to the 4.4.1 milestone Dec 21, 2016
@304NotModified 304NotModified merged commit a7293bb into NLog:master Dec 21, 2016
@snakefoot snakefoot deleted the FileTargetMaxArchiveFiles branch October 10, 2017 20:39
@304NotModified 304NotModified added the file-archiving Issues with archiving with the file target label Aug 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bug report / Bug fix file-archiving Issues with archiving with the file target file-target

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants