Skip to content
This repository has been archived by the owner on Jan 24, 2023. It is now read-only.

Metrics: Fix issue where wrong job can be matched up when there are multiple jobs #3057

Merged
merged 1 commit into from
Sep 25, 2018

Conversation

nwmac
Copy link
Contributor

@nwmac nwmac commented Sep 24, 2018

This fixes an issue when a metrics endpoint provides metrics for multiple endpoints - the last job will always been used.

Also adds extra metrics_job metadata that we will display in the UI to help with debugging.

@cfdreddbot
Copy link

Hey nwmac!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

@nwmac nwmac self-assigned this Sep 24, 2018
@codecov
Copy link

codecov bot commented Sep 24, 2018

Codecov Report

Merging #3057 into v2-master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           v2-master    #3057   +/-   ##
==========================================
  Coverage      71.02%   71.02%           
==========================================
  Files            607      607           
  Lines          26250    26250           
  Branches        5940     5940           
==========================================
  Hits           18645    18645           
  Misses          7605     7605

@KlapTrap KlapTrap merged commit 256259f into v2-master Sep 25, 2018
@KlapTrap KlapTrap deleted the metrics-fix-job-matching branch September 25, 2018 08:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants