New: User configurable minimum free disk space

Co-Authored-By: Mark McDowall <markus101@users.noreply.github.com>

frontend/src/Settings/MediaManagement/MediaManagement.js

@@ -147,6 +147,23 @@ class MediaManagement extends Component {
                           </FormGroup>
                       }
 
+                      <FormGroup
+                        advancedSettings={advancedSettings}
+                        isAdvanced={true}
+                        size={sizes.MEDIUM}
+                      >
+                        <FormLabel>Minimum Free Space</FormLabel>
+
+                        <FormInputGroup
+                          type={inputTypes.NUMBER}
+                          unit='MB'
+                          name="minimumFreeSpaceWhenImporting"
+                          helpText="Prevent import if it would leave less than this amount of disk space available"
+                          onChange={onInputChange}
+                          {...settings.minimumFreeSpaceWhenImporting}
+                        />
+                      </FormGroup>
+
                       <FormGroup
                         advancedSettings={advancedSettings}
                         isAdvanced={true}

src/Lidarr.Api.V1/Config/MediaManagementConfigModule.cs

@@ -13,6 +13,7 @@ namespace Lidarr.Api.V1.Config
             SharedValidator.RuleFor(c => c.FileChmod).NotEmpty();
             SharedValidator.RuleFor(c => c.FolderChmod).NotEmpty();
             SharedValidator.RuleFor(c => c.RecycleBin).IsValidPath().SetValidator(pathExistsValidator).When(c => !string.IsNullOrWhiteSpace(c.RecycleBin));
+            SharedValidator.RuleFor(c => c.MinimumFreeSpaceWhenImporting).GreaterThanOrEqualTo(100);
         }
 
         protected override MediaManagementConfigResource ToResource(IConfigService model)

src/Lidarr.Api.V1/Config/MediaManagementConfigResource.cs

@@ -24,6 +24,7 @@ namespace Lidarr.Api.V1.Config
         public string ChownGroup { get; set; }
 
         public bool SkipFreeSpaceCheckWhenImporting { get; set; }
+        public int MinimumFreeSpaceWhenImporting { get; set; }
         public bool CopyUsingHardlinks { get; set; }
         public bool ImportExtraFiles { get; set; }
         public string ExtraFileExtensions { get; set; }
@@ -52,6 +53,7 @@ namespace Lidarr.Api.V1.Config
                 ChownGroup = model.ChownGroup,
 
                 SkipFreeSpaceCheckWhenImporting = model.SkipFreeSpaceCheckWhenImporting,
+                MinimumFreeSpaceWhenImporting = model.MinimumFreeSpaceWhenImporting,
                 CopyUsingHardlinks = model.CopyUsingHardlinks,
                 ImportExtraFiles = model.ImportExtraFiles,
                 ExtraFileExtensions = model.ExtraFileExtensions,

src/NzbDrone.Core.Test/MediaFiles/TrackImport/Specifications/FreeSpaceSpecificationFixture.cs

@@ -66,8 +66,12 @@ namespace NzbDrone.Core.Test.MediaFiles.TrackImport.Specifications
         }
 
         [Test]
-        public void should_reject_when_there_isnt_enough_space_for_file_plus_100mb_padding()
+        public void should_reject_when_there_isnt_enough_space_for_file_plus_min_free_space()
         {
+            Mocker.GetMock<IConfigService>()
+                .Setup(s => s.MinimumFreeSpaceWhenImporting)
+                .Returns(100);
+
             GivenFileSize(100.Megabytes());
             GivenFreeSpace(150.Megabytes());
 

src/NzbDrone.Core/Configuration/ConfigService.cs

@@ -196,6 +196,13 @@ namespace NzbDrone.Core.Configuration
             set { SetValue("SkipFreeSpaceCheckWhenImporting", value); }
         }
 
+        public int MinimumFreeSpaceWhenImporting
+        {
+            get { return GetValueInt("MinimumFreeSpaceWhenImporting", 100); }
+
+            set { SetValue("MinimumFreeSpaceWhenImporting", value); }
+        }
+
         public bool CopyUsingHardlinks
         {
             get { return GetValueBoolean("CopyUsingHardlinks", true); }

src/NzbDrone.Core/Configuration/IConfigService.cs

@@ -32,6 +32,7 @@ namespace NzbDrone.Core.Configuration
         bool DeleteEmptyFolders { get; set; }
         FileDateType FileDate { get; set; }
         bool SkipFreeSpaceCheckWhenImporting { get; set; }
+        int MinimumFreeSpaceWhenImporting { get; set; }
         bool CopyUsingHardlinks { get; set; }
         bool ImportExtraFiles { get; set; }
         string ExtraFileExtensions { get; set; }

src/NzbDrone.Core/MediaFiles/TrackImport/Specifications/FreeSpaceSpecification.cs

@@ -46,7 +46,7 @@ namespace NzbDrone.Core.MediaFiles.TrackImport.Specifications
                     return Decision.Accept();
                 }
 
-                if (freeSpace < localTrack.Size + 100.Megabytes())
+                if (freeSpace < localTrack.Size + _configService.MinimumFreeSpaceWhenImporting.Megabytes())
                 {
                     _logger.Warn("Not enough free space ({0}) to import: {1} ({2})", freeSpace, localTrack, localTrack.Size);
                     return Decision.Reject("Not enough free space");