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

EC2 task networking for Windows: Build vpc-eni plugin #2871

Closed

Conversation

rawahars
Copy link
Contributor

Summary

The primary changes made in this PR are-

  • Build vpc-eni CNI plugin for Windows
  • Bug fix: Pause container sends nil network stats initially (when no network is present) which can cause the agent to crash and restart.

Implementation details

  • Invoke make command for vpc-eni CNI plugin
  • Add a nil check for the objects

Testing

A custom binary was built and tested.

New tests cover the changes:
No new functionality has been added

Description for the changelog

Build vpc-eni CNI plugin for Windows task networking.

Licensing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Harsh Rawat added 2 commits May 18, 2021 18:13
…k stats.

Initially, without any network, pause container returns nil network stats. When container stats are collected with pause namespace in place, lastStats.NetworkStats is nil which causes agent to crash and restart.
@rawahars
Copy link
Contributor Author

This PR is dependent upon the CNI plugin PR (aws/amazon-vpc-cni-plugins#52).

Until that PR is merged, the build will fail for this one.

@rawahars
Copy link
Contributor Author

rawahars commented Jun 4, 2021

These changes are now part of #2876 PR.

@rawahars rawahars closed this Jun 4, 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.

6 participants