fix(dlq): Metrics ignored messages should be Sentry Errors

[rahul-kumar-saini]

Overview

• Metrics DLQ is behind a runtime config right now
• Warns but should error if we encounter an invalid message and the DLQ is disabled

[codecov-commenter]

Codecov Report

Merging #2698 (35c512a) into master (c6f97e9) will decrease coverage by 0.06%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #2698      +/-   ##
==========================================
- Coverage   92.75%   92.69%   -0.07%     
==========================================
  Files         613      613              
  Lines       29043    29043              
==========================================
- Hits        26938    26920      -18     
- Misses       2105     2123      +18     

Impacted Files	Coverage Δ
snuba/datasets/metrics_bucket_processor.py	84.15% <0.00%> (ø)
snuba/settings/settings_distributed.py	0.00% <0.00%> (-100.00%)	⬇️
snuba/settings/settings_test_distributed.py	0.00% <0.00%> (-100.00%)	⬇️
...ts/0010_groupedmessages_onpremise_compatibility.py	95.55% <0.00%> (-4.45%)	⬇️
snuba/optimize.py	87.50% <0.00%> (-3.75%)	⬇️
snuba/migrations/table_engines.py	95.50% <0.00%> (-3.38%)	⬇️
...nsactions_onpremise_fix_orderby_and_partitionby.py	81.33% <0.00%> (-2.67%)	⬇️
tests/migrations/test_runner_individual.py	91.20% <0.00%> (-2.20%)	⬇️
snuba/clusters/cluster.py	94.61% <0.00%> (-1.54%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c6f97e9...35c512a. Read the comment docs.