Merge pull request #148 from osmocom/osmocom/fix-select

exometer_report: Fix static subscriber spec with Extra param
Feuerlabs · Sep 6, 2024 · d1d8b08 · d1d8b08
2 parents f65ec52 + f9c7abc
commit d1d8b08
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/exometer_report.erl b/src/exometer_report.erl
@@ -1731,9 +1731,9 @@ init_subscriber({apply, {M, F, A}}) ->
     lists:foreach(fun(Sub) ->
                           init_subscriber(Sub)
                   end, apply(M, F, A));
-init_subscriber({select, Expr}) when tuple_size(Expr)==3;
-                                     tuple_size(Expr)==4;
-                                     tuple_size(Expr)==5 ->
+init_subscriber({select, Expr}) when tuple_size(Expr)==4;
+                                     tuple_size(Expr)==5;
+                                     tuple_size(Expr)==6 ->
     {Pattern, Reporter, DataPoint, Interval, Retry, Extra} =
         case Expr of
             {P, R, D, I} -> {P, R, D, I, true, undefined};
@@ -1750,7 +1750,7 @@ init_subscriber({select, Expr}) when tuple_size(Expr)==3;
 
 init_subscriber(Other) ->
     ?log(warning, "Incorrect static subscriber spec ~p. "
-             "Use { Reporter, Metric, DataPoint, Interval [, Extra ]}~n",
+             "Use { Reporter, Metric, DataPoint [, Interval [, Extra ] ]}~n",
              [ Other ]).
 
 get_reporter_status(R) ->