Verifying the InventoryDatabase
throughnothing at gmail.com
Mon May 11 15:02:38 UTC 2009
One other thing to note. I was testing this with my CloudFiles
target, and I needed to add 'size' and 'chunkpath' functions because
the script requires them. Both functions were very trivial, but I
believe not all targets have these functions. Did you add these
functions to whatever target you are using Gavin? I looked and the
Filesystem, Filebased, Ftp, and Sftp targets seem to have the
'chunkpath' function, but none of the targets seemed to contain a size
function in the current svn head.
I'm planning on putting these 2 functions into the CloudFiles.pm
target when I commit it, but I guess before the
brackup-verify-inventory script is committed, it should be made sure
that these functions exist in all the targets?
Is there any objection to me including these 2 functions in the
I'm just trying to help get this script into the svn (and hopefully
into the next release), because it helps reduce my paranoia :)
On Thu, May 7, 2009 at 6:33 PM, Brad Fitzpatrick <brad at danga.com> wrote:
> Sure, check it in. Be sure to include it in the MANIFEST, ChangeLog, etc.
> On Tue, May 5, 2009 at 7:55 PM, Gavin Carr <gavin at openfusion.com.au> wrote:
>> On Tue, May 05, 2009 at 11:49:20AM -0400, William Wolf wrote:
>> > I'm trying to use your script to verify the inventory database, but I am
>> > running into errors. I am getting the following when I try to run
>> > ./brackup-verify-inventory -q --delete TARGET
>> > Can't locate object method "count" via package "Brackup::Dict::SQLite"
>> > at
>> > ./brackup-verify-inventory line 111.
>> > Did you have to modify lib/Brackup/Dict/SQLite.pm to add a count
>> > function
>> > there? It also looks like it will be missing an each function, but I
>> > may
>> > be missing something here. Thanks.
>> Eerk, sorry, I'd forgotten I'd patched Dict/SQLite. Patch attached.
>> Mainline only has get/set interfaces. Patch adds count, each, and delete.
>> Brad, are you interested in this?
More information about the brackup