Migration script: Wait for finalization, improve error handling and logging

utils/migration-scripts/.prettierignore

@@ -1,2 +1,3 @@
+lib
 results
 src/sumer-giza/sumer-query-node/generated

utils/migration-scripts/package.json

@@ -31,7 +31,9 @@
     "@types/sharp": "^0.29.2",
     "form-data": "^4.0.0",
     "node-cleanup": "^2.1.2",
-    "@types/node-cleanup": "^2.1.2"
+    "@types/node-cleanup": "^2.1.2",
+    "winston": "^3.3.3",
+    "fast-safe-stringify": "^2.1.1"
   },
   "devDependencies": {
     "@graphql-codegen/cli": "^1.21.4",

utils/migration-scripts/src/RuntimeApi.ts

@@ -61,7 +61,7 @@ export class RuntimeApi extends ApiPromise {
     return (events.sort((a, b) => new BN(a.index).cmp(new BN(b.index))) as unknown) as EventType<S, M>[]
   }
 
-  private formatDispatchError(err: DispatchError): string {
+  public formatDispatchError(err: DispatchError): string {
     try {
       const { name, docs } = this.registry.findMetaError(err.asModule)
       return `${name} (${docs.join(', ')})`
@@ -81,7 +81,11 @@ export class RuntimeApi extends ApiPromise {
     return entries.sort((a, b) => a[0].toNumber() - b[0].toNumber())
   }
 
-  sendExtrinsic(keyPair: KeyringPair, tx: SubmittableExtrinsic<'promise'>): Promise<SubmittableResult> {
+  sendExtrinsic(
+    keyPair: KeyringPair,
+    tx: SubmittableExtrinsic<'promise'>,
+    waitUntilFinalized = true
+  ): Promise<SubmittableResult> {
     let txName = `${tx.method.section}.${tx.method.method}`
     if (txName === 'sudo.sudo') {
       const innerCall = tx.args[0] as Call
@@ -95,7 +99,7 @@ export class RuntimeApi extends ApiPromise {
           return
         }
 
-        if (result.status.isInBlock) {
+        if ((!waitUntilFinalized && result.status.isInBlock) || result.status.isFinalized) {
           unsubscribe()
           result.events
             .filter(({ event }) => event.section === 'system')

utils/migration-scripts/src/logging.ts

@@ -0,0 +1,26 @@
+import winston, { Logger } from 'winston'
+import stringify from 'fast-safe-stringify'
+
+const colors = {
+  error: 'red',
+  warn: 'yellow',
+  info: 'green',
+}
+winston.addColors(colors)
+
+export function createLogger(label: string): Logger {
+  return winston.createLogger({
+    transports: [new winston.transports.Console()],
+    defaultMeta: { label },
+    format: winston.format.combine(
+      winston.format.timestamp({ format: 'YYYY-MM-DD HH:mm:ss:ms' }),
+      winston.format.metadata({ fillExcept: ['label', 'level', 'timestamp', 'message'] }),
+      winston.format.colorize(),
+      winston.format.printf(
+        (info) =>
+          `${info.timestamp} ${info.label} [${info.level}]: ${info.message}` +
+          (Object.keys(info.metadata).length ? `\n${stringify(info.metadata, undefined, 4)}` : '')
+      )
+    ),
+  })
+}

utils/migration-scripts/src/sumer-giza/AssetsManager.ts

@@ -16,7 +16,9 @@ import { ContentHash } from './ContentHash'
 import { promisify } from 'util'
 import { createType } from '@joystream/types'
 import { Readable, pipeline } from 'stream'
+import { Logger } from 'winston'
 import _ from 'lodash'
+import { createLogger } from '../logging'
 
 export type AssetsManagerConfig = {
   preferredDownloadSpEndpoints?: string[]
@@ -56,6 +58,7 @@ export class AssetsManager {
   private resizer: ImageResizer
   private queuedUploads: Set<string>
   private isQueueProcessing = false
+  private logger: Logger
 
   public get queueSize(): number {
     return this.queuedUploads.size
@@ -79,6 +82,7 @@ export class AssetsManager {
     this.config = config
     this.resizer = new ImageResizer()
     this.queuedUploads = new Set()
+    this.logger = createLogger('Assets Manager')
     fs.mkdirSync(this.tmpAssetPath(''), { recursive: true })
     fs.mkdirSync(this.assetPath(''), { recursive: true })
   }
@@ -116,7 +120,7 @@ export class AssetsManager {
     targetSize?: [number, number]
   ): Promise<DataObjectCreationParameters | undefined> {
     if (data.liaisonJudgement !== 'ACCEPTED') {
-      console.error(
+      this.logger.warn(
         `Data object ${data.joystreamContentId} has invalid liason judgement: ${data.liaisonJudgement}. Skipping...`
       )
       return
@@ -126,7 +130,7 @@ export class AssetsManager {
     try {
       path = await this.fetchAssetWithRetry(data.joystreamContentId, objectSize)
     } catch (e) {
-      console.error(`Data object ${data.joystreamContentId} was not fetched: ${(e as Error).message}`)
+      this.logger.error(`Data object ${data.joystreamContentId} was not fetched: ${(e as Error).message}`)
       return
     }
     if (targetSize) {
@@ -135,7 +139,7 @@ export class AssetsManager {
         // Re-estabilish object size
         objectSize = fs.statSync(path).size
       } catch (e) {
-        console.error(
+        this.logger.error(
           `Could not resize image ${path} to target size ${targetSize[0]}/${targetSize[1]}: ${(e as Error).message}`
         )
       }
@@ -256,7 +260,7 @@ export class AssetsManager {
     } catch (e) {
       uploadSuccesful = false
       const msg = this.reqErrorMessage(e)
-      console.error(`Upload of object ${dataObjectId} to ${uploadSpEndpoint} failed: ${msg}`)
+      this.logger.error(`Upload of object ${dataObjectId} to ${uploadSpEndpoint} failed: ${msg}`)
     }
 
     if (uploadSuccesful) {
@@ -265,7 +269,7 @@ export class AssetsManager {
       try {
         fs.rmSync(dataPath)
       } catch (e) {
-        console.error(`Could not remove file "${dataPath}" after succesful upload...`)
+        this.logger.error(`Could not remove file "${dataPath}" after succesful upload...`)
       }
     }
   }
@@ -275,7 +279,7 @@ export class AssetsManager {
       throw new Error('Uploads queue is already beeing processed!')
     }
     this.isQueueProcessing = true
-    console.log(`Uploading ${this.queueSize} data objects...`)
+    this.logger.info(`Uploading ${this.queueSize} data objects...`)
     await Promise.all(
       Array.from(this.queuedUploads).map((queuedUpload) => {
         const [bagId, objectId] = queuedUpload.split('|')
@@ -299,10 +303,14 @@ export class AssetsManager {
     this.queuedUploads.add(`${bagIdStr}|${objectId.toString()}`)
   }
 
-  public async uploadFromEvents(events: IEvent<[Vec<DataObjectId>, UploadParameters, Balance]>[]): Promise<void> {
+  public queueUploadsFromEvents(events: IEvent<[Vec<DataObjectId>, UploadParameters, Balance]>[]): void {
+    let queuedUploads = 0
     events.map(({ data: [objectIds, uploadParams] }) => {
-      objectIds.forEach((objectId) => this.queueUpload(uploadParams.bagId, objectId))
+      objectIds.forEach((objectId) => {
+        this.queueUpload(uploadParams.bagId, objectId)
+        ++queuedUploads
+      })
     })
-    await this.processQueuedUploads()
+    this.logger.info(`Added ${queuedUploads} new data object uploads to the upload queue`)
   }
 }

utils/migration-scripts/src/sumer-giza/BaseMigration.ts

@@ -3,6 +3,7 @@ import { KeyringPair } from '@polkadot/keyring/types'
 import { QueryNodeApi } from './sumer-query-node/api'
 import { RuntimeApi } from '../RuntimeApi'
 import { Keyring } from '@polkadot/keyring'
+import { Logger } from 'winston'
 import path from 'path'
 import nodeCleanup from 'node-cleanup'
 import _ from 'lodash'
@@ -37,6 +38,8 @@ export abstract class BaseMigration {
   protected config: BaseMigrationConfig
   protected failedMigrations: Set<number>
   protected idsMap: Map<number, number>
+  protected pendingMigrationIteration: Promise<void> | undefined
+  protected abstract logger: Logger
 
   public constructor({ api, queryNodeApi, config }: BaseMigrationParams) {
     this.api = api
@@ -54,12 +57,14 @@ export abstract class BaseMigration {
 
   public async init(): Promise<void> {
     this.loadMigrationState()
-    nodeCleanup(() => this.saveMigrationState())
+    nodeCleanup(this.onExit.bind(this))
     await this.loadSudoKey()
   }
 
   public abstract run(): Promise<MigrationResult>
 
+  protected abstract migrateBatch(batch: { id: string }[]): Promise<void>
+
   protected getMigrationStateJson(): MigrationStateJson {
     return {
       idsMapEntries: Array.from(this.idsMap.entries()),
@@ -76,7 +81,25 @@ export abstract class BaseMigration {
     }
   }
 
+  protected onExit(exitCode: number | null, signal: string | null): void | false {
+    nodeCleanup.uninstall() // don't call cleanup handler again
+    this.logger.info('Exitting...')
+    if (signal && this.pendingMigrationIteration) {
+      this.logger.info('Waiting for currently pending migration iteration to finalize...')
+      this.pendingMigrationIteration.then(() => {
+        this.saveMigrationState()
+        this.logger.info('Done.')
+        process.kill(process.pid, signal)
+      })
+      return false
+    } else {
+      this.saveMigrationState()
+      this.logger.info('Done.')
+    }
+  }
+
   protected saveMigrationState(): void {
+    this.logger.info('Saving migration state...')
     const stateFilePath = this.getMigrationStateFilePath()
     const migrationState = this.getMigrationStateJson()
     fs.writeFileSync(stateFilePath, JSON.stringify(migrationState, undefined, 2))
@@ -92,22 +115,63 @@ export abstract class BaseMigration {
     }
   }
 
-  protected extractFailedSudoAsMigrations<T extends { id: string }>(result: SubmittableResult, batch: T[]): void {
+  protected async executeBatchMigration<T extends { id: string }>(batch: T[]): Promise<void> {
+    this.pendingMigrationIteration = this.migrateBatch(batch)
+    await this.pendingMigrationIteration
+    this.pendingMigrationIteration = undefined
+  }
+
+  /**
+   * Extract failed migrations (entity ids) from batch transaction result.
+   * Assumptions:
+   * - Each entity is migrated with a constant number of calls (2 by default: balnces.transferKeepAlive and sudo.sudoAs)
+   * - Ordering of the entities in the `batch` array matches the ordering of the batched calls through which they are migrated
+   * - Last call for each entity is always sudo.sudoAs
+   * - There is only one sudo.sudoAs call per entity
+   *
+   * Entity migration is considered failed if sudo.sudoAs call failed or was not executed at all, regardless of
+   * the result of any of the previous calls associated with that entity migration.
+   * (This means that regardless of whether balnces.transferKeepAlive failed and interrupted the batch or balnces.transferKeepAlive
+   * succeeded, but sudo.sudoAs failed - in both cases the migration is considered failed and should be fully re-executed on
+   * the next script run)
+   */
+  protected extractFailedMigrations<T extends { id: string }>(
+    result: SubmittableResult,
+    batch: T[],
+    callsPerEntity = 2
+  ): void {
     const { api } = this
+    const batchInterruptedEvent = api.findEvent(result, 'utility', 'BatchInterrupted')
     const sudoAsDoneEvents = api.findEvents(result, 'sudo', 'SudoAsDone')
-    if (sudoAsDoneEvents.length !== batch.length) {
-      throw new Error(`Could not extract failed migrations from: ${JSON.stringify(result.toHuman())}`)
+    const numberOfSuccesfulCalls = batchInterruptedEvent
+      ? batchInterruptedEvent.data[0].toNumber()
+      : callsPerEntity * batch.length
+    const numberOfMigratedEntites = Math.floor(numberOfSuccesfulCalls / callsPerEntity)
+    if (sudoAsDoneEvents.length !== numberOfMigratedEntites) {
+      throw new Error(
+        `Unexpected number of SudoAsDone events (expected: ${numberOfMigratedEntites}, got: ${sudoAsDoneEvents.length})! ` +
+          `Could not extract failed migrations from: ${JSON.stringify(result.toHuman())}`
+      )
     }
     const failedIds: number[] = []
-    sudoAsDoneEvents.forEach(({ data: [sudoAsDone] }, i) => {
-      if (sudoAsDone.isFalse) {
-        const id = parseInt(batch[i].id)
-        failedIds.push(id)
-        this.failedMigrations.add(id)
+    batch.forEach((entity, i) => {
+      const entityId = parseInt(entity.id)
+      if (i >= numberOfMigratedEntites || sudoAsDoneEvents[i].data[0].isFalse) {
+        failedIds.push(entityId)
+        this.failedMigrations.add(entityId)
       }
     })
+
+    if (batchInterruptedEvent) {
+      this.logger.error(
+        `Batch interrupted at call ${numberOfSuccesfulCalls}: ${this.api.formatDispatchError(
+          batchInterruptedEvent.data[1]
+        )}`
+      )
+    }
+
     if (failedIds.length) {
-      console.error(`Failed to migrate:`, failedIds)
+      this.logger.error(`Failed to migrate:`, { failedIds })
     }
   }
 

utils/migration-scripts/src/sumer-giza/ChannelsMigration.ts

@@ -8,6 +8,8 @@ import { CHANNEL_AVATAR_TARGET_SIZE, CHANNEL_COVER_TARGET_SIZE } from './ImageRe
 import { ChannelId } from '@joystream/types/common'
 import _ from 'lodash'
 import { MigrationResult } from './BaseMigration'
+import { Logger } from 'winston'
+import { createLogger } from '../logging'
 
 export type ChannelsMigrationConfig = AssetsMigrationConfig & {
   channelIds: number[]
@@ -29,11 +31,13 @@ export class ChannelMigration extends AssetsMigration {
   protected config: ChannelsMigrationConfig
   protected videoIds: number[] = []
   protected forcedChannelOwner: { id: string; controllerAccount: string } | undefined
+  protected logger: Logger
 
   public constructor(params: ChannelsMigrationParams) {
     super(params)
     this.config = params.config
     this.forcedChannelOwner = params.forcedChannelOwner
+    this.logger = createLogger(this.name)
   }
 
   private getChannelOwnerMember({ id, ownerMember }: ChannelFieldsFragment) {
@@ -48,21 +52,46 @@ export class ChannelMigration extends AssetsMigration {
     return ownerMember
   }
 
+  protected async migrateBatch(channels: ChannelFieldsFragment[]): Promise<void> {
+    const { api } = this
+    const txs = _.flatten(await Promise.all(channels.map((c) => this.prepareChannel(c))))
+    const result = await api.sendExtrinsic(this.sudo, api.tx.utility.batch(txs))
+    const channelCreatedEvents = api.findEvents(result, 'content', 'ChannelCreated')
+    const newChannelIds: ChannelId[] = channelCreatedEvents.map((e) => e.data[1])
+    if (channelCreatedEvents.length !== channels.length) {
+      this.extractFailedMigrations(result, channels)
+    }
+    const newChannelMapEntries: [number, number][] = []
+    let newChannelIdIndex = 0
+    channels.forEach(({ id }) => {
+      if (this.failedMigrations.has(parseInt(id))) {
+        return
+      }
+      const newChannelId = newChannelIds[newChannelIdIndex++].toNumber()
+      this.idsMap.set(parseInt(id), newChannelId)
+      newChannelMapEntries.push([parseInt(id), newChannelId])
+    })
+    if (newChannelMapEntries.length) {
+      this.logger.info('Channel map entries added!', { newChannelMapEntries })
+      const dataObjectsUploadedEvents = this.api.findEvents(result, 'storage', 'DataObjectsUploaded')
+      this.assetsManager.queueUploadsFromEvents(dataObjectsUploadedEvents)
+    }
+  }
+
   public async run(): Promise<ChannelsMigrationResult> {
     await this.init()
     const {
-      api,
       config: { channelIds, channelBatchSize },
     } = this
     const ids = channelIds.sort((a, b) => a - b)
     while (ids.length) {
       const idsBatch = ids.splice(0, channelBatchSize)
-      console.log(`Fetching a batch of ${idsBatch.length} channels...`)
+      this.logger.info(`Fetching a batch of ${idsBatch.length} channels...`)
       const channelsBatch = (await this.queryNodeApi.getChannelsByIds(idsBatch)).sort(
         (a, b) => parseInt(a.id) - parseInt(b.id)
       )
       if (channelsBatch.length < idsBatch.length) {
-        console.error(
+        this.logger.warn(
           `Some channels were not be found: ${_.difference(
             idsBatch,
             channelsBatch.map((c) => parseInt(c.id))
@@ -71,34 +100,14 @@ export class ChannelMigration extends AssetsMigration {
       }
       const channelsToMigrate = channelsBatch.filter((c) => !this.idsMap.has(parseInt(c.id)))
       if (channelsToMigrate.length < channelsBatch.length) {
-        console.log(
+        this.logger.info(
           `${channelsToMigrate.length ? 'Some' : 'All'} channels in this batch were already migrated ` +
             `(${channelsBatch.length - channelsToMigrate.length}/${channelsBatch.length})`
         )
       }
       if (channelsToMigrate.length) {
-        const txs = _.flatten(await Promise.all(channelsToMigrate.map((c) => this.prepareChannel(c))))
-        const result = await api.sendExtrinsic(this.sudo, api.tx.utility.batch(txs))
-        const channelCreatedEvents = api.findEvents(result, 'content', 'ChannelCreated')
-        const newChannelIds: ChannelId[] = channelCreatedEvents.map((e) => e.data[1])
-        if (channelCreatedEvents.length !== channelsToMigrate.length) {
-          this.extractFailedSudoAsMigrations(result, channelsToMigrate)
-        }
-        const dataObjectsUploadedEvents = api.findEvents(result, 'storage', 'DataObjectsUploaded')
-        const newChannelMapEntries: [number, number][] = []
-        let newChannelIdIndex = 0
-        channelsToMigrate.forEach(({ id }) => {
-          if (this.failedMigrations.has(parseInt(id))) {
-            return
-          }
-          const newChannelId = newChannelIds[newChannelIdIndex++].toNumber()
-          this.idsMap.set(parseInt(id), newChannelId)
-          newChannelMapEntries.push([parseInt(id), newChannelId])
-        })
-        if (newChannelMapEntries.length) {
-          console.log('Channel map entries added!', newChannelMapEntries)
-          await this.assetsManager.uploadFromEvents(dataObjectsUploadedEvents)
-        }
+        await this.executeBatchMigration(channelsToMigrate)
+        await this.assetsManager.processQueuedUploads()
       }
       const videoIdsToMigrate: number[] = channelsBatch.reduce<number[]>(
         (res, { id, videos }) => (this.idsMap.has(parseInt(id)) ? res.concat(videos.map((v) => parseInt(v.id))) : res),
@@ -106,7 +115,7 @@ export class ChannelMigration extends AssetsMigration {
       )
       this.videoIds = this.videoIds.concat(videoIdsToMigrate)
       if (videoIdsToMigrate.length) {
-        console.log(`Added ${videoIdsToMigrate.length} video ids to the list of videos to migrate`)
+        this.logger.info(`Added ${videoIdsToMigrate.length} video ids to the list of videos to migrate`)
       }
     }
     return {

utils/migration-scripts/src/sumer-giza/ContentMigration.ts

@@ -34,7 +34,7 @@ export class ContentMigration {
 
   private async getForcedChannelOwner(): Promise<{ id: string; controllerAccount: string } | undefined> {
     const { forceChannelOwnerMemberId } = this.config
-    if (forceChannelOwnerMemberId) {
+    if (forceChannelOwnerMemberId !== undefined) {
       const ownerMember = await this.api.query.members.membershipById(forceChannelOwnerMemberId)
       if (ownerMember.isEmpty) {
         throw new Error(`Membership by id ${forceChannelOwnerMemberId} not found!`)

utils/migration-scripts/src/sumer-giza/VideosMigration.ts

@@ -8,6 +8,8 @@ import moment from 'moment'
 import { VIDEO_THUMB_TARGET_SIZE } from './ImageResizer'
 import { AssetsMigration, AssetsMigrationConfig, AssetsMigrationParams } from './AssetsMigration'
 import { MigrationResult } from './BaseMigration'
+import { Logger } from 'winston'
+import { createLogger } from '../logging'
 
 export type VideosMigrationConfig = AssetsMigrationConfig & {
   videoBatchSize: number
@@ -26,6 +28,7 @@ export class VideosMigration extends AssetsMigration {
   protected channelsMap: Map<number, number>
   protected videoIds: number[]
   protected forcedChannelOwner: { id: string; controllerAccount: string } | undefined
+  protected logger: Logger
 
   public constructor({ api, queryNodeApi, config, videoIds, channelsMap, forcedChannelOwner }: VideosMigrationParams) {
     super({ api, queryNodeApi, config })
@@ -33,6 +36,7 @@ export class VideosMigration extends AssetsMigration {
     this.channelsMap = channelsMap
     this.videoIds = videoIds
     this.forcedChannelOwner = forcedChannelOwner
+    this.logger = createLogger(this.name)
   }
 
   private getNewChannelId(oldChannelId: number): number {
@@ -43,58 +47,62 @@ export class VideosMigration extends AssetsMigration {
     return newChannelId
   }
 
+  protected async migrateBatch(videos: VideoFieldsFragment[]): Promise<void> {
+    const { api } = this
+    const txs = _.flatten(await Promise.all(videos.map((v) => this.prepareVideo(v))))
+    const result = await api.sendExtrinsic(this.sudo, api.tx.utility.batch(txs))
+    const videoCreatedEvents = api.findEvents(result, 'content', 'VideoCreated')
+    const newVideoIds: VideoId[] = videoCreatedEvents.map((e) => e.data[2])
+    if (videoCreatedEvents.length !== videos.length) {
+      this.extractFailedMigrations(result, videos)
+    }
+    const newVideoMapEntries: [number, number][] = []
+    let newVideoIdIndex = 0
+    videos.forEach(({ id }) => {
+      if (this.failedMigrations.has(parseInt(id))) {
+        return
+      }
+      const newVideoId = newVideoIds[newVideoIdIndex++].toNumber()
+      this.idsMap.set(parseInt(id), newVideoId)
+      newVideoMapEntries.push([parseInt(id), newVideoId])
+    })
+    if (newVideoMapEntries.length) {
+      this.logger.info('Video map entries added!', { newVideoMapEntries })
+      const dataObjectsUploadedEvents = api.findEvents(result, 'storage', 'DataObjectsUploaded')
+      this.assetsManager.queueUploadsFromEvents(dataObjectsUploadedEvents)
+    }
+  }
+
   public async run(): Promise<MigrationResult> {
     await this.init()
     const {
-      api,
       videoIds,
       config: { videoBatchSize },
     } = this
     const idsToMigrate = videoIds.filter((id) => !this.idsMap.has(id)).sort((a, b) => a - b)
     if (idsToMigrate.length < videoIds.length) {
       const alreadyMigratedVideosNum = videoIds.length - idsToMigrate.length
-      console.log(
+      this.logger.info(
         (idsToMigrate.length ? `${alreadyMigratedVideosNum}/${videoIds.length}` : 'All') +
           ' videos already migrated, skippping...'
       )
     }
     while (idsToMigrate.length) {
       const idsBatch = idsToMigrate.splice(0, videoBatchSize)
-      console.log(`Fetching a batch of ${idsBatch.length} videos...`)
+      this.logger.info(`Fetching a batch of ${idsBatch.length} videos...`)
       const videosBatch = (await this.queryNodeApi.getVideosByIds(idsBatch)).sort(
         (a, b) => parseInt(a.id) - parseInt(b.id)
       )
       if (videosBatch.length < idsBatch.length) {
-        console.error(
+        this.logger.warn(
           `Some videos were not be found: ${_.difference(
             idsBatch,
             videosBatch.map((v) => parseInt(v.id))
           )}`
         )
       }
-      const txs = _.flatten(await Promise.all(videosBatch.map((v) => this.prepareVideo(v))))
-      const result = await api.sendExtrinsic(this.sudo, api.tx.utility.batch(txs))
-      const videoCreatedEvents = api.findEvents(result, 'content', 'VideoCreated')
-      const newVideoIds: VideoId[] = videoCreatedEvents.map((e) => e.data[2])
-      if (videoCreatedEvents.length !== videosBatch.length) {
-        this.extractFailedSudoAsMigrations(result, videosBatch)
-      }
-
-      const dataObjectsUploadedEvents = api.findEvents(result, 'storage', 'DataObjectsUploaded')
-      const newVideoMapEntries: [number, number][] = []
-      let newVideoIdIndex = 0
-      videosBatch.forEach(({ id }) => {
-        if (this.failedMigrations.has(parseInt(id))) {
-          return
-        }
-        const newVideoId = newVideoIds[newVideoIdIndex++].toNumber()
-        this.idsMap.set(parseInt(id), newVideoId)
-        newVideoMapEntries.push([parseInt(id), newVideoId])
-      })
-      if (newVideoMapEntries.length) {
-        console.log('Video map entries added!', newVideoMapEntries)
-        await this.assetsManager.uploadFromEvents(dataObjectsUploadedEvents)
-      }
+      await this.executeBatchMigration(videosBatch)
+      await this.assetsManager.processQueuedUploads()
     }
     return this.getResult()
   }

utils/migration-scripts/src/sumer-giza/sumer-query-node/api.ts

@@ -30,12 +30,15 @@ import {
   VideoFieldsFragment,
   WorkerFieldsFragment,
 } from './generated/queries'
+import { Logger } from 'winston'
+import { createLogger } from '../../logging'
 
 export class QueryNodeApi {
   private endpoint: string
   private apolloClient: ApolloClient<NormalizedCacheObject>
   private retryAttempts: number
   private retryIntervalMs: number
+  private logger: Logger
 
   public constructor(endpoint: string, retryAttempts = 5, retryIntervalMs = 5000) {
     this.endpoint = endpoint
@@ -46,6 +49,7 @@ export class QueryNodeApi {
       cache: new InMemoryCache(),
       defaultOptions: { query: { fetchPolicy: 'no-cache', errorPolicy: 'all' } },
     })
+    this.logger = createLogger('Query Node Api')
   }
 
   private async query<T>(queryFunc: () => Promise<ApolloQueryResult<T>>): Promise<ApolloQueryResult<T>> {
@@ -56,11 +60,11 @@ export class QueryNodeApi {
         return result
       } catch (e) {
         if (e instanceof Error && isApolloError(e) && e.networkError) {
-          console.error(`Query node (${this.endpoint}) network error: ${e.networkError.message}`)
+          this.logger.error(`${this.endpoint} network error: ${e.networkError.message}`)
           if (attempts++ > this.retryAttempts) {
             throw new Error(`Maximum number of query retry attempts reached for ${this.endpoint}`)
           }
-          console.log(`Retrying in ${this.retryIntervalMs}ms...`)
+          this.logger.info(`Retrying in ${this.retryIntervalMs}ms...`)
           await new Promise((resolve) => setTimeout(resolve, this.retryIntervalMs))
         } else {
           throw e