Rudder

It seems that the logic of the migration of eventlogs contains some inconsistency between the call from rudder-upgrade script and the code in Scala.

The result is that eventlogs from version is never made and the PostgreSQL could contains this:

rudder=# \d migrationeventlog
 id                  | integer                  | non NULL Par défaut, nextval('migrationeventlogid'::regclass)
 detectiontime       | timestamp with time zone | non NULL
 detectedfileformat  | integer                  | 
 migrationstarttime  | timestamp with time zone | 
 migrationendtime    | timestamp with time zone | 
 migrationfileformat | integer                  | 
 description         | text                     |

rudder=# SELECT * FROM migrationeventlog;
  1 | 2013-08-01 14:01:21.003138+02 |                  1 |                            |                            |                     | 
  2 | 2013-08-01 14:01:21.245159+02 |                  2 |                            |                            |                     | 
  3 | 2013-08-01 14:10:31.346582+02 |                  1 |                            |                            |                     | 
  4 | 2013-08-01 14:10:31.558145+02 |                  2 |                            |                            |                     | 
  5 | 2013-08-01 14:33:01.578001+02 |                  1 |                            |                            |                     | 
  6 | 2013-08-01 14:33:02.058985+02 |                  2 | 2013-08-01 14:45:23.896+02 | 2013-08-01 14:45:25.961+02 |                   3 | 
  7 | 2013-08-08 10:06:55.36604+02  |                  2 |                            |                            |                     | 
  8 | 2013-08-08 10:06:55.762059+02 |                  3 | 2013-08-08 10:07:02.453+02 | 2013-08-08 10:07:06.033+02 |                   4 |

That's a server which contains a lot of eventlogs v1:

rudder=# select count(*) from (select xpath('/entry/*[@fileFormat=1]',data) AS x from eventlog) as Y where array_upper(x, 1) > 0;
   146

The logic in the script is:

• If I find a eventlog v1, I add a flag 1
• If I find a eventlog v2, I add a flag 2
• etc...

The logic in Scala is:

• I search for the last flag and if there is no date of migration:
  • I migrate from the version of the flag to the next version until I have reached the more recent on (3 on 2.6 and 4 on 2.7)
  • I write the date of the begining and the end of the migration

Finally, the logic to is to have a flag from the oldest version existing in PostgreSQL to let Rudder make a migration from it but the rudder-upgrade script add all the versions found and write them successively into PostgreSQL.

The fix will be to change rudder-upgrade script to only write the oldest eventlog version.