Skip to content

Commit

Permalink
common, agent: fix active deployment query
Browse files Browse the repository at this point in the history
  • Loading branch information
dwerner committed Jan 14, 2025
1 parent 3dc32d9 commit 74f1185
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 28 deletions.
41 changes: 15 additions & 26 deletions packages/indexer-agent/src/agent.ts
Original file line number Diff line number Diff line change
Expand Up @@ -329,33 +329,24 @@ export class Agent {
sequentialTimerMap(
{ logger, milliseconds: requestIntervalLarge },
async () => {
const deployments = await this.multiNetworks.map(
async ({ network }) => {
if (
this.deploymentManagement === DeploymentManagementMode.AUTO ||
network.networkMonitor.poiDisputeMonitoringEnabled()
) {
logger.trace('Fetching active deployments')
const assignments =
await this.graphNode.subgraphDeploymentsAssignments(
SubgraphStatus.ACTIVE,
)
return assignments.map(assignment => assignment.id)
} else {
logger.info(
"Skipping fetching active deployments fetch since DeploymentManagementMode = 'manual' and POI tracking is disabled",
)
return []
}
},
)
return deployments.values
if (this.deploymentManagement === DeploymentManagementMode.AUTO) {
logger.debug('Fetching active deployments')
const assignments =
await this.graphNode.subgraphDeploymentsAssignments(
SubgraphStatus.ACTIVE,
)
return assignments.map(assignment => assignment.id)
} else {
logger.info(
"Skipping fetching active deployments fetch since DeploymentManagementMode = 'manual' and POI tracking is disabled",
)
return []
}
},
{
onError: error =>
logger.warn(
`Failed to obtain active deployments, trying again later`,
{ error },
`Failed to obtain active deployments, trying again later ${error}`,
),
},
)
Expand Down Expand Up @@ -554,7 +545,6 @@ export class Agent {
networkDeploymentAllocationDecisions,
}).tryMap(
async ({ indexingRules, networkDeploymentAllocationDecisions }) => {
logger.trace('Resolving target deployments')
const targetDeploymentIDs: Set<SubgraphDeploymentID> =
consolidateAllocationDecisions(networkDeploymentAllocationDecisions)

Expand All @@ -576,8 +566,7 @@ export class Agent {
{
onError: error =>
logger.warn(
`Failed to obtain target deployments, trying again later`,
{ error },
`Failed to obtain target deployments, trying again later ${error}`,
),
},
)
Expand Down
4 changes: 2 additions & 2 deletions packages/indexer-common/src/sequential-timer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ export function sequentialTimerReduce<T, U>(
}
}
// initial call
setTimeout(work, milliseconds)
work()
return output
}

Expand Down Expand Up @@ -152,6 +152,6 @@ export function sequentialTimerMap<U>(
}

// initial call
setTimeout(work, milliseconds)
work()
return output
}

0 comments on commit 74f1185

Please sign in to comment.