chore: add type to providers_upload_duration_seconds metrics

snapshot-labs · Sep 3, 2023 · ea07bd3 · ea07bd3
1 parent abc03dd
commit ea07bd3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/metrics.ts b/src/metrics.ts
@@ -41,7 +41,7 @@ providersImageCount.set(IMAGE_PROVIDERS.filter(p => providersMap[p].isConfigured
 export const timeProvidersUpload = new client.Histogram({
   name: 'providers_upload_duration_seconds',
   help: "Duration in seconds of provider's upload requests.",
-  labelNames: ['name'],
+  labelNames: ['name', 'type'],
   buckets: [0.5, 1, 2, 5, 10, 15]
 });
 

diff --git a/src/providers/index.ts b/src/providers/index.ts
@@ -6,15 +6,15 @@ export default function uploadToProviders(providers: string[], params: any) {
 
   return Promise.any(
     configuredProviders.map(async name => {
-      const end = timeProvidersUpload.startTimer({ name });
+      const type = params instanceof Buffer ? 'image' : 'json';
+      const end = timeProvidersUpload.startTimer({ name, type });
 
       try {
         countOpenProvidersRequest.inc({ name });
 
         const result = await providersMap[name].set(params);
         const size = (params instanceof Buffer ? params : Buffer.from(JSON.stringify(params)))
           .length;
-        const type = params instanceof Buffer ? 'image' : 'json';
         providersUploadSize.inc({ name, type }, size);
 
         return result;