@fluidframework/test-utils 2.0.0-internal.3.0.1 → 2.0.0-internal.3.1.0

package/src/loaderContainerTracker.ts

@@ -6,7 +6,11 @@ import { assert } from "@fluidframework/common-utils";
 import { IContainer, IDeltaQueue, IHostLoader } from "@fluidframework/container-definitions";
 import { Container } from "@fluidframework/container-loader";
 import { canBeCoalescedByService } from "@fluidframework/driver-utils";
-import { IDocumentMessage, ISequencedDocumentMessage, MessageType } from "@fluidframework/protocol-definitions";
+import {
+	IDocumentMessage,
+	ISequencedDocumentMessage,
+	MessageType,
+} from "@fluidframework/protocol-definitions";
 import { debug } from "./debug";
 import { IOpProcessingController } from "./testObjectProvider";
 import { timeoutAwait, timeoutPromise } from "./timeoutUtils";
@@ -15,574 +19,632 @@ const debugOp = debug.extend("ops");
 const debugWait = debug.extend("wait");
 
 interface ContainerRecord {
-    // A short number for debug output
-    index: number;
+	// A short number for debug output
+	index: number;
 
-    // LoaderContainerTracker paused state
-    paused: boolean;
+	// LoaderContainerTracker paused state
+	paused: boolean;
 
-    // Tracking trailing no-op that may or may be acked by the server so we can discount them
-    // See issue #5629
-    startTrailingNoOps: number;
-    trailingNoOps: number;
+	// Tracking trailing no-op that may or may be acked by the server so we can discount them
+	// See issue #5629
+	startTrailingNoOps: number;
+	trailingNoOps: number;
 
-    // Track last proposal to ensure no unresolved proposal
-    lastProposal: number;
+	// Track last proposal to ensure no unresolved proposal
+	lastProposal: number;
 }
 
 export class LoaderContainerTracker implements IOpProcessingController {
-    private readonly containers = new Map<IContainer, ContainerRecord>();
-    private lastProposalSeqNum: number = 0;
-
-    constructor(private readonly syncSummarizerClients: boolean = false) {}
-
-    /**
-     * Add a loader to start to track any container created from them
-     * @param loader - loader to start tracking any container created.
-     */
-    public add<LoaderType extends IHostLoader>(loader: LoaderType) {
-        // TODO: Expose Loader API to able to intercept container creation (See issue #5114)
-        const patch = <T, C extends IContainer>(fn: (...args) => Promise<C>) => {
-            const boundFn = fn.bind(loader);
-            return async (...args: T[]) => {
-                const container = await boundFn(...args);
-                this.addContainer(container);
-                return container;
-            };
-        };
-        /* eslint-disable @typescript-eslint/unbound-method */
-        loader.resolve = patch(loader.resolve);
-        loader.createDetachedContainer = patch(loader.createDetachedContainer);
-        loader.rehydrateDetachedContainerFromSnapshot = patch(loader.rehydrateDetachedContainerFromSnapshot);
-        /* eslint-enable @typescript-eslint/unbound-method */
-    }
-
-    /**
-     * Utility function to add container to be tracked.
-     *
-     * @param container - container to add
-     */
-    private addContainer(container: IContainer) {
-        // ignore summarizer
-        if (!container.deltaManager.clientDetails.capabilities.interactive && !this.syncSummarizerClients) { return; }
-
-        // don't add container that is already tracked
-        if (this.containers.has(container)) { return; }
-
-        const record = {
-            index: this.containers.size,
-            paused: false,
-            startTrailingNoOps: 0,
-            trailingNoOps: 0,
-            lastProposal: 0,
-        };
-        this.containers.set(container, record);
-        this.trackTrailingNoOps(container, record);
-        this.trackLastProposal(container);
-        this.setupTrace(container, record.index);
-    }
-
-    /**
-     * Keep track of the trailing NoOp that was sent so we can discount them in the clientSequenceNumber tracking.
-     * The server might coalesce them with other ops, or a single NoOp, or delay it if it don't think it is necessary.
-     *
-     * @param container - the container to track
-     * @param record - the record to update the trailing op information
-     */
-    private trackTrailingNoOps(container: IContainer, record: ContainerRecord) {
-        container.deltaManager.outbound.on("op", (messages) => {
-            for (const msg of messages) {
-                if (canBeCoalescedByService(msg)) {
-                    // Track the NoOp that was sent.
-                    if (record.trailingNoOps === 0) {
-                        // record the starting sequence number of the trailing no ops if we haven't been tracking yet.
-                        record.startTrailingNoOps = msg.clientSequenceNumber;
-                    }
-                    record.trailingNoOps++;
-                } else {
-                    // Other ops has been sent. We would like to see those ack'ed, so no more need to track NoOps
-                    record.trailingNoOps = 0;
-                }
-            }
-        });
-
-        container.deltaManager.inbound.on("push", (message) => {
-            // Received the no op back, update the record if we are tracking
-            if (canBeCoalescedByService(message)
-                && message.clientId === (container as Container).clientId
-                && record.trailingNoOps !== 0
-                && record.startTrailingNoOps <= message.clientSequenceNumber
-            ) {
-                // NoOp might have coalesced and skipped ahead some sequence number
-                // update the record and skip ahead as well
-                const oldStartTrailingNoOps = record.startTrailingNoOps;
-                record.startTrailingNoOps = message.clientSequenceNumber + 1;
-                record.trailingNoOps -= (record.startTrailingNoOps - oldStartTrailingNoOps);
-            }
-        });
-
-        container.on("disconnected", () => {
-            // reset on disconnect.
-            record.trailingNoOps = 0;
-        });
-    }
-
-    private trackLastProposal(container: IContainer) {
-        container.on("codeDetailsProposed", (value, proposal) => {
-            if (proposal.sequenceNumber > this.lastProposalSeqNum) {
-                this.lastProposalSeqNum = proposal.sequenceNumber;
-            }
-        });
-    }
-
-    /**
-     * Reset the tracker, closing all containers and stop tracking them.
-     */
-    public reset() {
-        this.lastProposalSeqNum = 0;
-        for (const container of this.containers.keys()) {
-            container.close();
-        }
-        this.containers.clear();
-
-        // REVIEW: do we need to unpatch the loaders?
-    }
-
-    /**
-     * Ensure all tracked containers are synchronized
-     */
-    public async ensureSynchronized(...containers: IContainer[]): Promise<void> {
-        await this.processSynchronized(undefined, ...containers);
-    }
-
-    /**
-     * Ensure all tracked containers are synchronized with a time limit
-     */
-    public async ensureSynchronizedWithTimeout?(timeoutDuration: number | undefined, ...containers: IContainer[]) {
-        await this.processSynchronized(timeoutDuration, ...containers);
-    }
-
-    /**
-     * Make sure all the tracked containers are synchronized.
-     *
-     * No isDirty (non-readonly) containers
-     *
-     * No extra clientId in quorum of any container that is not tracked and still opened.
-     *
-     * - i.e. no pending Join/Leave message.
-     *
-     * No unresolved proposal (minSeqNum \>= lastProposalSeqNum)
-     *
-     * lastSequenceNumber of all container is the same
-     *
-     * clientSequenceNumberObserved is the same as clientSequenceNumber sent
-     *
-     * - this overlaps with !isDirty, but include task scheduler ops.
-     *
-     * - Trailing NoOp is tracked and don't count as pending ops.
-     */
-    private async processSynchronized(timeoutDuration: number | undefined, ...containers: IContainer[]) {
-        const resumed = this.resumeProcessing(...containers);
-
-        let waitingSequenceNumberSynchronized = false;
-        // eslint-disable-next-line no-constant-condition
-        while (true) {
-            const containersToApply = this.getContainers(containers);
-            if (containersToApply.length === 0) { break; }
-
-            // Ignore readonly dirty containers, because it can't sent up and nothing can be done about it being dirty
-            const dirtyContainers = containersToApply.filter((c) => {
-                const { deltaManager, isDirty } = c;
-                return deltaManager.readOnlyInfo.readonly !== true && isDirty;
-            });
-            if (dirtyContainers.length === 0) {
-                // Wait for all the leave messages
-                const pendingClients = this.getPendingClients(containersToApply);
-                if (pendingClients.length === 0) {
-                    if (this.isSequenceNumberSynchronized(containersToApply)) {
-                        // done, we are in sync
-                        break;
-                    }
-                    if (!waitingSequenceNumberSynchronized) {
-                        // Only write it out once
-                        waitingSequenceNumberSynchronized = true;
-                        debugWait("Waiting for sequence number synchronized");
-                        await timeoutAwait(this.waitForAnyInboundOps(containersToApply), {
-                            errorMsg: "Timeout on waiting for sequence number synchronized",
-                        });
-                    }
-                } else {
-                    waitingSequenceNumberSynchronized = false;
-                    await timeoutAwait(this.waitForPendingClients(pendingClients), {
-                        errorMsg: "Timeout on waiting for pending join or leave op",
-                    });
-                }
-            } else {
-                // Wait for all the containers to be saved
-                // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-                debugWait(`Waiting container to be saved ${dirtyContainers.map((c) => this.containers.get(c)!.index)}`);
-                waitingSequenceNumberSynchronized = false;
-                await Promise.all(dirtyContainers.map(async (c) => Promise.race(
-                    [timeoutPromise(
-                        (resolve) => c.once("saved", () => resolve()),
-                        {
-                            errorMsg: "Timeout on waiting a container to be saved",
-                        },
-                    ),
-                    new Promise((resolve) => c.once("closed", resolve)),
-                    ],
-                )));
-            }
-
-            // yield a turn to allow side effect of the ops we just processed execute before we check again
-            await new Promise<void>((resolve) => { setTimeout(resolve, 0); });
-        }
-
-        // Pause all container that was resumed
-        // don't call pause if resumed is empty and pause everything, which is not what we want
-        if (resumed.length !== 0) {
-            await timeoutAwait(this.pauseProcessing(...resumed), {
-                errorMsg: "Timeout on waiting for pausing all resumed containers",
-            });
-        }
-
-        debugWait("Synchronized");
-    }
-
-    /**
-     * Utility to calculate the set of clientId per container in quorum that is NOT associated with
-     * any container we tracked, indicating there is a pending join or leave op that we need to wait.
-     *
-     * @param containersToApply - the set of containers to check
-     */
-    private getPendingClients(containersToApply: IContainer[]) {
-        // All the clientId we track should be a superset of the quorum, otherwise, we are missing
-        // leave messages
-        const openedDocuments = Array.from(this.containers.keys()).filter((c) => !c.closed);
-        const openedClientId = openedDocuments.map((container) => (container as Container).clientId);
-
-        const pendingClients: [IContainer, Set<string>][] = [];
-        containersToApply.forEach((container) => {
-            const pendingClientId = new Set<string>();
-            const quorum = container.getQuorum();
-            quorum.getMembers().forEach((client, clientId) => {
-                // ignore summarizer
-                if (!client.client.details.capabilities.interactive && !this.syncSummarizerClients) { return; }
-                if (!openedClientId.includes(clientId)) {
-                    pendingClientId.add(clientId);
-                }
-            });
-
-            if (pendingClientId.size !== 0) {
-                pendingClients.push([container, pendingClientId]);
-            }
-        });
-        return pendingClients;
-    }
-
-    /**
-     * Utility to check synchronization based on sequence number
-     * See ensureSynchronized for more detail
-     *
-     * @param containersToApply - the set of containers to check
-     */
-    private isSequenceNumberSynchronized(containersToApply: IContainer[]) {
-        // clientSequenceNumber check detects ops in flight, both on the wire and in the outbound queue
-        // We need both client sequence number and isDirty check because:
-        // - Currently isDirty flag ignores ops for task scheduler, so we need the client sequence number check
-        // - But isDirty flags include ops during forceReadonly and disconnected, because we don't submit
-        //   the ops in the first place, clientSequenceNumber is not assigned
-
-        const isClientSequenceNumberSynchronized = containersToApply.every((container) => {
-            if (container.deltaManager.readOnlyInfo.readonly === true) {
-                // Ignore readonly container. the clientSeqNum and clientSeqNumObserved might be out of sync
-                // because we transition to readonly when outbound is not empty or the in transit op got lost
-                return true;
-            }
-            // Note that in read only mode, the op won't be submitted
-            let deltaManager = (container.deltaManager as any);
-            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-            const { trailingNoOps } = this.containers.get(container)!;
-            // Back-compat: clientSequenceNumber & clientSequenceNumberObserved moved to ConnectionManager in 0.53
-            if (!("clientSequenceNumber" in deltaManager)) {
-                deltaManager = deltaManager.connectionManager;
-            }
-            assert("clientSequenceNumber" in deltaManager, "no clientSequenceNumber");
-            assert("clientSequenceNumberObserved" in deltaManager, "no clientSequenceNumber");
-            return deltaManager.clientSequenceNumber ===
-                (deltaManager.clientSequenceNumberObserved as number) + trailingNoOps;
-        });
-
-        if (!isClientSequenceNumberSynchronized) {
-            return false;
-        }
-
-        const minSeqNum = containersToApply[0].deltaManager.minimumSequenceNumber;
-        if (minSeqNum < this.lastProposalSeqNum) {
-            // There is an unresolved proposal
-            return false;
-        }
-
-        // Check to see if all the container has process the same number of ops.
-        const seqNum = containersToApply[0].deltaManager.lastSequenceNumber;
-        return containersToApply.every((c) => c.deltaManager.lastSequenceNumber === seqNum);
-    }
-
-    /**
-     * Utility to wait for any clientId in quorum that is NOT associated with any container we
-     * tracked, indicating there is a pending join or leave op that we need to wait.
-     *
-     * Note that this function doesn't account for container that got added after we started waiting
-     *
-     * @param containersToApply - the set of containers to wait for any inbound ops for
-     */
-    private async waitForPendingClients(pendingClients: [IContainer, Set<string>][]) {
-        const unconnectedClients =
-            Array.from(this.containers.keys()).filter((c) => !c.closed && !(c as Container).connected);
-        return Promise.all(pendingClients.map(async ([container, pendingClientId]) => {
-            return new Promise<void>((resolve) => {
-                const cleanup = () => {
-                    unconnectedClients.forEach((c) => c.off("connected", handler));
-                    container.getQuorum().off("removeMember", handler);
-                };
-                const handler = (clientId: string) => {
-                    pendingClientId.delete(clientId);
-                    if (pendingClientId.size === 0) {
-                        cleanup();
-                        resolve();
-                    }
-                };
-                // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
-                const index = this.containers.get(container)!.index;
-                debugWait(`${index}: Waiting for pending clients ${Array.from(pendingClientId.keys())}`);
-                unconnectedClients.forEach((c) => c.on("connected", handler));
-                container.getQuorum().on("removeMember", handler);
-                container.on("closed", () => {
-                    cleanup();
-                    resolve();
-                });
-            });
-        }));
-    }
-
-    /**
-     * Utility to wait for any inbound ops from a set of containers
-     * @param containersToApply - the set of containers to wait for any inbound ops for
-     */
-    private async waitForAnyInboundOps(containersToApply: IContainer[]) {
-        return new Promise<void>((resolve) => {
-            const handler = () => {
-                containersToApply.map((c) => {
-                    c.deltaManager.inbound.off("push", handler);
-                });
-                resolve();
-            };
-            containersToApply.map((c) => {
-                c.deltaManager.inbound.on("push", handler);
-            });
-        });
-    }
-
-    /**
-     * Resume all queue activities on all paused tracked containers and return them
-     */
-    public resumeProcessing(...containers: IContainer[]) {
-        const resumed: IContainer[] = [];
-        const containersToApply = this.getContainers(containers);
-        for (const container of containersToApply) {
-            const record = this.containers.get(container);
-            if (record?.paused === true) {
-                debugWait(`${record.index}: container resumed`);
-                container.deltaManager.inbound.resume();
-                container.deltaManager.outbound.resume();
-                resumed.push(container);
-                record.paused = false;
-            }
-        }
-        return resumed;
-    }
-
-    /**
-     * Pause all queue activities on the containers given, or all tracked containers
-     * Any containers given that is not tracked will be ignored.
-     */
-    public async pauseProcessing(...containers: IContainer[]) {
-        const pauseP: Promise<void>[] = [];
-        const containersToApply = this.getContainers(containers);
-        for (const container of containersToApply) {
-            const record = this.containers.get(container);
-            if (record !== undefined && !record.paused) {
-                debugWait(`${record.index}: container paused`);
-                pauseP.push(container.deltaManager.inbound.pause());
-                pauseP.push(container.deltaManager.outbound.pause());
-                record.paused = true;
-            }
-        }
-        await Promise.all(pauseP);
-    }
-
-    /**
-     * Pause all queue activities on all tracked containers, and resume only
-     * inbound to process ops until it is idle. All queues are left in the paused state
-     * after the function
-     */
-    public async processIncoming(...containers: IContainer[]) {
-        return this.processQueue(containers, (container) => container.deltaManager.inbound);
-    }
-
-    /**
-     * Pause all queue activities on all tracked containers, and resume only
-     * outbound to process ops until it is idle. All queues are left in the paused state
-     * after the function
-     */
-    public async processOutgoing(...containers: IContainer[]) {
-        return this.processQueue(containers, (container) => container.deltaManager.outbound);
-    }
-
-    /**
-     * Implementation of processIncoming and processOutgoing
-     */
-    private async processQueue<U>(containers: IContainer[], getQueue: (container: IContainer) => IDeltaQueue<U>) {
-        await this.pauseProcessing(...containers);
-        const resumed: IDeltaQueue<U>[] = [];
-
-        const containersToApply = this.getContainers(containers);
-        const inflightTracker = new Map<IContainer, number>();
-        const cleanup: (() => void)[] = [];
-        for (const container of containersToApply) {
-            const queue = getQueue(container);
-
-            // track the outgoing ops (if any) to make sure they make the round trip to at least to the same client
-            // to make sure they are sequenced.
-            cleanup.push(this.setupInOutTracker(container, inflightTracker));
-            queue.resume();
-            resumed.push(queue);
-        }
-
-        while (resumed.some((queue) => !queue.idle)) {
-            debugWait("Wait until queue is idle");
-            await new Promise<void>((resolve) => { setTimeout(resolve, 0); });
-        }
-
-        // Make sure all the op that we sent out are acked first
-        // This is no op if we are processing incoming
-        if (inflightTracker.size) {
-            debugWait("Wait for inflight ops");
-            do {
-                await this.waitForAnyInboundOps(containersToApply);
-            } while (inflightTracker.size);
-        }
-
-        // remove the handlers
-        cleanup.forEach((clean) => clean());
-
-        await Promise.all(resumed.map(async (queue) => queue.pause()));
-    }
-
-    /**
-     * Utility to set up listener to track the outbound ops until it round trip back
-     * Returns a function to remove the handler after it is done.
-     *
-     * @param container - the container to setup
-     * @param inflightTracker - a map to track the clientSequenceNumber per container it expect to get ops back
-     */
-    private setupInOutTracker(container: IContainer, inflightTracker: Map<IContainer, number>) {
-        const outHandler = (messages: IDocumentMessage[]) => {
-            for (const message of messages) {
-                if (!canBeCoalescedByService(message)) {
-                    inflightTracker.set(container, message.clientSequenceNumber);
-                }
-            }
-        };
-        const inHandler = (message: ISequencedDocumentMessage) => {
-            if (!canBeCoalescedByService(message)
-                && message.clientId === (container as Container).clientId
-                && inflightTracker.get(container) === message.clientSequenceNumber) {
-                inflightTracker.delete(container);
-            }
-        };
-
-        container.deltaManager.outbound.on("op", outHandler);
-        container.deltaManager.inbound.on("push", inHandler);
-
-        return () => {
-            container.deltaManager.outbound.off("op", outHandler);
-            container.deltaManager.inbound.off("push", inHandler);
-        };
-    }
-
-    /**
-     * Setup debug traces for connection and ops
-     */
-    private setupTrace(container: IContainer, index: number) {
-        if (debugOp.enabled) {
-            const getContentsString = (type: string, msgContents: any) => {
-                try {
-                    if (type !== MessageType.Operation) {
-                        if (typeof msgContents === "string") { return msgContents; }
-                        return JSON.stringify(msgContents);
-                    }
-                    let address = "";
-
-                    // contents comes in the wire as JSON string ("push" event)
-                    // But already parsed when apply ("op" event)
-                    let contents = typeof msgContents === "string" ?
-                        JSON.parse(msgContents) : msgContents;
-                    while (contents !== undefined && contents !== null) {
-                        if (contents.contents?.address !== undefined) {
-                            address += `/${contents.contents.address}`;
-                            contents = contents.contents.contents;
-                        } else if (contents.content?.address !== undefined) {
-                            address += `/${contents.content.address}`;
-                            contents = contents.content.contents;
-                        } else {
-                            break;
-                        }
-                    }
-                    if (address) {
-                        return `${address} ${JSON.stringify(contents)}`;
-                    }
-                    return JSON.stringify(contents);
-                } catch (e: any) {
-                    return `${e.message}: ${e.stack}`;
-                }
-            };
-            debugOp(`${index}: ADD: clientId: ${(container as Container).clientId}`);
-            container.deltaManager.outbound.on("op", (messages) => {
-                for (const msg of messages) {
-                    debugOp(`${index}: OUT:          `
-                        + `cli: ${msg.clientSequenceNumber.toString().padStart(3)} `
-                        + `rsq: ${msg.referenceSequenceNumber.toString().padStart(3)} `
-                        + `${msg.type} ${getContentsString(msg.type, msg.contents)}`);
-                }
-            });
-            const getInboundHandler = (type: string) => {
-                return (msg: ISequencedDocumentMessage) => {
-                    const clientSeq = msg.clientId === (container as Container).clientId ?
-                        `cli: ${msg.clientSequenceNumber.toString().padStart(3)}` : "        ";
-                    debugOp(`${index}: ${type}: seq: ${msg.sequenceNumber.toString().padStart(3)} `
-                        + `${clientSeq} min: ${msg.minimumSequenceNumber.toString().padStart(3)} `
-                        + `${msg.type} ${getContentsString(msg.type, msg.contents)}`);
-                };
-            };
-            container.deltaManager.inbound.on("push", getInboundHandler("IN "));
-            container.deltaManager.inbound.on("op", getInboundHandler("OP "));
-            container.deltaManager.on("connect", (details) => {
-                debugOp(`${index}: CON: clientId: ${details.clientId}`);
-            });
-            container.deltaManager.on("disconnect", (reason) => {
-                debugOp(`${index}: DIS: ${reason}`);
-            });
-        }
-    }
-
-    /**
-     * Filter out the opened containers based on param.
-     * @param containers - The container to filter to.  If the array is empty, it means don't filter and return
-     * all open containers.
-     */
-    private getContainers(containers: IContainer[]) {
-        const containersToApply = containers.length === 0 ? Array.from(this.containers.keys()) : containers;
-        return containersToApply.filter((container) => !container.closed);
-    }
+	private readonly containers = new Map<IContainer, ContainerRecord>();
+	private lastProposalSeqNum: number = 0;
+
+	constructor(private readonly syncSummarizerClients: boolean = false) {}
+
+	/**
+	 * Add a loader to start to track any container created from them
+	 * @param loader - loader to start tracking any container created.
+	 */
+	public add<LoaderType extends IHostLoader>(loader: LoaderType) {
+		// TODO: Expose Loader API to able to intercept container creation (See issue #5114)
+		const patch = <T, C extends IContainer>(fn: (...args) => Promise<C>) => {
+			const boundFn = fn.bind(loader);
+			return async (...args: T[]) => {
+				const container = await boundFn(...args);
+				this.addContainer(container);
+				return container;
+			};
+		};
+		/* eslint-disable @typescript-eslint/unbound-method */
+		loader.resolve = patch(loader.resolve);
+		loader.createDetachedContainer = patch(loader.createDetachedContainer);
+		loader.rehydrateDetachedContainerFromSnapshot = patch(
+			loader.rehydrateDetachedContainerFromSnapshot,
+		);
+		/* eslint-enable @typescript-eslint/unbound-method */
+	}
+
+	/**
+	 * Utility function to add container to be tracked.
+	 *
+	 * @param container - container to add
+	 */
+	private addContainer(container: IContainer) {
+		// ignore summarizer
+		if (
+			!container.deltaManager.clientDetails.capabilities.interactive &&
+			!this.syncSummarizerClients
+		) {
+			return;
+		}
+
+		// don't add container that is already tracked
+		if (this.containers.has(container)) {
+			return;
+		}
+
+		const record = {
+			index: this.containers.size,
+			paused: false,
+			startTrailingNoOps: 0,
+			trailingNoOps: 0,
+			lastProposal: 0,
+		};
+		this.containers.set(container, record);
+		this.trackTrailingNoOps(container, record);
+		this.trackLastProposal(container);
+		this.setupTrace(container, record.index);
+	}
+
+	/**
+	 * Keep track of the trailing NoOp that was sent so we can discount them in the clientSequenceNumber tracking.
+	 * The server might coalesce them with other ops, or a single NoOp, or delay it if it don't think it is necessary.
+	 *
+	 * @param container - the container to track
+	 * @param record - the record to update the trailing op information
+	 */
+	private trackTrailingNoOps(container: IContainer, record: ContainerRecord) {
+		container.deltaManager.outbound.on("op", (messages) => {
+			for (const msg of messages) {
+				if (canBeCoalescedByService(msg)) {
+					// Track the NoOp that was sent.
+					if (record.trailingNoOps === 0) {
+						// record the starting sequence number of the trailing no ops if we haven't been tracking yet.
+						record.startTrailingNoOps = msg.clientSequenceNumber;
+					}
+					record.trailingNoOps++;
+				} else {
+					// Other ops has been sent. We would like to see those ack'ed, so no more need to track NoOps
+					record.trailingNoOps = 0;
+				}
+			}
+		});
+
+		container.deltaManager.inbound.on("push", (message) => {
+			// Received the no op back, update the record if we are tracking
+			if (
+				canBeCoalescedByService(message) &&
+				message.clientId === (container as Container).clientId &&
+				record.trailingNoOps !== 0 &&
+				record.startTrailingNoOps <= message.clientSequenceNumber
+			) {
+				// NoOp might have coalesced and skipped ahead some sequence number
+				// update the record and skip ahead as well
+				const oldStartTrailingNoOps = record.startTrailingNoOps;
+				record.startTrailingNoOps = message.clientSequenceNumber + 1;
+				record.trailingNoOps -= record.startTrailingNoOps - oldStartTrailingNoOps;
+			}
+		});
+
+		container.on("disconnected", () => {
+			// reset on disconnect.
+			record.trailingNoOps = 0;
+		});
+	}
+
+	private trackLastProposal(container: IContainer) {
+		container.on("codeDetailsProposed", (value, proposal) => {
+			if (proposal.sequenceNumber > this.lastProposalSeqNum) {
+				this.lastProposalSeqNum = proposal.sequenceNumber;
+			}
+		});
+	}
+
+	/**
+	 * Reset the tracker, closing all containers and stop tracking them.
+	 */
+	public reset() {
+		this.lastProposalSeqNum = 0;
+		for (const container of this.containers.keys()) {
+			container.close();
+		}
+		this.containers.clear();
+
+		// REVIEW: do we need to unpatch the loaders?
+	}
+
+	/**
+	 * Ensure all tracked containers are synchronized
+	 */
+	public async ensureSynchronized(...containers: IContainer[]): Promise<void> {
+		await this.processSynchronized(undefined, ...containers);
+	}
+
+	/**
+	 * Ensure all tracked containers are synchronized with a time limit
+	 */
+	public async ensureSynchronizedWithTimeout?(
+		timeoutDuration: number | undefined,
+		...containers: IContainer[]
+	) {
+		await this.processSynchronized(timeoutDuration, ...containers);
+	}
+
+	/**
+	 * Make sure all the tracked containers are synchronized.
+	 *
+	 * No isDirty (non-readonly) containers
+	 *
+	 * No extra clientId in quorum of any container that is not tracked and still opened.
+	 *
+	 * - i.e. no pending Join/Leave message.
+	 *
+	 * No unresolved proposal (minSeqNum \>= lastProposalSeqNum)
+	 *
+	 * lastSequenceNumber of all container is the same
+	 *
+	 * clientSequenceNumberObserved is the same as clientSequenceNumber sent
+	 *
+	 * - this overlaps with !isDirty, but include task scheduler ops.
+	 *
+	 * - Trailing NoOp is tracked and don't count as pending ops.
+	 */
+	private async processSynchronized(
+		timeoutDuration: number | undefined,
+		...containers: IContainer[]
+	) {
+		const resumed = this.resumeProcessing(...containers);
+
+		let waitingSequenceNumberSynchronized = false;
+		// eslint-disable-next-line no-constant-condition
+		while (true) {
+			const containersToApply = this.getContainers(containers);
+			if (containersToApply.length === 0) {
+				break;
+			}
+
+			// Ignore readonly dirty containers, because it can't sent up and nothing can be done about it being dirty
+			const dirtyContainers = containersToApply.filter((c) => {
+				const { deltaManager, isDirty } = c;
+				return deltaManager.readOnlyInfo.readonly !== true && isDirty;
+			});
+			if (dirtyContainers.length === 0) {
+				// Wait for all the leave messages
+				const pendingClients = this.getPendingClients(containersToApply);
+				if (pendingClients.length === 0) {
+					if (this.isSequenceNumberSynchronized(containersToApply)) {
+						// done, we are in sync
+						break;
+					}
+					if (!waitingSequenceNumberSynchronized) {
+						// Only write it out once
+						waitingSequenceNumberSynchronized = true;
+						debugWait("Waiting for sequence number synchronized");
+						await timeoutAwait(this.waitForAnyInboundOps(containersToApply), {
+							errorMsg: "Timeout on waiting for sequence number synchronized",
+						});
+					}
+				} else {
+					waitingSequenceNumberSynchronized = false;
+					await timeoutAwait(this.waitForPendingClients(pendingClients), {
+						errorMsg: "Timeout on waiting for pending join or leave op",
+					});
+				}
+			} else {
+				// Wait for all the containers to be saved
+				debugWait(
+					`Waiting container to be saved ${dirtyContainers.map(
+						// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+						(c) => this.containers.get(c)!.index,
+					)}`,
+				);
+				waitingSequenceNumberSynchronized = false;
+				await Promise.all(
+					dirtyContainers.map(async (c) =>
+						Promise.race([
+							timeoutPromise((resolve) => c.once("saved", () => resolve()), {
+								errorMsg: "Timeout on waiting a container to be saved",
+							}),
+							new Promise((resolve) => c.once("closed", resolve)),
+						]),
+					),
+				);
+			}
+
+			// yield a turn to allow side effect of the ops we just processed execute before we check again
+			await new Promise<void>((resolve) => {
+				setTimeout(resolve, 0);
+			});
+		}
+
+		// Pause all container that was resumed
+		// don't call pause if resumed is empty and pause everything, which is not what we want
+		if (resumed.length !== 0) {
+			await timeoutAwait(this.pauseProcessing(...resumed), {
+				errorMsg: "Timeout on waiting for pausing all resumed containers",
+			});
+		}
+
+		debugWait("Synchronized");
+	}
+
+	/**
+	 * Utility to calculate the set of clientId per container in quorum that is NOT associated with
+	 * any container we tracked, indicating there is a pending join or leave op that we need to wait.
+	 *
+	 * @param containersToApply - the set of containers to check
+	 */
+	private getPendingClients(containersToApply: IContainer[]) {
+		// All the clientId we track should be a superset of the quorum, otherwise, we are missing
+		// leave messages
+		const openedDocuments = Array.from(this.containers.keys()).filter((c) => !c.closed);
+		const openedClientId = openedDocuments.map(
+			(container) => (container as Container).clientId,
+		);
+
+		const pendingClients: [IContainer, Set<string>][] = [];
+		containersToApply.forEach((container) => {
+			const pendingClientId = new Set<string>();
+			const quorum = container.getQuorum();
+			quorum.getMembers().forEach((client, clientId) => {
+				// ignore summarizer
+				if (
+					!client.client.details.capabilities.interactive &&
+					!this.syncSummarizerClients
+				) {
+					return;
+				}
+				if (!openedClientId.includes(clientId)) {
+					pendingClientId.add(clientId);
+				}
+			});
+
+			if (pendingClientId.size !== 0) {
+				pendingClients.push([container, pendingClientId]);
+			}
+		});
+		return pendingClients;
+	}
+
+	/**
+	 * Utility to check synchronization based on sequence number
+	 * See ensureSynchronized for more detail
+	 *
+	 * @param containersToApply - the set of containers to check
+	 */
+	private isSequenceNumberSynchronized(containersToApply: IContainer[]) {
+		// clientSequenceNumber check detects ops in flight, both on the wire and in the outbound queue
+		// We need both client sequence number and isDirty check because:
+		// - Currently isDirty flag ignores ops for task scheduler, so we need the client sequence number check
+		// - But isDirty flags include ops during forceReadonly and disconnected, because we don't submit
+		//   the ops in the first place, clientSequenceNumber is not assigned
+
+		const isClientSequenceNumberSynchronized = containersToApply.every((container) => {
+			if (container.deltaManager.readOnlyInfo.readonly === true) {
+				// Ignore readonly container. the clientSeqNum and clientSeqNumObserved might be out of sync
+				// because we transition to readonly when outbound is not empty or the in transit op got lost
+				return true;
+			}
+			// Note that in read only mode, the op won't be submitted
+			let deltaManager = container.deltaManager as any;
+			// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+			const { trailingNoOps } = this.containers.get(container)!;
+			// Back-compat: clientSequenceNumber & clientSequenceNumberObserved moved to ConnectionManager in 0.53
+			if (!("clientSequenceNumber" in deltaManager)) {
+				deltaManager = deltaManager.connectionManager;
+			}
+			assert("clientSequenceNumber" in deltaManager, "no clientSequenceNumber");
+			assert("clientSequenceNumberObserved" in deltaManager, "no clientSequenceNumber");
+			return (
+				deltaManager.clientSequenceNumber ===
+				(deltaManager.clientSequenceNumberObserved as number) + trailingNoOps
+			);
+		});
+
+		if (!isClientSequenceNumberSynchronized) {
+			return false;
+		}
+
+		const minSeqNum = containersToApply[0].deltaManager.minimumSequenceNumber;
+		if (minSeqNum < this.lastProposalSeqNum) {
+			// There is an unresolved proposal
+			return false;
+		}
+
+		// Check to see if all the container has process the same number of ops.
+		const seqNum = containersToApply[0].deltaManager.lastSequenceNumber;
+		return containersToApply.every((c) => c.deltaManager.lastSequenceNumber === seqNum);
+	}
+
+	/**
+	 * Utility to wait for any clientId in quorum that is NOT associated with any container we
+	 * tracked, indicating there is a pending join or leave op that we need to wait.
+	 *
+	 * Note that this function doesn't account for container that got added after we started waiting
+	 *
+	 * @param containersToApply - the set of containers to wait for any inbound ops for
+	 */
+	private async waitForPendingClients(pendingClients: [IContainer, Set<string>][]) {
+		const unconnectedClients = Array.from(this.containers.keys()).filter(
+			(c) => !c.closed && !(c as Container).connected,
+		);
+		return Promise.all(
+			pendingClients.map(async ([container, pendingClientId]) => {
+				return new Promise<void>((resolve) => {
+					const cleanup = () => {
+						unconnectedClients.forEach((c) => c.off("connected", handler));
+						container.getQuorum().off("removeMember", handler);
+					};
+					const handler = (clientId: string) => {
+						pendingClientId.delete(clientId);
+						if (pendingClientId.size === 0) {
+							cleanup();
+							resolve();
+						}
+					};
+					// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+					const index = this.containers.get(container)!.index;
+					debugWait(
+						`${index}: Waiting for pending clients ${Array.from(
+							pendingClientId.keys(),
+						)}`,
+					);
+					unconnectedClients.forEach((c) => c.on("connected", handler));
+					container.getQuorum().on("removeMember", handler);
+					container.on("closed", () => {
+						cleanup();
+						resolve();
+					});
+				});
+			}),
+		);
+	}
+
+	/**
+	 * Utility to wait for any inbound ops from a set of containers
+	 * @param containersToApply - the set of containers to wait for any inbound ops for
+	 */
+	private async waitForAnyInboundOps(containersToApply: IContainer[]) {
+		return new Promise<void>((resolve) => {
+			const handler = () => {
+				containersToApply.map((c) => {
+					c.deltaManager.inbound.off("push", handler);
+				});
+				resolve();
+			};
+			containersToApply.map((c) => {
+				c.deltaManager.inbound.on("push", handler);
+			});
+		});
+	}
+
+	/**
+	 * Resume all queue activities on all paused tracked containers and return them
+	 */
+	public resumeProcessing(...containers: IContainer[]) {
+		const resumed: IContainer[] = [];
+		const containersToApply = this.getContainers(containers);
+		for (const container of containersToApply) {
+			const record = this.containers.get(container);
+			if (record?.paused === true) {
+				debugWait(`${record.index}: container resumed`);
+				container.deltaManager.inbound.resume();
+				container.deltaManager.outbound.resume();
+				resumed.push(container);
+				record.paused = false;
+			}
+		}
+		return resumed;
+	}
+
+	/**
+	 * Pause all queue activities on the containers given, or all tracked containers
+	 * Any containers given that is not tracked will be ignored.
+	 */
+	public async pauseProcessing(...containers: IContainer[]) {
+		const pauseP: Promise<void>[] = [];
+		const containersToApply = this.getContainers(containers);
+		for (const container of containersToApply) {
+			const record = this.containers.get(container);
+			if (record !== undefined && !record.paused) {
+				debugWait(`${record.index}: container paused`);
+				pauseP.push(container.deltaManager.inbound.pause());
+				pauseP.push(container.deltaManager.outbound.pause());
+				record.paused = true;
+			}
+		}
+		await Promise.all(pauseP);
+	}
+
+	/**
+	 * Pause all queue activities on all tracked containers, and resume only
+	 * inbound to process ops until it is idle. All queues are left in the paused state
+	 * after the function
+	 */
+	public async processIncoming(...containers: IContainer[]) {
+		return this.processQueue(containers, (container) => container.deltaManager.inbound);
+	}
+
+	/**
+	 * Pause all queue activities on all tracked containers, and resume only
+	 * outbound to process ops until it is idle. All queues are left in the paused state
+	 * after the function
+	 */
+	public async processOutgoing(...containers: IContainer[]) {
+		return this.processQueue(containers, (container) => container.deltaManager.outbound);
+	}
+
+	/**
+	 * Implementation of processIncoming and processOutgoing
+	 */
+	private async processQueue<U>(
+		containers: IContainer[],
+		getQueue: (container: IContainer) => IDeltaQueue<U>,
+	) {
+		await this.pauseProcessing(...containers);
+		const resumed: IDeltaQueue<U>[] = [];
+
+		const containersToApply = this.getContainers(containers);
+		const inflightTracker = new Map<IContainer, number>();
+		const cleanup: (() => void)[] = [];
+		for (const container of containersToApply) {
+			const queue = getQueue(container);
+
+			// track the outgoing ops (if any) to make sure they make the round trip to at least to the same client
+			// to make sure they are sequenced.
+			cleanup.push(this.setupInOutTracker(container, inflightTracker));
+			queue.resume();
+			resumed.push(queue);
+		}
+
+		while (resumed.some((queue) => !queue.idle)) {
+			debugWait("Wait until queue is idle");
+			await new Promise<void>((resolve) => {
+				setTimeout(resolve, 0);
+			});
+		}
+
+		// Make sure all the op that we sent out are acked first
+		// This is no op if we are processing incoming
+		if (inflightTracker.size) {
+			debugWait("Wait for inflight ops");
+			do {
+				await this.waitForAnyInboundOps(containersToApply);
+			} while (inflightTracker.size);
+		}
+
+		// remove the handlers
+		cleanup.forEach((clean) => clean());
+
+		await Promise.all(resumed.map(async (queue) => queue.pause()));
+	}
+
+	/**
+	 * Utility to set up listener to track the outbound ops until it round trip back
+	 * Returns a function to remove the handler after it is done.
+	 *
+	 * @param container - the container to setup
+	 * @param inflightTracker - a map to track the clientSequenceNumber per container it expect to get ops back
+	 */
+	private setupInOutTracker(container: IContainer, inflightTracker: Map<IContainer, number>) {
+		const outHandler = (messages: IDocumentMessage[]) => {
+			for (const message of messages) {
+				if (!canBeCoalescedByService(message)) {
+					inflightTracker.set(container, message.clientSequenceNumber);
+				}
+			}
+		};
+		const inHandler = (message: ISequencedDocumentMessage) => {
+			if (
+				!canBeCoalescedByService(message) &&
+				message.clientId === (container as Container).clientId &&
+				inflightTracker.get(container) === message.clientSequenceNumber
+			) {
+				inflightTracker.delete(container);
+			}
+		};
+
+		container.deltaManager.outbound.on("op", outHandler);
+		container.deltaManager.inbound.on("push", inHandler);
+
+		return () => {
+			container.deltaManager.outbound.off("op", outHandler);
+			container.deltaManager.inbound.off("push", inHandler);
+		};
+	}
+
+	/**
+	 * Setup debug traces for connection and ops
+	 */
+	private setupTrace(container: IContainer, index: number) {
+		if (debugOp.enabled) {
+			const getContentsString = (type: string, msgContents: any) => {
+				try {
+					if (type !== MessageType.Operation) {
+						if (typeof msgContents === "string") {
+							return msgContents;
+						}
+						return JSON.stringify(msgContents);
+					}
+					let address = "";
+
+					// contents comes in the wire as JSON string ("push" event)
+					// But already parsed when apply ("op" event)
+					let contents =
+						typeof msgContents === "string" ? JSON.parse(msgContents) : msgContents;
+					while (contents !== undefined && contents !== null) {
+						if (contents.contents?.address !== undefined) {
+							address += `/${contents.contents.address}`;
+							contents = contents.contents.contents;
+						} else if (contents.content?.address !== undefined) {
+							address += `/${contents.content.address}`;
+							contents = contents.content.contents;
+						} else {
+							break;
+						}
+					}
+					if (address) {
+						return `${address} ${JSON.stringify(contents)}`;
+					}
+					return JSON.stringify(contents);
+				} catch (e: any) {
+					return `${e.message}: ${e.stack}`;
+				}
+			};
+			debugOp(`${index}: ADD: clientId: ${(container as Container).clientId}`);
+			container.deltaManager.outbound.on("op", (messages) => {
+				for (const msg of messages) {
+					debugOp(
+						`${index}: OUT:          ` +
+							`cli: ${msg.clientSequenceNumber.toString().padStart(3)} ` +
+							`rsq: ${msg.referenceSequenceNumber.toString().padStart(3)} ` +
+							`${msg.type} ${getContentsString(msg.type, msg.contents)}`,
+					);
+				}
+			});
+			const getInboundHandler = (type: string) => {
+				return (msg: ISequencedDocumentMessage) => {
+					const clientSeq =
+						msg.clientId === (container as Container).clientId
+							? `cli: ${msg.clientSequenceNumber.toString().padStart(3)}`
+							: "        ";
+					debugOp(
+						`${index}: ${type}: seq: ${msg.sequenceNumber.toString().padStart(3)} ` +
+							`${clientSeq} min: ${msg.minimumSequenceNumber
+								.toString()
+								.padStart(3)} ` +
+							`${msg.type} ${getContentsString(msg.type, msg.contents)}`,
+					);
+				};
+			};
+			container.deltaManager.inbound.on("push", getInboundHandler("IN "));
+			container.deltaManager.inbound.on("op", getInboundHandler("OP "));
+			container.deltaManager.on("connect", (details) => {
+				debugOp(`${index}: CON: clientId: ${details.clientId}`);
+			});
+			container.deltaManager.on("disconnect", (reason) => {
+				debugOp(`${index}: DIS: ${reason}`);
+			});
+		}
+	}
+
+	/**
+	 * Filter out the opened containers based on param.
+	 * @param containers - The container to filter to.  If the array is empty, it means don't filter and return
+	 * all open containers.
+	 */
+	private getContainers(containers: IContainer[]) {
+		const containersToApply =
+			containers.length === 0 ? Array.from(this.containers.keys()) : containers;
+		return containersToApply.filter((container) => !container.closed);
+	}
 }