LoggingContext.tag being lost on metrics and traces

[MadLittleMods]

As discovered during the discussion in #15708 (comment),

Background

LoggingContext.tag was introduced to differentiate what kind of sync was happening in our metrics. The only place we populate it is for this use case even though it could be used generally:

synapse/synapse/handlers/sync.py

Lines 348 to 357 in a424318

	if since_token is None:
	sync_type = "initial_sync"
	elif full_state:
	sync_type = "full_state_sync"
	else:
	sync_type = "incremental_sync"
	context = current_context()
	if context:
	context.tag = sync_type

We also use it in the tracing with SynapseTags.REQUEST_TAG which says that it's "used to distinguish full vs incremental syncs, etc".

Problem

If I look at the Prometheus data, I don't see any data for matrix.org. I only see it for one other instance we track which is running Synapse 1.83.0. Has it broken since then?

And if I look at the traces, request_tag is always set to the default empty string when I look at those SyncRestServlet traces (also checked RoomInitialSyncRestServlet).

It seems like the context is being lost somewhere.