Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

allocator: Remove deleted networks from the unallocated set #2019

Merged

Conversation

aaronlehmann
Copy link
Collaborator

This looks like an oversight. Networks may have been kept in this set
forever after deletion.

cc @alexmavr @yongtang @aboch

This looks like an oversight. Networks may have been kept in this set
forever after deletion.

Signed-off-by: Aaron Lehmann <aaron.lehmann@docker.com>
@codecov
Copy link

codecov bot commented Mar 8, 2017

Codecov Report

Merging #2019 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2019      +/-   ##
==========================================
+ Coverage   53.66%   53.69%   +0.03%     
==========================================
  Files         109      109              
  Lines       18991    18992       +1     
==========================================
+ Hits        10191    10198       +7     
+ Misses       7578     7566      -12     
- Partials     1222     1228       +6

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 e4762bc...5f52c14. Read the comment docs.

@aboch
Copy link

aboch commented Mar 8, 2017

Looks good to me

@aluzzardi
Copy link
Member

LGTM

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.

3 participants