12345678910111213141516171819202122232425262728293031323334353637383940414243444546474849505152535455565758596061626364656667686970717273747576777879808182838485868788899091929394959697989910010110210310410510610710810911011111211311411511611711811912012112212312412512612712812913013113213313413513613713813914014114214314414514614714814915015115215315415515615715815916016116216316416516616716816917017117217317417517617717817918018118218318418518618718818919019119219319419519619719819920020120220320420520620720820921021121221321421521621721821922022122222322422522622722822923023123223323423523623723823924024124224324424524624724824925025125225325425525625725825926026126226326426526626726826927027127227327427527627727827928028128228328428528628728828929029129229329429529629729829930030130230330430530630730830931031131231331431531631731831932032132232332432532632732832933033133233333433533633733833934034134234334434534634734834935035135235335435535635735835936036136236336436536636736836937037137237337437537637737837938038138238338438538638738838939039139239339439539639739839940040140240340440540640740840941041141241341441541641741841942042142242342442542642742842943043143243343443543643743843944044144244344444544644744844945045145245345445545645745845946046146246346446546646746846947047147247347447547647747847948048148248348448548648748848949049149249349449549649749849950050150250350450550650750850951051151251351451551651751851952052152252352452552652752852953053153253353453553653753853954054154254354454554654754854955055155255355455555655755855956056156256356456556656756856957057157257357457557657757857958058158258358458558658758858959059159259359459559659759859960060160260360460560660760860961061161261361461561661761861962062162262362462562662762862963063163263363463563663763863964064164264364464564664764864965065165265365465565665765865966066166266366466566666766866967067167267367467567667767867968068168268368468568668768868969069169269369469569669769869970070170270370470570670770870971071171271371471571671771871972072172272372472572672772872973073173273373473573673773873974074174274374474574674774874975075175275375475575675775875976076176276376476576676776876977077177277377477577677777877978078178278378478578678778878979079179279379479579679779879980080180280380480580680780880981081181281381481581681781881982082182282382482582682782882983083183283383483583683783883984084184284384484584684784884985085185285385485585685785885986086186286386486586686786886987087187287387487587687787887988088188288388488588688788888989089189289389489589689789889990090190290390490590690790890991091191291391491591691791891992092192292392492592692792892993093193293393493593693793893994094194294394494594694794894995095195295395495595695795895996096196296396496596696796896997097197297397497597697797897998098198298398498598698798898999099199299399499599699799899910001001100210031004100510061007100810091010101110121013101410151016101710181019102010211022102310241025102610271028102910301031103210331034 |
- import pytest
- from flexmock import flexmock
- from borgmatic.borg.pattern import Pattern, Pattern_source, Pattern_style, Pattern_type
- from borgmatic.hooks.data_source import btrfs as module
- def test_get_subvolume_mount_points_parses_findmnt_output():
- flexmock(module.borgmatic.execute).should_receive(
- 'execute_command_and_capture_output'
- ).and_return(
- '''{
- "filesystems": [
- {
- "target": "/mnt0",
- "source": "/dev/loop0",
- "fstype": "btrfs",
- "options": "rw,relatime,ssd,space_cache=v2,subvolid=5,subvol=/"
- },
- {
- "target": "/mnt1",
- "source": "/dev/loop0",
- "fstype": "btrfs",
- "options": "rw,relatime,ssd,space_cache=v2,subvolid=5,subvol=/"
- }
- ]
- }
- '''
- )
- assert module.get_subvolume_mount_points('findmnt') == ('/mnt0', '/mnt1')
- def test_get_subvolume_mount_points_with_invalid_findmnt_json_errors():
- flexmock(module.borgmatic.execute).should_receive(
- 'execute_command_and_capture_output'
- ).and_return('{')
- with pytest.raises(ValueError):
- module.get_subvolume_mount_points('findmnt')
- def test_get_subvolume_mount_points_with_findmnt_json_missing_filesystems_errors():
- flexmock(module.borgmatic.execute).should_receive(
- 'execute_command_and_capture_output'
- ).and_return('{"wtf": "something is wrong here"}')
- with pytest.raises(ValueError):
- module.get_subvolume_mount_points('findmnt')
- def test_get_subvolume_property_with_invalid_btrfs_output_errors():
- flexmock(module.borgmatic.execute).should_receive(
- 'execute_command_and_capture_output'
- ).and_return('invalid')
- with pytest.raises(ValueError):
- module.get_subvolume_property('btrfs', '/foo', 'ro')
- def test_get_subvolume_property_with_true_output_returns_true_bool():
- flexmock(module.borgmatic.execute).should_receive(
- 'execute_command_and_capture_output'
- ).and_return('ro=true')
- assert module.get_subvolume_property('btrfs', '/foo', 'ro') is True
- def test_get_subvolume_property_with_false_output_returns_false_bool():
- flexmock(module.borgmatic.execute).should_receive(
- 'execute_command_and_capture_output'
- ).and_return('ro=false')
- assert module.get_subvolume_property('btrfs', '/foo', 'ro') is False
- def test_get_subvolume_property_passes_through_general_value():
- flexmock(module.borgmatic.execute).should_receive(
- 'execute_command_and_capture_output'
- ).and_return('thing=value')
- assert module.get_subvolume_property('btrfs', '/foo', 'thing') == 'value'
- def test_omit_read_only_subvolume_mount_points_filters_out_read_only():
- flexmock(module).should_receive('get_subvolume_property').with_args(
- 'btrfs', '/foo', 'ro'
- ).and_return(False)
- flexmock(module).should_receive('get_subvolume_property').with_args(
- 'btrfs', '/bar', 'ro'
- ).and_return(True)
- flexmock(module).should_receive('get_subvolume_property').with_args(
- 'btrfs', '/baz', 'ro'
- ).and_return(False)
- assert module.omit_read_only_subvolume_mount_points('btrfs', ('/foo', '/bar', '/baz')) == (
- '/foo',
- '/baz',
- )
- def test_get_subvolumes_collects_subvolumes_matching_patterns():
- flexmock(module).should_receive('get_subvolume_mount_points').and_return(('/mnt1', '/mnt2'))
- flexmock(module).should_receive('omit_read_only_subvolume_mount_points').and_return(
- ('/mnt1', '/mnt2')
- )
- contained_pattern = Pattern(
- '/mnt1',
- type=Pattern_type.ROOT,
- source=Pattern_source.CONFIG,
- )
- flexmock(module.borgmatic.hooks.data_source.snapshot).should_receive(
- 'get_contained_patterns'
- ).with_args('/mnt1', object).and_return((contained_pattern,))
- flexmock(module.borgmatic.hooks.data_source.snapshot).should_receive(
- 'get_contained_patterns'
- ).with_args('/mnt2', object).and_return(())
- assert module.get_subvolumes(
- 'btrfs',
- 'findmnt',
- patterns=[
- Pattern('/mnt1'),
- Pattern('/mnt3'),
- ],
- ) == (module.Subvolume('/mnt1', contained_patterns=(contained_pattern,)),)
- def test_get_subvolumes_skips_non_root_patterns():
- flexmock(module).should_receive('get_subvolume_mount_points').and_return(('/mnt1', '/mnt2'))
- flexmock(module).should_receive('omit_read_only_subvolume_mount_points').and_return(
- ('/mnt1', '/mnt2')
- )
- flexmock(module.borgmatic.hooks.data_source.snapshot).should_receive(
- 'get_contained_patterns'
- ).with_args('/mnt1', object).and_return(
- (
- Pattern(
- '/mnt1',
- type=Pattern_type.EXCLUDE,
- source=Pattern_source.CONFIG,
- ),
- )
- )
- flexmock(module.borgmatic.hooks.data_source.snapshot).should_receive(
- 'get_contained_patterns'
- ).with_args('/mnt2', object).and_return(())
- assert (
- module.get_subvolumes(
- 'btrfs',
- 'findmnt',
- patterns=[
- Pattern('/mnt1'),
- Pattern('/mnt3'),
- ],
- )
- == ()
- )
- def test_get_subvolumes_skips_non_config_patterns():
- flexmock(module).should_receive('get_subvolume_mount_points').and_return(('/mnt1', '/mnt2'))
- flexmock(module).should_receive('omit_read_only_subvolume_mount_points').and_return(
- ('/mnt1', '/mnt2')
- )
- flexmock(module.borgmatic.hooks.data_source.snapshot).should_receive(
- 'get_contained_patterns'
- ).with_args('/mnt1', object).and_return(
- (
- Pattern(
- '/mnt1',
- type=Pattern_type.ROOT,
- source=Pattern_source.HOOK,
- ),
- )
- )
- flexmock(module.borgmatic.hooks.data_source.snapshot).should_receive(
- 'get_contained_patterns'
- ).with_args('/mnt2', object).and_return(())
- assert (
- module.get_subvolumes(
- 'btrfs',
- 'findmnt',
- patterns=[
- Pattern('/mnt1'),
- Pattern('/mnt3'),
- ],
- )
- == ()
- )
- def test_get_subvolumes_without_patterns_collects_all_subvolumes():
- flexmock(module).should_receive('get_subvolume_mount_points').and_return(('/mnt1', '/mnt2'))
- flexmock(module).should_receive('omit_read_only_subvolume_mount_points').and_return(
- ('/mnt1', '/mnt2')
- )
- flexmock(module.borgmatic.hooks.data_source.snapshot).should_receive(
- 'get_contained_patterns'
- ).with_args('/mnt1', object).and_return((Pattern('/mnt1'),))
- flexmock(module.borgmatic.hooks.data_source.snapshot).should_receive(
- 'get_contained_patterns'
- ).with_args('/mnt2', object).and_return((Pattern('/mnt2'),))
- assert module.get_subvolumes('btrfs', 'findmnt') == (
- module.Subvolume('/mnt1', contained_patterns=(Pattern('/mnt1'),)),
- module.Subvolume('/mnt2', contained_patterns=(Pattern('/mnt2'),)),
- )
- @pytest.mark.parametrize(
- 'subvolume_path,expected_snapshot_path',
- (
- ('/foo/bar', '/foo/bar/.borgmatic-snapshot-1234/foo/bar'),
- ('/', '/.borgmatic-snapshot-1234'),
- ),
- )
- def test_make_snapshot_path_includes_stripped_subvolume_path(
- subvolume_path, expected_snapshot_path
- ):
- flexmock(module.os).should_receive('getpid').and_return(1234)
- assert module.make_snapshot_path(subvolume_path) == expected_snapshot_path
- @pytest.mark.parametrize(
- 'subvolume_path,pattern,expected_pattern',
- (
- (
- '/foo/bar',
- Pattern('/foo/bar/baz'),
- Pattern('/foo/bar/.borgmatic-snapshot-1234/./foo/bar/baz'),
- ),
- ('/foo/bar', Pattern('/foo/bar'), Pattern('/foo/bar/.borgmatic-snapshot-1234/./foo/bar')),
- (
- '/foo/bar',
- Pattern('^/foo/bar', Pattern_type.INCLUDE, Pattern_style.REGULAR_EXPRESSION),
- Pattern(
- '^/foo/bar/.borgmatic-snapshot-1234/./foo/bar',
- Pattern_type.INCLUDE,
- Pattern_style.REGULAR_EXPRESSION,
- ),
- ),
- (
- '/foo/bar',
- Pattern('/foo/bar', Pattern_type.INCLUDE, Pattern_style.REGULAR_EXPRESSION),
- Pattern(
- '/foo/bar/.borgmatic-snapshot-1234/./foo/bar',
- Pattern_type.INCLUDE,
- Pattern_style.REGULAR_EXPRESSION,
- ),
- ),
- ('/', Pattern('/foo'), Pattern('/.borgmatic-snapshot-1234/./foo')),
- ('/', Pattern('/'), Pattern('/.borgmatic-snapshot-1234/./')),
- ),
- )
- def test_make_borg_snapshot_pattern_includes_slashdot_hack_and_stripped_pattern_path(
- subvolume_path, pattern, expected_pattern
- ):
- flexmock(module.os).should_receive('getpid').and_return(1234)
- assert module.make_borg_snapshot_pattern(subvolume_path, pattern) == expected_pattern
- def test_dump_data_sources_snapshots_each_subvolume_and_updates_patterns():
- patterns = [Pattern('/foo'), Pattern('/mnt/subvol1')]
- config = {'btrfs': {}}
- flexmock(module).should_receive('get_subvolumes').and_return(
- (
- module.Subvolume('/mnt/subvol1', contained_patterns=(Pattern('/mnt/subvol1'),)),
- module.Subvolume('/mnt/subvol2', contained_patterns=(Pattern('/mnt/subvol2'),)),
- )
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol1').and_return(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1'
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol2').and_return(
- '/mnt/subvol2/.borgmatic-1234/mnt/subvol2'
- )
- flexmock(module).should_receive('snapshot_subvolume').with_args(
- 'btrfs', '/mnt/subvol1', '/mnt/subvol1/.borgmatic-1234/mnt/subvol1'
- ).once()
- flexmock(module).should_receive('snapshot_subvolume').with_args(
- 'btrfs', '/mnt/subvol2', '/mnt/subvol2/.borgmatic-1234/mnt/subvol2'
- ).once()
- flexmock(module).should_receive('make_snapshot_exclude_pattern').with_args(
- '/mnt/subvol1'
- ).and_return(
- Pattern(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1/.borgmatic-1234',
- Pattern_type.NO_RECURSE,
- Pattern_style.FNMATCH,
- )
- )
- flexmock(module).should_receive('make_snapshot_exclude_pattern').with_args(
- '/mnt/subvol2'
- ).and_return(
- Pattern(
- '/mnt/subvol2/.borgmatic-1234/mnt/subvol2/.borgmatic-1234',
- Pattern_type.NO_RECURSE,
- Pattern_style.FNMATCH,
- )
- )
- flexmock(module).should_receive('make_borg_snapshot_pattern').with_args(
- '/mnt/subvol1', object
- ).and_return(Pattern('/mnt/subvol1/.borgmatic-1234/mnt/subvol1'))
- flexmock(module).should_receive('make_borg_snapshot_pattern').with_args(
- '/mnt/subvol2', object
- ).and_return(Pattern('/mnt/subvol2/.borgmatic-1234/mnt/subvol2'))
- assert (
- module.dump_data_sources(
- hook_config=config['btrfs'],
- config=config,
- config_paths=('test.yaml',),
- borgmatic_runtime_directory='/run/borgmatic',
- patterns=patterns,
- dry_run=False,
- )
- == []
- )
- assert patterns == [
- Pattern('/foo'),
- Pattern('/mnt/subvol1/.borgmatic-1234/mnt/subvol1'),
- Pattern(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1/.borgmatic-1234',
- Pattern_type.NO_RECURSE,
- Pattern_style.FNMATCH,
- ),
- Pattern('/mnt/subvol2/.borgmatic-1234/mnt/subvol2'),
- Pattern(
- '/mnt/subvol2/.borgmatic-1234/mnt/subvol2/.borgmatic-1234',
- Pattern_type.NO_RECURSE,
- Pattern_style.FNMATCH,
- ),
- ]
- assert config == {
- 'btrfs': {},
- }
- def test_dump_data_sources_uses_custom_btrfs_command_in_commands():
- patterns = [Pattern('/foo'), Pattern('/mnt/subvol1')]
- config = {'btrfs': {'btrfs_command': '/usr/local/bin/btrfs'}}
- flexmock(module).should_receive('get_subvolumes').and_return(
- (module.Subvolume('/mnt/subvol1', contained_patterns=(Pattern('/mnt/subvol1'),)),)
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol1').and_return(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1'
- )
- flexmock(module).should_receive('snapshot_subvolume').with_args(
- '/usr/local/bin/btrfs', '/mnt/subvol1', '/mnt/subvol1/.borgmatic-1234/mnt/subvol1'
- ).once()
- flexmock(module).should_receive('make_snapshot_exclude_pattern').with_args(
- '/mnt/subvol1'
- ).and_return(
- Pattern(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1/.borgmatic-1234',
- Pattern_type.NO_RECURSE,
- Pattern_style.FNMATCH,
- )
- )
- flexmock(module).should_receive('make_borg_snapshot_pattern').with_args(
- '/mnt/subvol1', object
- ).and_return(Pattern('/mnt/subvol1/.borgmatic-1234/mnt/subvol1'))
- assert (
- module.dump_data_sources(
- hook_config=config['btrfs'],
- config=config,
- config_paths=('test.yaml',),
- borgmatic_runtime_directory='/run/borgmatic',
- patterns=patterns,
- dry_run=False,
- )
- == []
- )
- assert patterns == [
- Pattern('/foo'),
- Pattern('/mnt/subvol1/.borgmatic-1234/mnt/subvol1'),
- Pattern(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1/.borgmatic-1234',
- Pattern_type.NO_RECURSE,
- Pattern_style.FNMATCH,
- ),
- ]
- assert config == {
- 'btrfs': {
- 'btrfs_command': '/usr/local/bin/btrfs',
- },
- }
- def test_dump_data_sources_uses_custom_findmnt_command_in_commands():
- patterns = [Pattern('/foo'), Pattern('/mnt/subvol1')]
- config = {'btrfs': {'findmnt_command': '/usr/local/bin/findmnt'}}
- flexmock(module).should_receive('get_subvolumes').with_args(
- 'btrfs', '/usr/local/bin/findmnt', patterns
- ).and_return(
- (module.Subvolume('/mnt/subvol1', contained_patterns=(Pattern('/mnt/subvol1'),)),)
- ).once()
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol1').and_return(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1'
- )
- flexmock(module).should_receive('snapshot_subvolume').with_args(
- 'btrfs', '/mnt/subvol1', '/mnt/subvol1/.borgmatic-1234/mnt/subvol1'
- ).once()
- flexmock(module).should_receive('make_snapshot_exclude_pattern').with_args(
- '/mnt/subvol1'
- ).and_return(
- Pattern(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1/.borgmatic-1234',
- Pattern_type.NO_RECURSE,
- Pattern_style.FNMATCH,
- )
- )
- flexmock(module).should_receive('make_borg_snapshot_pattern').with_args(
- '/mnt/subvol1', object
- ).and_return(Pattern('/mnt/subvol1/.borgmatic-1234/mnt/subvol1'))
- assert (
- module.dump_data_sources(
- hook_config=config['btrfs'],
- config=config,
- config_paths=('test.yaml',),
- borgmatic_runtime_directory='/run/borgmatic',
- patterns=patterns,
- dry_run=False,
- )
- == []
- )
- assert patterns == [
- Pattern('/foo'),
- Pattern('/mnt/subvol1/.borgmatic-1234/mnt/subvol1'),
- Pattern(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1/.borgmatic-1234',
- Pattern_type.NO_RECURSE,
- Pattern_style.FNMATCH,
- ),
- ]
- assert config == {
- 'btrfs': {
- 'findmnt_command': '/usr/local/bin/findmnt',
- },
- }
- def test_dump_data_sources_with_dry_run_skips_snapshot_and_patterns_update():
- patterns = [Pattern('/foo'), Pattern('/mnt/subvol1')]
- config = {'btrfs': {}}
- flexmock(module).should_receive('get_subvolumes').and_return(
- (module.Subvolume('/mnt/subvol1', contained_patterns=(Pattern('/mnt/subvol1'),)),)
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol1').and_return(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1'
- )
- flexmock(module).should_receive('snapshot_subvolume').never()
- flexmock(module).should_receive('make_snapshot_exclude_pattern').never()
- assert (
- module.dump_data_sources(
- hook_config=config['btrfs'],
- config=config,
- config_paths=('test.yaml',),
- borgmatic_runtime_directory='/run/borgmatic',
- patterns=patterns,
- dry_run=True,
- )
- == []
- )
- assert patterns == [Pattern('/foo'), Pattern('/mnt/subvol1')]
- assert config == {'btrfs': {}}
- def test_dump_data_sources_without_matching_subvolumes_skips_snapshot_and_patterns_update():
- patterns = [Pattern('/foo'), Pattern('/mnt/subvol1')]
- config = {'btrfs': {}}
- flexmock(module).should_receive('get_subvolumes').and_return(())
- flexmock(module).should_receive('make_snapshot_path').never()
- flexmock(module).should_receive('snapshot_subvolume').never()
- flexmock(module).should_receive('make_snapshot_exclude_pattern').never()
- assert (
- module.dump_data_sources(
- hook_config=config['btrfs'],
- config=config,
- config_paths=('test.yaml',),
- borgmatic_runtime_directory='/run/borgmatic',
- patterns=patterns,
- dry_run=False,
- )
- == []
- )
- assert patterns == [Pattern('/foo'), Pattern('/mnt/subvol1')]
- assert config == {'btrfs': {}}
- def test_dump_data_sources_snapshots_adds_to_existing_exclude_patterns():
- patterns = [Pattern('/foo'), Pattern('/mnt/subvol1')]
- config = {'btrfs': {}, 'exclude_patterns': ['/bar']}
- flexmock(module).should_receive('get_subvolumes').and_return(
- (
- module.Subvolume('/mnt/subvol1', contained_patterns=(Pattern('/mnt/subvol1'),)),
- module.Subvolume('/mnt/subvol2', contained_patterns=(Pattern('/mnt/subvol2'),)),
- )
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol1').and_return(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1'
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol2').and_return(
- '/mnt/subvol2/.borgmatic-1234/mnt/subvol2'
- )
- flexmock(module).should_receive('snapshot_subvolume').with_args(
- 'btrfs', '/mnt/subvol1', '/mnt/subvol1/.borgmatic-1234/mnt/subvol1'
- ).once()
- flexmock(module).should_receive('snapshot_subvolume').with_args(
- 'btrfs', '/mnt/subvol2', '/mnt/subvol2/.borgmatic-1234/mnt/subvol2'
- ).once()
- flexmock(module).should_receive('make_snapshot_exclude_pattern').with_args(
- '/mnt/subvol1'
- ).and_return(
- Pattern(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1/.borgmatic-1234',
- Pattern_type.NO_RECURSE,
- Pattern_style.FNMATCH,
- )
- )
- flexmock(module).should_receive('make_snapshot_exclude_pattern').with_args(
- '/mnt/subvol2'
- ).and_return(
- Pattern(
- '/mnt/subvol2/.borgmatic-1234/mnt/subvol2/.borgmatic-1234',
- Pattern_type.NO_RECURSE,
- Pattern_style.FNMATCH,
- )
- )
- flexmock(module).should_receive('make_borg_snapshot_pattern').with_args(
- '/mnt/subvol1', object
- ).and_return(Pattern('/mnt/subvol1/.borgmatic-1234/mnt/subvol1'))
- flexmock(module).should_receive('make_borg_snapshot_pattern').with_args(
- '/mnt/subvol2', object
- ).and_return(Pattern('/mnt/subvol2/.borgmatic-1234/mnt/subvol2'))
- assert (
- module.dump_data_sources(
- hook_config=config['btrfs'],
- config=config,
- config_paths=('test.yaml',),
- borgmatic_runtime_directory='/run/borgmatic',
- patterns=patterns,
- dry_run=False,
- )
- == []
- )
- assert patterns == [
- Pattern('/foo'),
- Pattern('/mnt/subvol1/.borgmatic-1234/mnt/subvol1'),
- Pattern(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1/.borgmatic-1234',
- Pattern_type.NO_RECURSE,
- Pattern_style.FNMATCH,
- ),
- Pattern('/mnt/subvol2/.borgmatic-1234/mnt/subvol2'),
- Pattern(
- '/mnt/subvol2/.borgmatic-1234/mnt/subvol2/.borgmatic-1234',
- Pattern_type.NO_RECURSE,
- Pattern_style.FNMATCH,
- ),
- ]
- assert config == {
- 'btrfs': {},
- 'exclude_patterns': ['/bar'],
- }
- def test_remove_data_source_dumps_deletes_snapshots():
- config = {'btrfs': {}}
- flexmock(module).should_receive('get_subvolumes').and_return(
- (
- module.Subvolume('/mnt/subvol1', contained_patterns=(Pattern('/mnt/subvol1'),)),
- module.Subvolume('/mnt/subvol2', contained_patterns=(Pattern('/mnt/subvol2'),)),
- )
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol1').and_return(
- '/mnt/subvol1/.borgmatic-1234/./mnt/subvol1'
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol2').and_return(
- '/mnt/subvol2/.borgmatic-1234/./mnt/subvol2'
- )
- flexmock(module.borgmatic.config.paths).should_receive(
- 'replace_temporary_subdirectory_with_glob'
- ).with_args(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1',
- temporary_directory_prefix=module.BORGMATIC_SNAPSHOT_PREFIX,
- ).and_return(
- '/mnt/subvol1/.borgmatic-*/mnt/subvol1'
- )
- flexmock(module.borgmatic.config.paths).should_receive(
- 'replace_temporary_subdirectory_with_glob'
- ).with_args(
- '/mnt/subvol2/.borgmatic-1234/mnt/subvol2',
- temporary_directory_prefix=module.BORGMATIC_SNAPSHOT_PREFIX,
- ).and_return(
- '/mnt/subvol2/.borgmatic-*/mnt/subvol2'
- )
- flexmock(module.glob).should_receive('glob').with_args(
- '/mnt/subvol1/.borgmatic-*/mnt/subvol1'
- ).and_return(
- ('/mnt/subvol1/.borgmatic-1234/mnt/subvol1', '/mnt/subvol1/.borgmatic-5678/mnt/subvol1')
- )
- flexmock(module.glob).should_receive('glob').with_args(
- '/mnt/subvol2/.borgmatic-*/mnt/subvol2'
- ).and_return(
- ('/mnt/subvol2/.borgmatic-1234/mnt/subvol2', '/mnt/subvol2/.borgmatic-5678/mnt/subvol2')
- )
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1'
- ).and_return(True)
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol1/.borgmatic-5678/mnt/subvol1'
- ).and_return(True)
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol2/.borgmatic-1234/mnt/subvol2'
- ).and_return(True)
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol2/.borgmatic-5678/mnt/subvol2'
- ).and_return(False)
- flexmock(module).should_receive('delete_snapshot').with_args(
- 'btrfs', '/mnt/subvol1/.borgmatic-1234/mnt/subvol1'
- ).once()
- flexmock(module).should_receive('delete_snapshot').with_args(
- 'btrfs', '/mnt/subvol1/.borgmatic-5678/mnt/subvol1'
- ).once()
- flexmock(module).should_receive('delete_snapshot').with_args(
- 'btrfs', '/mnt/subvol2/.borgmatic-1234/mnt/subvol2'
- ).once()
- flexmock(module).should_receive('delete_snapshot').with_args(
- 'btrfs', '/mnt/subvol2/.borgmatic-5678/mnt/subvol2'
- ).never()
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol1/.borgmatic-1234'
- ).and_return(True)
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol1/.borgmatic-5678'
- ).and_return(True)
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol2/.borgmatic-1234'
- ).and_return(True)
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol2/.borgmatic-5678'
- ).and_return(True)
- flexmock(module.shutil).should_receive('rmtree').with_args(
- '/mnt/subvol1/.borgmatic-1234'
- ).once()
- flexmock(module.shutil).should_receive('rmtree').with_args(
- '/mnt/subvol1/.borgmatic-5678'
- ).once()
- flexmock(module.shutil).should_receive('rmtree').with_args(
- '/mnt/subvol2/.borgmatic-1234'
- ).once()
- flexmock(module.shutil).should_receive('rmtree').with_args(
- '/mnt/subvol2/.borgmatic-5678'
- ).never()
- module.remove_data_source_dumps(
- hook_config=config['btrfs'],
- config=config,
- borgmatic_runtime_directory='/run/borgmatic',
- dry_run=False,
- )
- def test_remove_data_source_dumps_without_hook_configuration_bails():
- flexmock(module).should_receive('get_subvolumes').never()
- flexmock(module).should_receive('make_snapshot_path').never()
- flexmock(module.borgmatic.config.paths).should_receive(
- 'replace_temporary_subdirectory_with_glob'
- ).never()
- flexmock(module).should_receive('delete_snapshot').never()
- flexmock(module.shutil).should_receive('rmtree').never()
- module.remove_data_source_dumps(
- hook_config=None,
- config={'source_directories': '/mnt/subvolume'},
- borgmatic_runtime_directory='/run/borgmatic',
- dry_run=False,
- )
- def test_remove_data_source_dumps_with_get_subvolumes_file_not_found_error_bails():
- config = {'btrfs': {}}
- flexmock(module).should_receive('get_subvolumes').and_raise(FileNotFoundError)
- flexmock(module).should_receive('make_snapshot_path').never()
- flexmock(module.borgmatic.config.paths).should_receive(
- 'replace_temporary_subdirectory_with_glob'
- ).never()
- flexmock(module).should_receive('delete_snapshot').never()
- flexmock(module.shutil).should_receive('rmtree').never()
- module.remove_data_source_dumps(
- hook_config=config['btrfs'],
- config=config,
- borgmatic_runtime_directory='/run/borgmatic',
- dry_run=False,
- )
- def test_remove_data_source_dumps_with_get_subvolumes_called_process_error_bails():
- config = {'btrfs': {}}
- flexmock(module).should_receive('get_subvolumes').and_raise(
- module.subprocess.CalledProcessError(1, 'command', 'error')
- )
- flexmock(module).should_receive('make_snapshot_path').never()
- flexmock(module.borgmatic.config.paths).should_receive(
- 'replace_temporary_subdirectory_with_glob'
- ).never()
- flexmock(module).should_receive('delete_snapshot').never()
- flexmock(module.shutil).should_receive('rmtree').never()
- module.remove_data_source_dumps(
- hook_config=config['btrfs'],
- config=config,
- borgmatic_runtime_directory='/run/borgmatic',
- dry_run=False,
- )
- def test_remove_data_source_dumps_with_dry_run_skips_deletes():
- config = {'btrfs': {}}
- flexmock(module).should_receive('get_subvolumes').and_return(
- (
- module.Subvolume('/mnt/subvol1', contained_patterns=(Pattern('/mnt/subvol1'),)),
- module.Subvolume('/mnt/subvol2', contained_patterns=(Pattern('/mnt/subvol2'),)),
- )
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol1').and_return(
- '/mnt/subvol1/.borgmatic-1234/./mnt/subvol1'
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol2').and_return(
- '/mnt/subvol2/.borgmatic-1234/./mnt/subvol2'
- )
- flexmock(module.borgmatic.config.paths).should_receive(
- 'replace_temporary_subdirectory_with_glob'
- ).with_args(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1',
- temporary_directory_prefix=module.BORGMATIC_SNAPSHOT_PREFIX,
- ).and_return(
- '/mnt/subvol1/.borgmatic-*/mnt/subvol1'
- )
- flexmock(module.borgmatic.config.paths).should_receive(
- 'replace_temporary_subdirectory_with_glob'
- ).with_args(
- '/mnt/subvol2/.borgmatic-1234/mnt/subvol2',
- temporary_directory_prefix=module.BORGMATIC_SNAPSHOT_PREFIX,
- ).and_return(
- '/mnt/subvol2/.borgmatic-*/mnt/subvol2'
- )
- flexmock(module.glob).should_receive('glob').with_args(
- '/mnt/subvol1/.borgmatic-*/mnt/subvol1'
- ).and_return(
- ('/mnt/subvol1/.borgmatic-1234/mnt/subvol1', '/mnt/subvol1/.borgmatic-5678/mnt/subvol1')
- )
- flexmock(module.glob).should_receive('glob').with_args(
- '/mnt/subvol2/.borgmatic-*/mnt/subvol2'
- ).and_return(
- ('/mnt/subvol2/.borgmatic-1234/mnt/subvol2', '/mnt/subvol2/.borgmatic-5678/mnt/subvol2')
- )
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1'
- ).and_return(True)
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol1/.borgmatic-5678/mnt/subvol1'
- ).and_return(True)
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol2/.borgmatic-1234/mnt/subvol2'
- ).and_return(True)
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol2/.borgmatic-5678/mnt/subvol2'
- ).and_return(False)
- flexmock(module).should_receive('delete_snapshot').never()
- flexmock(module.shutil).should_receive('rmtree').never()
- module.remove_data_source_dumps(
- hook_config=config['btrfs'],
- config=config,
- borgmatic_runtime_directory='/run/borgmatic',
- dry_run=True,
- )
- def test_remove_data_source_dumps_without_subvolumes_skips_deletes():
- config = {'btrfs': {}}
- flexmock(module).should_receive('get_subvolumes').and_return(())
- flexmock(module).should_receive('make_snapshot_path').never()
- flexmock(module.borgmatic.config.paths).should_receive(
- 'replace_temporary_subdirectory_with_glob'
- ).never()
- flexmock(module).should_receive('delete_snapshot').never()
- flexmock(module.shutil).should_receive('rmtree').never()
- module.remove_data_source_dumps(
- hook_config=config['btrfs'],
- config=config,
- borgmatic_runtime_directory='/run/borgmatic',
- dry_run=False,
- )
- def test_remove_data_source_without_snapshots_skips_deletes():
- config = {'btrfs': {}}
- flexmock(module).should_receive('get_subvolumes').and_return(
- (
- module.Subvolume('/mnt/subvol1', contained_patterns=(Pattern('/mnt/subvol1'),)),
- module.Subvolume('/mnt/subvol2', contained_patterns=(Pattern('/mnt/subvol2'),)),
- )
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol1').and_return(
- '/mnt/subvol1/.borgmatic-1234/./mnt/subvol1'
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol2').and_return(
- '/mnt/subvol2/.borgmatic-1234/./mnt/subvol2'
- )
- flexmock(module.borgmatic.config.paths).should_receive(
- 'replace_temporary_subdirectory_with_glob'
- ).with_args(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1',
- temporary_directory_prefix=module.BORGMATIC_SNAPSHOT_PREFIX,
- ).and_return(
- '/mnt/subvol1/.borgmatic-*/mnt/subvol1'
- )
- flexmock(module.borgmatic.config.paths).should_receive(
- 'replace_temporary_subdirectory_with_glob'
- ).with_args(
- '/mnt/subvol2/.borgmatic-1234/mnt/subvol2',
- temporary_directory_prefix=module.BORGMATIC_SNAPSHOT_PREFIX,
- ).and_return(
- '/mnt/subvol2/.borgmatic-*/mnt/subvol2'
- )
- flexmock(module.glob).should_receive('glob').and_return(())
- flexmock(module.os.path).should_receive('isdir').never()
- flexmock(module).should_receive('delete_snapshot').never()
- flexmock(module.shutil).should_receive('rmtree').never()
- module.remove_data_source_dumps(
- hook_config=config['btrfs'],
- config=config,
- borgmatic_runtime_directory='/run/borgmatic',
- dry_run=False,
- )
- def test_remove_data_source_dumps_with_delete_snapshot_file_not_found_error_bails():
- config = {'btrfs': {}}
- flexmock(module).should_receive('get_subvolumes').and_return(
- (
- module.Subvolume('/mnt/subvol1', contained_patterns=(Pattern('/mnt/subvol1'),)),
- module.Subvolume('/mnt/subvol2', contained_patterns=(Pattern('/mnt/subvol2'),)),
- )
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol1').and_return(
- '/mnt/subvol1/.borgmatic-1234/./mnt/subvol1'
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol2').and_return(
- '/mnt/subvol2/.borgmatic-1234/./mnt/subvol2'
- )
- flexmock(module.borgmatic.config.paths).should_receive(
- 'replace_temporary_subdirectory_with_glob'
- ).with_args(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1',
- temporary_directory_prefix=module.BORGMATIC_SNAPSHOT_PREFIX,
- ).and_return(
- '/mnt/subvol1/.borgmatic-*/mnt/subvol1'
- )
- flexmock(module.borgmatic.config.paths).should_receive(
- 'replace_temporary_subdirectory_with_glob'
- ).with_args(
- '/mnt/subvol2/.borgmatic-1234/mnt/subvol2',
- temporary_directory_prefix=module.BORGMATIC_SNAPSHOT_PREFIX,
- ).and_return(
- '/mnt/subvol2/.borgmatic-*/mnt/subvol2'
- )
- flexmock(module.glob).should_receive('glob').with_args(
- '/mnt/subvol1/.borgmatic-*/mnt/subvol1'
- ).and_return(
- ('/mnt/subvol1/.borgmatic-1234/mnt/subvol1', '/mnt/subvol1/.borgmatic-5678/mnt/subvol1')
- )
- flexmock(module.glob).should_receive('glob').with_args(
- '/mnt/subvol2/.borgmatic-*/mnt/subvol2'
- ).and_return(
- ('/mnt/subvol2/.borgmatic-1234/mnt/subvol2', '/mnt/subvol2/.borgmatic-5678/mnt/subvol2')
- )
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1'
- ).and_return(True)
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol1/.borgmatic-5678/mnt/subvol1'
- ).and_return(True)
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol2/.borgmatic-1234/mnt/subvol2'
- ).and_return(True)
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol2/.borgmatic-5678/mnt/subvol2'
- ).and_return(False)
- flexmock(module).should_receive('delete_snapshot').and_raise(FileNotFoundError)
- flexmock(module.shutil).should_receive('rmtree').never()
- module.remove_data_source_dumps(
- hook_config=config['btrfs'],
- config=config,
- borgmatic_runtime_directory='/run/borgmatic',
- dry_run=False,
- )
- def test_remove_data_source_dumps_with_delete_snapshot_called_process_error_bails():
- config = {'btrfs': {}}
- flexmock(module).should_receive('get_subvolumes').and_return(
- (
- module.Subvolume('/mnt/subvol1', contained_patterns=(Pattern('/mnt/subvol1'),)),
- module.Subvolume('/mnt/subvol2', contained_patterns=(Pattern('/mnt/subvol2'),)),
- )
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol1').and_return(
- '/mnt/subvol1/.borgmatic-1234/./mnt/subvol1'
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/mnt/subvol2').and_return(
- '/mnt/subvol2/.borgmatic-1234/./mnt/subvol2'
- )
- flexmock(module.borgmatic.config.paths).should_receive(
- 'replace_temporary_subdirectory_with_glob'
- ).with_args(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1',
- temporary_directory_prefix=module.BORGMATIC_SNAPSHOT_PREFIX,
- ).and_return(
- '/mnt/subvol1/.borgmatic-*/mnt/subvol1'
- )
- flexmock(module.borgmatic.config.paths).should_receive(
- 'replace_temporary_subdirectory_with_glob'
- ).with_args(
- '/mnt/subvol2/.borgmatic-1234/mnt/subvol2',
- temporary_directory_prefix=module.BORGMATIC_SNAPSHOT_PREFIX,
- ).and_return(
- '/mnt/subvol2/.borgmatic-*/mnt/subvol2'
- )
- flexmock(module.glob).should_receive('glob').with_args(
- '/mnt/subvol1/.borgmatic-*/mnt/subvol1'
- ).and_return(
- ('/mnt/subvol1/.borgmatic-1234/mnt/subvol1', '/mnt/subvol1/.borgmatic-5678/mnt/subvol1')
- )
- flexmock(module.glob).should_receive('glob').with_args(
- '/mnt/subvol2/.borgmatic-*/mnt/subvol2'
- ).and_return(
- ('/mnt/subvol2/.borgmatic-1234/mnt/subvol2', '/mnt/subvol2/.borgmatic-5678/mnt/subvol2')
- )
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol1/.borgmatic-1234/mnt/subvol1'
- ).and_return(True)
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol1/.borgmatic-5678/mnt/subvol1'
- ).and_return(True)
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol2/.borgmatic-1234/mnt/subvol2'
- ).and_return(True)
- flexmock(module.os.path).should_receive('isdir').with_args(
- '/mnt/subvol2/.borgmatic-5678/mnt/subvol2'
- ).and_return(False)
- flexmock(module).should_receive('delete_snapshot').and_raise(
- module.subprocess.CalledProcessError(1, 'command', 'error')
- )
- flexmock(module.shutil).should_receive('rmtree').never()
- module.remove_data_source_dumps(
- hook_config=config['btrfs'],
- config=config,
- borgmatic_runtime_directory='/run/borgmatic',
- dry_run=False,
- )
- def test_remove_data_source_dumps_with_root_subvolume_skips_duplicate_removal():
- config = {'btrfs': {}}
- flexmock(module).should_receive('get_subvolumes').and_return(
- (module.Subvolume('/', contained_patterns=(Pattern('/etc'),)),)
- )
- flexmock(module).should_receive('make_snapshot_path').with_args('/').and_return(
- '/.borgmatic-1234'
- )
- flexmock(module.borgmatic.config.paths).should_receive(
- 'replace_temporary_subdirectory_with_glob'
- ).with_args(
- '/.borgmatic-1234',
- temporary_directory_prefix=module.BORGMATIC_SNAPSHOT_PREFIX,
- ).and_return(
- '/.borgmatic-*'
- )
- flexmock(module.glob).should_receive('glob').with_args('/.borgmatic-*').and_return(
- ('/.borgmatic-1234', '/.borgmatic-5678')
- )
- flexmock(module.os.path).should_receive('isdir').with_args('/.borgmatic-1234').and_return(
- True
- ).and_return(False)
- flexmock(module.os.path).should_receive('isdir').with_args('/.borgmatic-5678').and_return(
- True
- ).and_return(False)
- flexmock(module).should_receive('delete_snapshot').with_args('btrfs', '/.borgmatic-1234').once()
- flexmock(module).should_receive('delete_snapshot').with_args('btrfs', '/.borgmatic-5678').once()
- flexmock(module.os.path).should_receive('isdir').with_args('').and_return(False)
- flexmock(module.shutil).should_receive('rmtree').never()
- module.remove_data_source_dumps(
- hook_config=config['btrfs'],
- config=config,
- borgmatic_runtime_directory='/run/borgmatic',
- dry_run=False,
- )
|