Fix messages about retention (stored in seconds).

softlayer / softlayer-object-storage-backup

Other

11 stars 3 forks source link

Fix messages about retention (stored in seconds). #19

Closed smola closed 11 years ago

smola commented 11 years ago

Retention is stored in seconds. Some comments and message say it is stored in days.

smola commented 11 years ago

It took me a while to figure this out when I specified --retention 7 and getting the following message:

Override: Setting retention days to 604800

CrackerJackMack commented 11 years ago

Good catch! Yeah this was due to a refactoring bug when I introduce retention suffixes. But to not break anything I had to assume that none-suffixed periods were in days. This lead me to completely overlook the actual words I was using and just verifying that I indeed correctly converted the old retention days to the proper seconds.

TL;DR: whoops, sorry about that. I would have had a small heart attack to.