New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ref: Remove references to deprecated settings #66532
Conversation
settings.KAFKA_INGEST_PERFORMANCE_METRICS and settings.KAFKA_INGEST_METRICS are deprecated.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #66532 +/- ##
=======================================
Coverage 84.30% 84.31%
=======================================
Files 5311 5311
Lines 237358 237421 +63
Branches 41058 41075 +17
=======================================
+ Hits 200107 200170 +63
Misses 37032 37032
Partials 219 219
|
@@ -116,8 +113,7 @@ def get_ingest_config( | |||
MetricsIngestConfiguration( | |||
db_backend=IndexerStorage.MOCK, | |||
db_backend_options={}, | |||
input_topic="topic", | |||
output_topic="output-topic", | |||
output_topic=Topic.SNUBA_METRICS, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This type was wrong, it's only used in tests though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How did it pass mypy ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's on the excluded from checking list 😆
@@ -116,8 +113,7 @@ def get_ingest_config( | |||
MetricsIngestConfiguration( | |||
db_backend=IndexerStorage.MOCK, | |||
db_backend_options={}, | |||
input_topic="topic", | |||
output_topic="output-topic", | |||
output_topic=Topic.SNUBA_METRICS, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How did it pass mypy ?
settings.KAFKA_INGEST_PERFORMANCE_METRICS and settings.KAFKA_INGEST_METRICS are deprecated.
The IngestConfiguration.input_topic was never used anywhere.