Граф коммитов

4 Коммитов

Автор SHA1 Сообщение Дата
tamilmani1989 99a856982c
cni/network unit test coverage (#1020)
* adding uts

* feat: update cns client (#992)

* fix debug commands

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* fix: update cns client

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* add ctx to debug calls

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* repackage cns client

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* add ctx to all methods and preinit all route urls

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* down-scope cns client interface and move to consumer packages

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* no unkeyed struct literals

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* trace updated client method signatures out through windows paths

* delint

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* fix windows build

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* delint

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* Remove dead codes from telemetry package (#1004)

* Netlink package interfacing and adding a fake (#996)

* Initial pass at Netlink interface

* changing some netlink and epc

* Resolcing all dependencies on netlink package

* first pass at adding a netlinkinterface

* windows working now

* feat: update cns client (#992)

* fix debug commands

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* fix: update cns client

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* add ctx to debug calls

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* repackage cns client

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* add ctx to all methods and preinit all route urls

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* down-scope cns client interface and move to consumer packages

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* no unkeyed struct literals

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* trace updated client method signatures out through windows paths

* delint

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* fix windows build

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* delint

Signed-off-by: Evan Baker <rbtr@users.noreply.github.com>

* windows working now

* Some golints checks

* commenting a flaky NPM UT and adding some golint checks

* renaming fakenetlink to mocknetlink

* removing a mock netlink usage

* fixing more golints and a test fix

* fixing more go lints

* Adding in netlink from higher level as input

* adding netlinkinterface to windows endpoint impl

* removing netlink name confusion

Co-authored-by: Evan Baker <rbtr@users.noreply.github.com>

* test: add tests for CNI Azure invoker (#1010)

* include add tests

* test delete

* gci

* chore: Refactor UTs in telemetry packages (#1011)

* Refactor UTs to cleanup UTs and increase UT coverages

* User assert for consistency

* Applied comments and resolve lint error

* Delete unnecessary license header

* Add UT coverage ovs_network_client (#1008)

* Added ovsctl mock
* Changed iptables and ovsctl to use interface instead of concrete
  classes
* Added tests for ovs_networkclient_linux.go
* Fix linter issues

Co-authored-by: Shriroop <shrjo@microsoft.com>

* unitest for add, delete, get
added test for handling second add call in windows
added linux and windows specific tests
added multitenancy, baremetal tests
fixed linter errors

* fix linter issue

* fix nns test
added comment
linter fixes and dependency injection from top

* adding back removed file
fixed merge issues

* linter fixes

Co-authored-by: Evan Baker <rbtr@users.noreply.github.com>
Co-authored-by: JungukCho <jungukcho@microsoft.com>
Co-authored-by: Vamsi Kalapala <vakr@microsoft.com>
Co-authored-by: Mathew Merrick <matmerr@users.noreply.github.com>
Co-authored-by: Shriroop Joshi <shriroopjoshi@users.noreply.github.com>
Co-authored-by: Shriroop <shrjo@microsoft.com>
2021-09-20 16:58:18 -07:00
Evan Baker 96bec09d41
chore: appease the linter (3/?), the big gofumpt (#987)
* gofumpt -w -s .

* small addtl cleanups after gofumpt

* rerun after rebase
2021-09-02 16:33:18 -05:00
Mathew Merrick 1fa243e5f5
CI: Add golint-ci (#888)
* add golint-ci

* add gofmt

* enable linters

* uncap count

* fix linting/fmt issues
2021-06-01 16:58:56 -07:00
msvik 28cf37bc9d
CNI changes to support baremetal scenario (#835)
* Modify CNI for windows baremetal scenario in Azure

* Fix the CNI result for baremetal case

* Addressed PR comments from Tamilmani and Ramiro

* Remove azure-telemetry.exe from cni-baremetal package

* Fix formatting in logging

* Addressed Ramiro's PR review comments

* Formatting fix in printf

Co-authored-by: VK <abc@gmail.com>
2021-03-26 10:28:42 -07:00