Skip to content

tp: allow TRACE_FILE as a clock name in manifest clocks blocks - #7877

Open
LalitMaganti wants to merge 1 commit into
dev/lalitm/manifest-offset-fixfrom
dev/lalitm/manifest-trace-file-clock
Open

LalitMaganti wants to merge 1 commit into
dev/lalitm/manifest-offset-fixfrom
dev/lalitm/manifest-trace-file-clock

Conversation

@LalitMaganti

@LalitMaganti LalitMaganti commented Oct 9, 2026 •

Copy link
Copy Markdown
Member

The file's private per-file timeline could only be selected by omitting
clock / sync_to.clock. Accept "TRACE_FILE" as an explicit spelling of
the same thing on both sides.

Also accept it for trace_time.clock (with a required trace_time.file and
no machine) to make that file's private timeline the trace time.

@LalitMaganti
LalitMaganti requested a review from a team as a code owner October 9, 2026 21:09
@LalitMaganti
LalitMaganti added this pull request to stack #7878 October 9, 2026 21:10
@github-actions

github-actions Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

@LalitMaganti
LalitMaganti force-pushed the dev/lalitm/manifest-trace-file-clock branch 2 times, most recently from 3e3f9df to 162f5eb Compare October 9, 2026 21:18
The file's private per-file timeline could only be selected by omitting
`clock` / `sync_to.clock`. Accept "TRACE_FILE" as an explicit spelling of
the same thing on both sides.

Also accept it for trace_time.clock (with a required trace_time.file and
no machine) to make that file's private timeline the trace time.
@LalitMaganti
LalitMaganti force-pushed the dev/lalitm/manifest-trace-file-clock branch from 162f5eb to d63733d Compare October 9, 2026 21:19
@LalitMaganti

Copy link
Copy Markdown
Member Author

@eladalon1983 cc

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant