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

support config dns via network stanza #65

Merged

Conversation

lisongmin
Copy link
Contributor

Hi, I added the dns config support in this PR which defined in job-specification/network#dns-parameters.

This PR will resolve following scenario:

  1. host is the dns provider, and fill nameserver 127.0.0.1 in /etc/resolv.conf
  2. running job with containerd-driver, app in task can't resolve the dns.

this PR requires building with nomad 1.0 or above, i don't know if it can run with nomad 0.10 or not.
I just comfirm that it can running on nomad 1.0.2.

@shishir-a412ed
Copy link
Contributor

@lisongmin Looks like the DNS support was added in version 0.12.4 hashicorp/nomad#7661
I think it's okay to update to nomad 1.0.2

Can you check why the circleci tests are failing? Also, this might needs a rebase.

@github-actions
Copy link

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

tests/006-test-dns.sh Outdated Show resolved Hide resolved
tests/run_tests.sh Outdated Show resolved Hide resolved
tests/run_tests.sh Show resolved Hide resolved
@shishir-a412ed shishir-a412ed requested review from a team and chuckyz January 26, 2021 20:33
@shishir-a412ed
Copy link
Contributor

@lisongmin There was some intermittent issue with circleci I have re-run the tests and now they are passing!

@lisongmin lisongmin force-pushed the support-dns-via-network-stanza branch from 00e68df to a46f601 Compare January 27, 2021 07:45
Copy link
Contributor

@shishir-a412ed shishir-a412ed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@lisongmin Will merge once this is addressed.

@lisongmin lisongmin force-pushed the support-dns-via-network-stanza branch from a46f601 to e0abb30 Compare January 28, 2021 00:25
@shishir-a412ed shishir-a412ed merged commit d1c51d3 into Roblox:master Jan 28, 2021
@lisongmin lisongmin deleted the support-dns-via-network-stanza branch February 11, 2021 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants