Skip to content

Commit 924e052

Browse files
committed
fix(metrics): fix metrics counter
fix metrics counter
1 parent bc39263 commit 924e052

File tree

4 files changed

+6
-5
lines changed

4 files changed

+6
-5
lines changed

commitlint.config.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ const scope = [
55
"route",
66
"task",
77
"validator",
8+
"metrics"
89
];
910

1011
module.exports = {

controller/sentry/metrics/celery.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@
22

33

44
def celery_merger(old, new):
5-
return Counter(old) + Counter(new)
5+
return dict(Counter(old) + Counter(new))

controller/sentry/metrics/wsgi.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@
22

33

44
def wsgi_merger(old, new):
5-
return Counter(old) + Counter(new)
5+
return dict(Counter(old) + Counter(new))

controller/sentry/views.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,12 @@ def retrieve(self, request, *args, **kwargs):
3333
def metrics(
3434
self, request, pk=None, metric_name=None
3535
): # pylint: disable=W0613,C0103
36-
serializer = MetricSerializer(data=request.data)
37-
serializer.is_valid(raise_exception=True)
38-
3936
app, _ = App.objects.get_or_create(reference=pk)
4037
metric, _ = Metric.objects.get_or_create(app=app, type=metric_name, defaults={})
4138

39+
serializer = MetricSerializer(data=request.data)
40+
serializer.is_valid(raise_exception=True)
41+
4242
metric.merge(serializer.validated_data["data"])
4343
metric.save()
4444
return Response(MetricSerializer(instance=metric).data)

0 commit comments

Comments
 (0)