-
Notifications
You must be signed in to change notification settings - Fork 638
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
user-v2 regression in master: DNS wouldn't work with existing VMs #1783
Comments
I checked this. Iptables looks fine. But the nameservers in user-data is not updated to new record after new version update |
I couldn't find a way to fix this yet. One thing i noticed is the dns nameserver not updated correctly. Apart from that all were looking fine only. Let me know if we have some ways to enforce cloud-init to update nameservers on every boot |
If the issue happens only on upgrade we can just let hostagent fix up DNS? |
@AkihiroSuda Started net-user-v2 in old version. Did stop on old version. Started the same in current master DNS is working fine. There are few other issue but unrelated to regression |
Thank you, seems we can close this |
I could definitely reproduce this reliably when I reported the issue. I would create a VM with colima, upgrade lima to HEAD, and always get broken DNS. I didn't have this problem when I started with lima HEAD. I'm traveling right now but would be happy to provide more details when I get back. |
@AkihiroSuda @balajiv113: Repro steps:
|
I don't think this has anything to do with user-v2 mode. But i will check on these following steps and see if its failing |
@aaronlehmann could you try directly with lima ?? To see if that works. As far as i know colima adds their own dns resolver as well for qemu, this might be something related to that as well |
@aaronlehmann Thanks for the bug. I was able to re-produce it with alpine image @AkihiroSuda Looks like if a nameserver gets updated in cloud-init that's not reflected in /etc/resolv.conf on next start |
It works if i do |
Should have been solved in: |
Confirmed the fix, thanks! |
Originally posted by @aaronlehmann in #1333 (comment)
The text was updated successfully, but these errors were encountered: