Verifying the InventoryDatabase

[William Wolf]

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
CloudFiles target?


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?
>>
>> Cheers,
>> Gavin
>>
>
>



-- 

William Wolf