Skip to content

Conversation

@ck319
Copy link
Contributor

@ck319 ck319 commented Mar 25, 2022

Reason for Change:

Added tests for egress policy

Issue Fixed:

Requirements:

Notes:

@huntergregory huntergregory added the npm Related to NPM. label Mar 25, 2022
@ck319 ck319 requested a review from a team as a code owner April 2, 2022 01:20
@ck319 ck319 requested review from matmerr and removed request for a team and matmerr April 2, 2022 01:20
@rbtr rbtr merged commit 438d0b3 into master Apr 7, 2022
@rbtr rbtr deleted the egressunittest branch April 7, 2022 18:48
matmerr added a commit to matmerr/azure-container-networking that referenced this pull request Jun 29, 2022
* adding egress tests

* fixed lint issue

* npm: fix error wrapping with go 1.18 (Azure#1304)

* fix npm error wrapping typo with go 1.18

* pesky lines

* vendor: bump google.golang.org/protobuf from 1.27.1 to 1.28.0 (Azure#1297)

* vendor: bump k8s.io/klog/v2 from 2.60.0 to 2.60.1 (Azure#1296)

* adding egress tests

* fixed lint issue

* added error test

* added check

* cleaned up method

Co-authored-by: Mathew Merrick <matmerr@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

npm Related to NPM.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants