go-cni icon indicating copy to clipboard operation
go-cni copied to clipboard

Open items for completion

Open abhi opened this issue 7 years ago • 4 comments

  • [x] Readme and Usage guidlines
  • [x] Port mapping testing
  • [x] Unit Test cases
  • [ ] Supporting specific network attachments
  • [x] More hooks and extensions
  • [ ] Evaluate use cases like these: https://github.com/clearcontainers/runtime/blob/master/docs/architecture/architecture.md#cni . If we can make it easier for them to use cni through our library.
  • [ ] Makefile,formats etc

abhi avatar Feb 22 '18 06:02 abhi

cc @crosbymichael @ehazlett Please add in things you feel will be needed to complete this effort.

abhi avatar Feb 22 '18 07:02 abhi

Can you add an example usage in a small Go app for CNI?

crosbymichael avatar Feb 22 '18 16:02 crosbymichael

The use of populateNetworkConfig is pretty bad. It can generate so many different errors and they are just handled silently.

crosbymichael avatar Feb 22 '18 16:02 crosbymichael

@crosbymichael thanks for looking into it . I can take care of it.

abhi avatar Feb 22 '18 16:02 abhi