Skip to content
This repository was archived by the owner on Feb 27, 2018. It is now read-only.

Conversation

@SvenDowideit
Copy link
Contributor

replaces @jamtur01 and @zeeyang 's PR's

I've added code to get the first network device's IP address from the isc dhcpd's log files - now you don't need to have the vmware tools installed in the OS for it to work. (it tries the vmware tools first)

I've added the second eth device - the host only one to the vmx

@jamtur01 if you re-do your PR to separate your changes from @zeeyang 's, then I will rejig based on that, as I don't really feel thrilled about his work being subsumed into your commit.

@jamtur01
Copy link

jamtur01 commented Jan 4, 2015

You've taken all my changes so it's moot. LGTM

@zeeyang
Copy link
Contributor

zeeyang commented Jan 4, 2015

I'm working on a new lib that's meant to consolidate the effort around fusion/workstation. The goal is to have much better error handling and logging baked into the lib layer. https://github.com/zeeyang/fusia

Getting ip from dhcp lease table as a fall back is one of the things I'm planning on putting in the fusia lib, but it should also check if the NIC is generated vs static first. Can you guys hangs tight for a few days until the first release is ready?

@SvenDowideit
Copy link
Contributor Author

@zeeyang oh, with pleasure!

@SvenDowideit
Copy link
Contributor Author

@jamtur01 grrr, you mean you're forcing me to rebase your changes? thats mean!

@SvenDowideit
Copy link
Contributor Author

mmm, intersting. the vmware dhcp server is sending an invalid nameserver when you add a second host only network.

@SvenDowideit
Copy link
Contributor Author

actually, @zeeyang @tianon can we merge this as its useful today, and then move over to your new code when its ready? the UX should be the same.

There are quite alot of people that would like to work with fusion, and having code that gets tested is useful - I've found (and made a PR) an issue in the iso already :)

jamtur01 and others added 2 commits January 6, 2015 17:20
Also updated to support the error changes in boot2docker#290.

Uses the code started by the excellent Zee Yang (https://github.com/zeeyang)

Signed-off-by: James Turnbull <james@lovedthanlost.net>
…P using dhcp server lease files

Signed-off-by: Sven Dowideit <SvenDowideit@docker.com>
@SvenDowideit SvenDowideit force-pushed the fusion-doesnt-need-vmware-os-addons branch from 1dd7613 to 2e12861 Compare January 6, 2015 07:23
@SvenDowideit
Copy link
Contributor Author

Closing as #333 makes it clear b2d-cli is already in stasis.

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