Skip to content

Fix for flaky DNS test in the switch setup#918

Merged
grafnu merged 2 commits into
faucetsdn:masterfrom
grafnu:dnsfix
Aug 31, 2021
Merged

Fix for flaky DNS test in the switch setup#918
grafnu merged 2 commits into
faucetsdn:masterfrom
grafnu:dnsfix

Conversation

@grafnu
Copy link
Copy Markdown
Collaborator

@grafnu grafnu commented Aug 31, 2021

No description provided.

@grafnu grafnu requested a review from henry54809 August 31, 2021 17:38
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 31, 2021

Codecov Report

Merging #918 (3ba4f89) into master (8ddc146) will decrease coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #918      +/-   ##
==========================================
- Coverage   82.22%   82.13%   -0.09%     
==========================================
  Files          45       45              
  Lines        5637     5637              
==========================================
- Hits         4635     4630       -5     
- Misses       1002     1007       +5     
Flag Coverage Δ
ata 62.35% <ø> (ø)
aux 67.60% <ø> (-0.02%) ⬇️
base 65.54% <ø> (-0.08%) ⬇️
dhcp 66.86% <ø> (-0.02%) ⬇️
many 66.77% <ø> (-0.31%) ⬇️
mud 71.69% <ø> (-0.04%) ⬇️
switch 66.97% <ø> (-0.04%) ⬇️
topo 65.83% <ø> (ø)
unit 32.87% <ø> (ø)

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

Impacted Files Coverage Δ
daq/acl_state_collector.py 80.82% <0.00%> (-1.37%) ⬇️
daq/runner.py 84.42% <0.00%> (-0.33%) ⬇️
daq/topology.py 95.74% <0.00%> (-0.17%) ⬇️

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 8ddc146...3ba4f89. Read the comment docs.

@grafnu grafnu merged commit aeab2cc into faucetsdn:master Aug 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants