Skip to content
This repository was archived by the owner on Jun 28, 2024. It is now read-only.

[DNM]: test commit for docker connect#743

Closed
bergwolf wants to merge 1 commit intokata-containers:masterfrom
bergwolf:test-netmon
Closed

[DNM]: test commit for docker connect#743
bergwolf wants to merge 1 commit intokata-containers:masterfrom
bergwolf:test-netmon

Conversation

@bergwolf
Copy link
Copy Markdown
Member

@sboeuf and I were getting different results w.r.t. network hotplug functionality. Still not sure what makes the difference. Let's see what we can get in CI environment. So we can have a base line for environment comparison.

Please DO NOT MERGE!

@egernst egernst added the review label Sep 13, 2018
@bergwolf bergwolf removed the review label Sep 13, 2018
@bergwolf bergwolf changed the title wip: test commit for docker connect [DNM]: test commit for docker connect Sep 13, 2018
@bergwolf bergwolf force-pushed the test-netmon branch 2 times, most recently from 31d98f7 to 5fe7e00 Compare September 14, 2018 09:37
@bergwolf
Copy link
Copy Markdown
Member Author

I've triggered tests for this for three times, once passed and twice failed on getting the IP of the added nic. Still not sure what makes the difference. Seems like CI successfully simulated both @sboeuf's case and mine. @sboeuf @devimc @amshinde any ideas? This basically tests network hotplug using the netmon PR.

Let's see what we can get in CI environment. Please DO NOT MERGE!

Fixes: #999999999

Signed-off-by: Peng Tao <bergwolf@gmail.com>
@caoruidong
Copy link
Copy Markdown
Member

@bergwolf Can this be closed?

@bergwolf
Copy link
Copy Markdown
Member Author

bergwolf commented Nov 5, 2018

yup @caoruidong

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants