Merge pull request #480 from oetiker/fix_regression_479

fix for regression introduced by 60e7afe as mentioned in #479
oetiker · Mar 2, 2020 · df62921 · df62921
2 parents 2134114 + 7629a02
commit df62921
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 14 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -3,12 +3,11 @@ name: Test
 on:
   push:
     branches:
-    - "*"
+    - master
     tags-ignore:
     - "*"
   pull_request:
-    types: [opened, reopened]
-
+
 jobs:
   unit:
     runs-on: ubuntu-latest

diff --git a/lib/ZnapZend/Config.pm b/lib/ZnapZend/Config.pm
@@ -89,17 +89,6 @@ my $checkBackupSets = sub {
            next;
         }
 
-        if ( $dataSet =~ m/[\@]/ ) {
-            # If we are here, somebody fed us a snapshot in the list of
-            # datasets, which is likely a bug elsewhere in discovery.
-            # We do not want to fail whole backup below due to faulted
-            # dataSetExists() below, so just ignore this entry.
-            # If we really do get here, take a hard look at recursive
-            # and/or inherited modes for run-once.
-            print STDERR "#checkBackupSets# SKIP $backupSet->{src} because it is not a filesystem,volume. BUG: Should not get here.\n";# if $self->debug;
-            next;
-        }
-
         for my $prop (keys %{$self->mandProperties}){
             exists $backupSet->{$prop} || do {
                 $self->zLog->info("WARNING: property $prop not set on backup for " . $backupSet->{src} . ". Skipping to next dataset");