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

Sync DNS Resolver Nameserver List #450

Merged
merged 3 commits into from
Jan 14, 2025

Conversation

fatanugraha
Copy link
Contributor

@fatanugraha fatanugraha commented Jan 9, 2025

gvisor-tap-vsock only reads the /etc/resolv.conf during init time and does not subscribe into the changes that is made on that file. That is fine, but if i start gvisor-tap-vsock before I connected to my work VPN (which modifies /etc/resolv.conf) after I connected to the VPN i will no longer be able to resolve any DNS queries made from my VM because the original IP that gvisor-tap-vsock is using is no longer reachable once I connected to my work VPN.

gvisor-tap-vsock log:

time="2025-01-07T16:57:43+08:00" level=error msg="Error during DNS Exchange: read udp <my-ip>:52316->192.168.50.1:53: i/o timeout"

In this PR, I added functionality to ensure that the nameservers that we use to resolve DNS queries are up-to-date with the content of the /etc/resolv.conf

@fatanugraha fatanugraha force-pushed the main branch 2 times, most recently from 626a4a8 to e7bf941 Compare January 9, 2025 10:31
@evidolob evidolob self-requested a review January 10, 2025 07:24
@evidolob evidolob requested a review from cfergeau January 14, 2025 09:57
@cfergeau
Copy link
Collaborator

/approve

@fatanugraha fatanugraha force-pushed the main branch 2 times, most recently from 90df25d to 0c2f9f4 Compare January 14, 2025 10:45
@cfergeau
Copy link
Collaborator

/approve
/lgtm

@cfergeau
Copy link
Collaborator

There is now a conflict in pkg/services/dns/dns_config_unix.go because of #443

Signed-off-by: Fata Nugraha <fatanugraha@outlook.com>
Signed-off-by: Fata Nugraha <fatanugraha@outlook.com>
Signed-off-by: Fata Nugraha <fatanugraha@outlook.com>
@cfergeau
Copy link
Collaborator

/lgtm
/approve

@openshift-ci openshift-ci bot added the lgtm label Jan 14, 2025
Copy link
Contributor

openshift-ci bot commented Jan 14, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cfergeau, evidolob, fatanugraha

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit 479f0c3 into containers:main Jan 14, 2025
20 checks passed
@AkihiroSuda
Copy link
Contributor

Any chance to have a new tagged release with this? 🙏

@lstocchi
Copy link
Collaborator

@AkihiroSuda we are going to make a release today

@cfergeau
Copy link
Collaborator

The release was just done https://github.com/containers/gvisor-tap-vsock/releases/tag/v0.8.2

tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request Jan 26, 2025
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [lima-vm/lima](https://github.com/lima-vm/lima) | patch | `v1.0.3` -> `v1.0.4` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>lima-vm/lima (lima-vm/lima)</summary>

### [`v1.0.4`](https://github.com/lima-vm/lima/releases/tag/v1.0.4)

[Compare Source](lima-vm/lima@v1.0.3...v1.0.4)

#### Changes

-   network:
    -   Use MAC address as dhcpd identifier ([#&#8203;3123](lima-vm/lima#3123), thanks to [@&#8203;nirs](https://github.com/nirs))
    -   Updated gvisor-tap-vsock to v0.8.2 to [fix a DNS issue](containers/gvisor-tap-vsock#450) ([#&#8203;3133](lima-vm/lima#3133))
-   YAML:
    -   Un-deprecate non-strict YAML ([#&#8203;3104](lima-vm/lima#3104), thanks to [@&#8203;jandubois](https://github.com/jandubois))
-   nerdctl:
    -   Updated from v2.0.1 to [v2.0.3](https://github.com/containerd/nerdctl/releases/tag/v2.0.3) ([#&#8203;3134](lima-vm/lima#3134))
-   Templates:
    -   Updated to the latest revisions ([#&#8203;3134](lima-vm/lima#3134))

Full changes: https://github.com/lima-vm/lima/milestone/54?closed=1
Thanks to [@&#8203;afbjorklund](https://github.com/afbjorklund) [@&#8203;alexandear](https://github.com/alexandear) [@&#8203;jandubois](https://github.com/jandubois) [@&#8203;nirs](https://github.com/nirs) [@&#8203;olamilekan000](https://github.com/olamilekan000) [@&#8203;paulinek13](https://github.com/paulinek13)

#### Usage

```console
[macOS]$ limactl create
[macOS]$ limactl start
...
INFO[0029] READY. Run `lima` to open the shell.

[macOS]$ lima uname
Linux
```

***

The binaries were built automatically on GitHub Actions.
The build log is available for 90 days: https://github.com/lima-vm/lima/actions/runs/12899702091

The sha256sum of the SHA256SUMS file itself is `05b809c6e23fa411fd6987c4fab1ceccb8efda36241130cc5269ba746a2a7762` .

***

Release manager: [@&#8203;AkihiroSuda](https://github.com/AkihiroSuda)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOS4xMjIuMCIsInVwZGF0ZWRJblZlciI6IjM5LjEyMi4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJSZW5vdmF0ZSBCb3QiXX0=-->
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.

5 participants