Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1160667 - [RFE] Allow explicit dns configuration
[RFE] Allow explicit dns configuration
Status: CLOSED CURRENTRELEASE
Product: ovirt-engine
Classification: oVirt
Component: RFEs (Show other bugs)
---
Unspecified Unspecified
medium Severity medium (vote)
: ovirt-4.2.0
: 4.2.0
Assigned To: Martin Mucha
Michael Burman
: FutureFeature
: 1534290 (view as bug list)
Depends On: 1451261 1451245 1451313
Blocks: 1160423
  Show dependency treegraph
 
Reported: 2014-11-05 05:57 EST by Antoni Segura Puimedon
Modified: 2018-04-01 09:16 EDT (History)
21 users (show)

See Also:
Fixed In Version:
Doc Type: Enhancement
Doc Text:
The engine now supports explicit DNS configuration.
Story Points: ---
Clone Of:
Environment:
Last Closed: 2017-12-20 06:05:09 EST
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: Network
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
rule-engine: ovirt‑4.2+
mburman: testing_plan_complete+
ylavi: planning_ack+
danken: devel_ack+
myakove: testing_ack+


Attachments (Terms of Use)


External Trackers
Tracker ID Priority Status Summary Last Updated
oVirt gerrit 65309 master POST core: fixes of ipv6 annotations 2016-11-02 06:31 EDT
oVirt gerrit 65310 master MERGED core: introducing ipv4 constraint 2017-05-11 08:41 EDT
oVirt gerrit 65431 master MERGED core,webadmin: new business entities 2017-05-11 08:46 EDT
oVirt gerrit 65432 master MERGED core: update db to store DnsResolverConfiguration 2017-05-14 06:29 EDT
oVirt gerrit 65433 master MERGED core: pass DnsConfiguration to SetupNetworks and consume it from GetVdsCaps 2017-05-14 13:52 EDT
oVirt gerrit 65434 master MERGED core: DnsResolverConfiguration should be reported among ReportedConfigurations and evaluated in 'out-of-sync' mechanism 2017-05-14 13:52 EDT
oVirt gerrit 65436 master MERGED restapi: new mapper for DnsResolverConfiguration 2017-05-24 09:30 EDT
oVirt gerrit 65563 master MERGED webadmin: dns configuration 2017-05-14 15:19 EDT
oVirt gerrit 65792 master NEW core: added missing toString method 2016-10-27 05:29 EDT
oVirt gerrit 70346 master MERGED core: empty string should not be considered as valid ipv4 address 2017-05-30 06:33 EDT

  None (edit)
Description Antoni Segura Puimedon 2014-11-05 05:57:40 EST
Description of problem:
Currently, for setting DNS configuration on static IP networks, we rely on:
 - 'cfg' field for reporting.
 - Legacy Ifcfg option passthrough in SetupNetworks with a blacklist of what
   can and what can't be passed through.

The problem with the above approach is that it just can't work with other
configurators without hooks and hacks. In order to address this, we should:

- Read /etc/resolv.conf and report the dns configuration on the networks that
  have defaultRoute=True
- Accept a dns attribute to setupNetworks network definition API that takes a
  possibly empty list of addresses where one can find the dns servers. This
  would then be processed differently by configurators:
     + ifcfg: Would just have it added to the top device of the network as:
              DNS<NUM_OF_ELEMENT> for each element in the 'dns' list.
     + others: Would just write each element to /etc/resolv.conf

Steps to verify:
1. setupNetworks({'ovirtmgmt': {'nic': 'eth1', 'addr': '192.168.11.3', 'defaultRoute': True, 'netmask': '255.255.255.0', 'gateway': '192.168.1.1', 'dns': ['10.10.10.10', '10.10.10.11'], 'bridged': True}}, {}, {'connectivityCheck': False})
2. cat /etc/resolv.conf
3. getVdsCapabilities and check that the ovirtmgmt network reports dns servers
   as a list like: ['10.10.10.10', '10.10.10.11'].
Comment 2 Bob Doolittle 2015-03-11 13:41:59 EDT
I believe this bug should be pursued with high priority, since the most typical server configuration is with static addressing.

Thus, this bug affects the majority of ovirt installations, resulting in a fatal error for hosted-engine-setup which is difficult to recover from.

This should be considered a serious bug, rather than a low priority feature.

A fix should be provided in the earliest possible release of 3.5.1.
Comment 3 Bob Doolittle 2015-03-11 13:44:24 EDT
Previous note should have read "3.5.x"
Comment 4 Bob Doolittle 2015-03-11 13:44:47 EDT
Previous note should have read "3.5.x"
Comment 5 Barak 2015-06-11 07:48:47 EDT
It did not make it to 3.6, moving to 4.0.
as this is an API it may be a bit more than described above especially since cluster levels are now involved (with the API changes)
Comment 7 Dan Kenigsberg 2016-08-24 04:48:20 EDT
Vdsm side has been merged to 4.0 branch. Engine side still needs design.
Comment 8 Sandro Bonazzola 2016-10-05 08:31:28 EDT
Any update on this?
Comment 9 Martin Mucha 2016-10-05 11:25:26 EDT
please read:
https://github.com/oVirt/ovirt-site/pull/456

feature page is not approved/merged yet, but REST details are present and probably (.9) final. IIUC that's what you need most, right?
Comment 10 Yaniv Kaul 2017-06-18 10:08:18 EDT
Dan, any reason it cannot be in 4.2?
Comment 11 Dan Kenigsberg 2017-06-19 02:47:23 EDT
Moving back to 4.2, as the feature is already in final bug-fixing phase.
Comment 12 Dan Kenigsberg 2017-09-20 18:33:11 EDT
According to bug 1451261, setting DNS with DHCP most often ends with host out-of-sync. This can be mitigated by refreshing caps.

Except this bug, I believe the feature has been implemented.
Comment 13 Michael Burman 2017-09-24 04:49:05 EDT
This should be part of the known issues for the DNS feature. 
In case of setting DNS with DHCP, most of the times ends with host out-of-sync and additional manual refresh caps is needed by the user/admin.

Verified on -  4.2.0-0.0.master.20170921184504.gitfcfc9a7.el7.centos
Comment 14 Sandro Bonazzola 2017-12-20 06:05:09 EST
This bugzilla is included in oVirt 4.2.0 release, published on Dec 20th 2017.

Since the problem described in this bug report should be
resolved in oVirt 4.2.0 release, published on Dec 20th 2017, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.
Comment 15 Dan Kenigsberg 2018-04-01 09:16:18 EDT
*** Bug 1534290 has been marked as a duplicate of this bug. ***

Note You need to log in before you can comment on or make changes to this bug.