opensearch-project / opensearch-metrics

OpenSearch Metrics
https://metrics.opensearch.org
Apache License 2.0
2 stars 4 forks source link

Support component name filter for release metrics #67

Closed prudhvigodithi closed 1 month ago

prudhvigodithi commented 1 month ago

Description

Support component name for release metrics. This should be able to help user filter the documents with repo name or component name. In the following example find the result either by anomaly-detection-dashboards-plugin or by anomalyDetectionDashboards should yield the same results.

 "_source": {
    "release_issue_exists": true,
    "release_notes": false,
    "version_increment": true,
    "release_issue": "https://github.com/opensearch-project/anomaly-detection-dashboards-plugin/issues/802",
    "autocut_issues_open": 0,
    "repository": "anomaly-detection-dashboards-plugin",
    "release_state": "closed",
    "version": "2.15.0",
    "pulls_open": 0,
    "release_owner_exists": false,
    "current_date": "2024-08-26T21:25:20.901042281",
    "component": "anomalyDetectionDashboards",
    "release_branch": true,
    "issues_closed": 5,
    "pulls_closed": 5,
    "id": "1c8b312c-dd16-3443-988e-d3525b2b7715",
    "issues_open": 0,
    "release_version": "2.15.0",
    "release_owners": []
  },

Issues Resolved

Part of https://github.com/opensearch-project/opensearch-metrics/issues/51

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 56.75676% with 16 lines in your changes missing coverage. Please review.

Project coverage is 82.96%. Comparing base (8f33f15) to head (310431b). Report is 1 commits behind head on main.

Files Patch % Lines
.../opensearchmetrics/metrics/MetricsCalculation.java 25.00% 15 Missing :warning:
...rchmetrics/metrics/release/ReleaseRepoFetcher.java 93.33% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #67 +/- ## ============================================ + Coverage 76.70% 82.96% +6.26% - Complexity 190 202 +12 ============================================ Files 53 53 Lines 1189 1186 -3 Branches 49 48 -1 ============================================ + Hits 912 984 +72 + Misses 254 180 -74 + Partials 23 22 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

prudhvigodithi commented 1 month ago

Merging this as this PR improved the code coverage to 82.96%.