Skip to content

Commit c13c3ed

Browse files
committed
test(observability-map): name the environment on the queues page failure logs
Throwaway PR to verify the observability-map CI comment reports an improvement. Not for merge.
1 parent a681f21 commit c13c3ed

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

  • apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.queues

apps/webapp/app/routes/_app.orgs.$organizationSlug.projects.$projectParam.env.$envParam.queues/route.tsx

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,10 @@ export const loader = async ({ request, params }: LoaderFunctionArgs) => {
217217
};
218218
}
219219
} catch (error) {
220-
logger.warn("Queue list metrics unavailable, rendering without them", { error });
220+
logger.warn("Queue list metrics unavailable, rendering without them", {
221+
error,
222+
environmentId: environment.id,
223+
});
221224
}
222225
}
223226

@@ -228,7 +231,10 @@ export const loader = async ({ request, params }: LoaderFunctionArgs) => {
228231
try {
229232
allocation = await new QueueAllocationPresenter().call({ environment });
230233
} catch (error) {
231-
logger.warn("Queue allocation summary unavailable, rendering without it", { error });
234+
logger.warn("Queue allocation summary unavailable, rendering without it", {
235+
error,
236+
environmentId: environment.id,
237+
});
232238
}
233239
}
234240

0 commit comments

Comments
 (0)