Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix nfs if there is xenbr0 as public_network, remove nokogiri dependency, update REDME #30

Closed
wants to merge 7 commits into from

Conversation

chenull
Copy link

@chenull chenull commented Apr 10, 2016

Hi,

this is my first ever code in ruby, so please review / test it. feel free to discard my PR, and use your own implementation so the same result can be achieved. I've tested in Windows 7, Windows 10, and Mac OS 10.9 (maverick), but not linux.

This PR will fix issue #18
and i think PR #27 is not needed anymore since actually i don't see nokogiri's calls inside the prepare_nfs_settings.rb code.

Thanks

@chenull chenull changed the title Fix nfs if there is xenbr0 as public_network, update REDME Fix nfs if there is xenbr0 as public_network, remove nokogiri dependency, update REDME Apr 10, 2016
@darkn3rd
Copy link

darkn3rd commented May 3, 2016

This looks nice. It would be nice to test on Ubuntu (or Debian) and CentOS flavor, but other than that, it looks really good.

@wkoszek
Copy link
Contributor

wkoszek commented Apr 10, 2017

@jonludlam Any chance to bring these changes in? There are some good improvements here in documentation.

@lindig
Copy link

lindig commented Nov 26, 2018

Nobody seems to be working on this - please re-open.

@lindig lindig closed this Nov 26, 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.

4 participants