Skip to content

Conversation

@jkoelker
Copy link
Contributor

@jkoelker jkoelker commented Apr 9, 2020

@jkoelker jkoelker force-pushed the fwd_vip_weight_regression branch from 054405d to 9e5d3c4 Compare April 9, 2020 20:46
@codecov
Copy link

codecov bot commented Apr 9, 2020

Codecov Report

Merging #198 into master will decrease coverage by 0.57%.
The diff coverage is 51.92%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #198      +/-   ##
==========================================
- Coverage   66.60%   66.02%   -0.58%     
==========================================
  Files          55       55              
  Lines        3731     3759      +28     
==========================================
- Hits         2485     2482       -3     
- Misses       1246     1277      +31     
Flag Coverage Δ
#ct 45.99% <28.84%> (-0.78%) ⬇️
#ct2 45.99% <28.84%> (-0.78%) ⬇️
#eunit 24.79% <40.38%> (+0.40%) ⬆️
Impacted Files Coverage Δ
apps/dcos_l4lb/src/dcos_l4lb_mesos_poller.erl 80.39% <ø> (+3.26%) ⬆️
apps/dcos_l4lb/src/dcos_l4lb_ipvs_mgr.erl 58.97% <7.14%> (-13.06%) ⬇️
apps/dcos_l4lb/src/dcos_l4lb_mgr.erl 78.54% <60.00%> (-3.27%) ⬇️
...ps/dcos_l4lb/src/dcos_l4lb_lashup_vip_listener.erl 77.19% <100.00%> (+1.26%) ⬆️
apps/dcos_net/src/dcos_net_sysmon.erl 56.66% <0.00%> (-3.34%) ⬇️
apps/dcos_l4lb/src/dcos_l4lb_ipset_mgr.erl 67.82% <0.00%> (-0.87%) ⬇️
apps/dcos_dns/src/dcos_dns_key_mgr.erl 44.15% <0.00%> (+2.59%) ⬆️

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 1670c37...dc8680e. Read the comment docs.

@jkoelker jkoelker force-pushed the fwd_vip_weight_regression branch from dca2783 to e8c4d88 Compare April 13, 2020 19:52
@jkoelker jkoelker force-pushed the fwd_vip_weight_regression branch from e8c4d88 to f8addb3 Compare April 13, 2020 20:21
@jkoelker jkoelker self-assigned this Apr 16, 2020
@jkoelker jkoelker requested review from jongiddy and wavesoft April 21, 2020 22:00
@jkoelker jkoelker force-pushed the fwd_vip_weight_regression branch from 24aa5e0 to ae2affe Compare April 21, 2020 22:03
Adds testing that the different backend selection and manupulation
functions work with and without the weight being specified. Confirms
that if an agent populates a VIP using the old code that the new code
won't crash.
@jkoelker jkoelker force-pushed the fwd_vip_weight_regression branch from ae2affe to dc8680e Compare April 21, 2020 22:46
Copy link

@wavesoft wavesoft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Per my review in the back-port : #197 (review)

@jkoelker jkoelker merged commit d35033b into dcos:master Apr 22, 2020
@jkoelker jkoelker deleted the fwd_vip_weight_regression branch April 22, 2020 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants