Skip to content

Pipe name servers to resolvconf rather than write them to resolv.conf #18

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

Merged
merged 1 commit into from
Aug 22, 2015

Conversation

CropCircleSys
Copy link
Contributor

i don't ever use this openstack cloud stuff or whatever so if theres some mode or code path where this is making an offline image or something watch out this may break it but if this runs for you as straightforward as it looked and worked for me then great, i was in the neighborhood and a little birdie told me you needed this so try it out and let me know what you think

@CropCircleSys
Copy link
Contributor Author

Thank you so much let me know what you think or if I totally screwed the pooch on this one. now of course it only works with freebsd 9+ using my patch. if you are trying to support 8.4 still could you add a case switch please? i’m not sure how openstack os selection works early on perhaps image offline mode not happen same as python standard lib.

Rationale for pull request of course being this necessary to run local verifying caching resolver such as unbound_enable from ports or local_unbound_enable in base.

@pellaeon
Copy link
Owner

pellaeon commented Aug 8, 2015

Thanks, I'm testing it now.

@CropCircleSys
Copy link
Contributor Author

Thank you sir! Very much appreciate your great contribution to FreeBSD.

@pellaeon
Copy link
Owner

Hello, I'm sorry it took a while, during testing, I found some bugs in bsd-cloudinit itself and committed a fix, so please rebase your pull request so that I can continue testing.
Thanks!

@CropCircleSys
Copy link
Contributor Author

That should be all set now. Let me know if there's anything else I can do. Thanks.

pellaeon added a commit that referenced this pull request Aug 22, 2015
Pipe name servers to resolvconf rather than write them to resolv.conf
@pellaeon pellaeon merged commit 0a15509 into pellaeon:master Aug 22, 2015
@pellaeon
Copy link
Owner

Tested and works! Thanks for your contribution.

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