cloudflare / cloudflare-ingress-controller

A Kubernetes ingress controller for Cloudflare's Argo Tunnels
Apache License 2.0
364 stars 55 forks source link

issues-139: incorrect server to subset port match #141

Closed mattalberts closed 5 years ago

mattalberts commented 5 years ago

It is sufficient to verify existience of ready addresses without attempting to match ports.

resolves #139

codecov-io commented 5 years ago

Codecov Report

Merging #141 into master will increase coverage by 0.22%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #141      +/-   ##
=========================================
+ Coverage   58.97%   59.2%   +0.22%     
=========================================
  Files          16      16              
  Lines        1097    1103       +6     
=========================================
+ Hits          647     653       +6     
  Misses        428     428              
  Partials       22      22
Impacted Files Coverage Δ
internal/k8s/audit.go 100% <100%> (ø) :arrow_up:
internal/argotunnel/translator.go 62.5% <100%> (ø) :arrow_up:

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 1715dca...43142c5. Read the comment docs.