Skip to content

Initialize Config Windows Network for Windows Namespace#666

Merged
zhouhao3 merged 1 commit intoopencontainers:masterfrom
madhanrm:winns2
Oct 11, 2018
Merged

Initialize Config Windows Network for Windows Namespace#666
zhouhao3 merged 1 commit intoopencontainers:masterfrom
madhanrm:winns2

Conversation

@madhanrm
Copy link
Copy Markdown
Contributor

@madhanrm madhanrm commented Sep 28, 2018

Initialize the windows network config during Network Namespace initialization

Signed-off-by: Madhan Raj Mookkandy madhanm@microsoft.com

@madhanrm madhanrm changed the title Initialize Config Windows Network Initialize Config Windows Network for Windows Namespace Sep 28, 2018
@madhanrm
Copy link
Copy Markdown
Contributor Author

madhanrm commented Oct 3, 2018

@vbatts Can you provide some clue on how to fix the failing test?

@jterry75
Copy link
Copy Markdown
Contributor

jterry75 commented Oct 3, 2018

@crosbymichael - Can you take a look at #667 and this one. Would be helpful for us to get these in to finish the CRI work.

@madhanrm
Copy link
Copy Markdown
Contributor Author

/retest

@zhouhao3
Copy link
Copy Markdown

Need rebase.

@madhanrm
Copy link
Copy Markdown
Contributor Author

@q384566678 can you take a look?

Comment thread generate/generate.go
@jterry75
Copy link
Copy Markdown
Contributor

Nice! @madhanrm - Passed CI.

@jterry75 jterry75 mentioned this pull request Oct 10, 2018
@madhanrm
Copy link
Copy Markdown
Contributor Author

@q384566678 Can you take a look?

@zhouhao3
Copy link
Copy Markdown

Please update the relevant instructions to man/oci-runtime-tool-generate.1.md and completions/bash/oci-runtime-tool.

Signed-off-by: Madhan Raj Mookkandy <madhanm@microsoft.com>
@zhouhao3
Copy link
Copy Markdown

zhouhao3 commented Oct 11, 2018

LGTM

Approved with PullApprove

@zhouhao3 zhouhao3 merged commit 1d69bd0 into opencontainers:master Oct 11, 2018
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.

3 participants