Skip to content

Commit 93e6f36

Browse files
committed
fix: apply limit also in dry run
Signed-off-by: Umberto Sgueglia <usgueglia@contractor.linuxfoundation.org>
1 parent e12aad1 commit 93e6f36

1 file changed

Lines changed: 10 additions & 1 deletion

File tree

services/apps/script_executor_worker/src/bin/recalculate-all-affiliations.ts

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -267,11 +267,20 @@ async function main() {
267267

268268
if (brokenMembers.length > 0) {
269269
if (opts.dryRun) {
270-
for (const { memberId, activeOrgIds } of brokenMembers) {
270+
const loggedSoFar = totalBroken - brokenMembers.length
271+
const remaining = opts.limit !== null ? opts.limit - loggedSoFar : brokenMembers.length
272+
const toLog = brokenMembers.slice(0, remaining)
273+
for (const { memberId, activeOrgIds } of toLog) {
271274
log.info(
272275
`[DRY RUN] memberUpdate | memberId: ${memberId} | activeOrgs: ${activeOrgIds.length}`,
273276
)
274277
}
278+
if (opts.limit !== null && loggedSoFar + toLog.length >= opts.limit) {
279+
log.info(`Limit of ${opts.limit} members reached.`)
280+
hasMore = false
281+
cursor = lastMemberId
282+
continue
283+
}
275284
} else {
276285
const triggeredSoFar = totalSucceeded + totalFailed
277286
const remaining = opts.limit !== null ? opts.limit - triggeredSoFar : brokenMembers.length

0 commit comments

Comments
 (0)