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

fix(distributor): Include event filter for project, stage, service #6972

Merged
merged 1 commit into from
Feb 23, 2022

Conversation

bacherfl
Copy link
Member

Forward port of #6968 (see description of this PR for an explanation of this solution)
Closes #6964

…6968)

* fix(distributor): Include event filter for project, stage, service

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>

* fix typo

Signed-off-by: Florian Bacher <florian.bacher@dynatrace.com>
@bacherfl bacherfl requested a review from a team as a code owner February 23, 2022 11:25
@sonarcloud
Copy link

sonarcloud bot commented Feb 23, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@codecov
Copy link

codecov bot commented Feb 23, 2022

Codecov Report

Merging #6972 (7016add) into master (f98155c) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #6972      +/-   ##
==========================================
+ Coverage   57.69%   57.72%   +0.03%     
==========================================
  Files         510      510              
  Lines       29391    29406      +15     
  Branches     1378     1378              
==========================================
+ Hits        16956    16974      +18     
+ Misses      11214    11212       -2     
+ Partials     1221     1220       -1     
Impacted Files Coverage Δ
distributor/pkg/lib/events/poller.go 68.88% <100.00%> (+6.22%) ⬆️
distributor/pkg/lib/controlplane/uniformlog.go 76.56% <0.00%> (+4.68%) ⬆️
Flag Coverage Δ
distributor 72.21% <100.00%> (+0.67%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@bacherfl bacherfl merged commit 6ab050d into master Feb 23, 2022
@bacherfl bacherfl deleted the fix/6964/distributor-filter-fp branch February 23, 2022 13:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

distributor: Polling the events does not include the filters for project, stage, and service
2 participants