Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

Update Windows to use Azure cni v1.0.12 #3988

Closed
wants to merge 2 commits into from

Conversation

PatrickLang
Copy link
Contributor

What this PR does / why we need it:

Improving Windows networking reliability

Which issue this PR fixes
Deploying fix for Azure/azure-container-networking#232 and mitigation for Azure/azure-container-networking#242

Resolves #3405 in most cases. More fixes may still be needed but this is the first step.

@acs-bot
Copy link

acs-bot commented Oct 10, 2018

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: PatrickLang
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: brendandburns

If they are not already assigned, you can assign the PR to them by writing /assign @brendandburns in a comment when ready.

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

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ghost ghost assigned PatrickLang Oct 10, 2018
@ghost ghost added the in progress label Oct 10, 2018
@PatrickLang
Copy link
Contributor Author

/hold

@codecov
Copy link

codecov bot commented Oct 10, 2018

Codecov Report

Merging #3988 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3988      +/-   ##
==========================================
- Coverage   57.53%   57.48%   -0.05%     
==========================================
  Files         108      109       +1     
  Lines       16749    16666      -83     
==========================================
- Hits         9637     9581      -56     
+ Misses       6341     6314      -27     
  Partials      771      771

@acs-bot acs-bot added size/L and removed size/XS labels Oct 10, 2018
@PatrickLang
Copy link
Contributor Author

/close
broken branch+rebase. will open new PR

@ghost ghost removed the in progress label Oct 10, 2018
@PatrickLang PatrickLang deleted the azure-cni-v1.12 branch October 10, 2018 23:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unexpected end of JSON input
2 participants