This is a snapshot of Indico's old Trac site. Any information contained herein is most probably outdated. Access our new GitHub site here.

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#1027 closed defect (fixed)

migration script misses a parameter buildIndex() method

Reported by: makub Owned by: arescope
Priority: normal Milestone: v0.98.2
Component: Installation Version: 0.98-dev
Keywords: Cc:

Description

When upgrading from 0.97 to 0.98.1, the migrate_0.97_0.98.py on line 321 calls 

        newIdx.buildIndex()

but the method as defined in MaKaC/common/indexes.py on line 1030 needs the parameter 'dbi'

The script fails with

Migration failed! DB may be in  an inconsistent state:
Traceback (most recent call last):
  File "/data/indico/bin/migration/migrate_0.97_0.98.py", line 554, in main
    specified=filter(lambda x: x, map(lambda x: x.strip(), args.specified.split(','))))
  File "/data/indico/bin/migration/migrate_0.97_0.98.py", line 519, in runMigration
    task(dbi, withRBDB, prevVersion)
  File "/data/indico/bin/migration/migrate_0.97_0.98.py", line 321, in categoryDateIndexMigration
    newIdx.buildIndex()
TypeError: buildIndex() takes exactly 2 arguments (1 given)

A fix is to add the parameter 'dbi' which is available.

Change History (5)

comment:1 Changed 3 years ago by arescope

  • Owner set to arescope
  • Status changed from new to assigned

comment:2 Changed 3 years ago by arescope

  • Status changed from assigned to in_work

comment:3 Changed 3 years ago by arescope

  • Status changed from in_work to awaiting_merge

comment:4 Changed 3 years ago by Alberto Resco Perez <alberto.resco.perez@…>

  • Resolution set to fixed
  • Status changed from awaiting_merge to closed

In [6619b643a52827b0367bd1f6daed55bcebd54540/indico]:

[FIX] Migration script missing parameter

comment:5 Changed 3 years ago by Alberto Resco Perez <alberto.resco.perez@…>

In 6619b643a52827b0367bd1f6daed55bcebd54540/indico:

[FIX] Migration script missing parameter

Note: See TracTickets for help on using tickets.