hammer cleanup removes snapshots even when retention time is 0 when it ought to have disabled removing snapshots

[Siju George <sgeorge.ml@xxxxxxxxx>]

Hi,

The hammer man page says


             If the snapshots directive has a period of 0 and a retention time
             of 0 then snapshot generation is disabled, removal of old snap-
             shots are disabled,

So I configured it with the folowing values.

snapshots 0d 0d
prune     1d 5m
rebalance 1d 5m
reblock   1d 5m
recopy    30d 10m

and did a hammer cleanup but lost all snapshots ?


dfly-bkpsrv# hammer cleanup  /Backup2/www-5mbak/www-hot/
cleanup /Backup2/www-5mbak/www-hot/ - handle PFS #4 using
/var/hammer/Backup2/www-5mbak/www-hot/
           snapshots - disabled
               prune - run
           rebalance - run..
             reblock - run....
              recopy - run....^Cdfly-bkpsrv#
dfly-bkpsrv# hammer snapls /Backup2/www-5mbak/www-hot/
Snapshots on (null)     PFS #4
Transaction ID          Timestamp               Note
dfly-bkpsrv#

Did I miss some thing some where?

It is just a slave mirror PFS so nothing to worry but I just didn't
get what the man page said.
The retention time inthis case was 0d but removal of old snapshots was
not disabled :-(

Thanks

--Siju