istio / old_mixer_repo

Deprecated home of Istio's Mixer and its adapters, now in istio/istio's mixer dir
https://github.com/istio/istio/tree/master/mixer
Apache License 2.0
67 stars 93 forks source link

Reduce attrgenmgr logging #1514

Closed douglas-reid closed 6 years ago

douglas-reid commented 6 years ago

Fixes https://github.com/istio/mixer/issues/1513.

Release note:


This change is Reviewable

geeknoid commented 6 years ago

/lgtm

istio-merge-robot commented 6 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: geeknoid

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these OWNERS Files: - ~~[OWNERS](https://github.com/istio/mixer/blob/master/OWNERS)~~ [geeknoid] You can indicate your approval by writing `/approve` in a comment You can cancel your approval by writing `/approve cancel` in a comment
istio-merge-robot commented 6 years ago

/test all [submit-queue is verifying that this PR is safe to merge]

codecov[bot] commented 6 years ago

Codecov Report

Merging #1514 into master will decrease coverage by 0.02%. The diff coverage is 25%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1514      +/-   ##
==========================================
- Coverage   84.57%   84.55%   -0.03%     
==========================================
  Files         122      122              
  Lines       11997    11999       +2     
==========================================
- Hits        10147    10146       -1     
- Misses       1649     1651       +2     
- Partials      201      202       +1
Impacted Files Coverage Δ
pkg/aspect/attrgenmgr.go 91.54% <25%> (-4.11%) :arrow_down:

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 2382662...adf25a3. Read the comment docs.

istio-merge-robot commented 6 years ago

Automatic merge from submit-queue.