Skip to content

Conversation

@matmerr
Copy link
Member

@matmerr matmerr commented Sep 17, 2021

Reason for Change:

Issue Fixed:

Requirements:

Notes:

@matmerr matmerr marked this pull request as ready for review September 17, 2021 23:19
@rbtr rbtr changed the base branch from dev/utcoverage to master September 20, 2021 19:07
func getContainerNetworkConfiguration(
ctx context.Context, nwCfg *cni.NetworkConfig, podName string, podNamespace string, ifName string) (*cniTypesCurr.Result, *cns.GetNetworkContainerResponse, net.IPNet, error) {
func (a *azureMultitenancyClient) getContainerNetworkConfiguration(
ctx context.Context, nwCfg *cni.NetworkConfig, podName1 string, podNamespace string, ifName string) (*cniTypesCurr.Result, *cns.GetNetworkContainerResponse, net.IPNet, error) {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove rename

Comment on lines 50 to 51
cnsclient cnsclient
netioshim netioshim
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

where we are intializing this?

@matmerr matmerr force-pushed the multitenancyut branch 2 times, most recently from 176ae3d to ebfbc45 Compare September 21, 2021 04:49
Copy link
Member

@tamilmani1989 tamilmani1989 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@matmerr matmerr merged commit f9b65e9 into Azure:master Sep 21, 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.

2 participants