instr(pdb): Upgrade warning to error

[jjbayer]

Temporarily update warning about .NET with Windows PDB to an error so we can aggregate by request.scope on the sentry error to see how many projects are affected.

ref: getsentry/symbolic#871

#skip-changelog

[Dav1dde]

Should we instead have a metric for this?

[jjbayer]

Should we instead have a metric for this?

@Dav1dde I want to enumerate the projects that are affected. Currently we're seeing this warning in our logs ~1000 / day, so volume should be manageable.

[Dav1dde]

@jjbayer 👍 I assume the project id gets added through the context/scope from the caller?

[jjbayer]

I assume the project id gets added through the context/scope from the caller?

Yes, configure_scope attaches tags to the sentry scope

symbolicator/crates/symbolicator-native/src/caches/ppdb_caches.rs

Lines 123 to 128 in 1f3a74e

	object_handle.configure_scope();
	let ppdb_obj = match object_handle.object() {
	Object::PortablePdb(ppdb_obj) => ppdb_obj,
	_ => {
	tracing::error!("Trying to symbolicate a .NET event with a non-PPDB object file");

and the "scope" tag contains the project ID IIUC:

symbolicator/crates/symbolicator-service/src/objects/data_cache.rs

Lines 83 to 86 in 1f3a74e

	impl ConfigureScope for ObjectHandle {
	fn to_scope(&self, scope: &mut ::sentry::Scope) {
	self.object_id.to_scope(scope);
	scope.set_tag("object_file.scope", &self.scope);