Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.
Bug 1172495 - oo-accept-broker should look for testrecord DNS with absolute domain
oo-accept-broker should look for testrecord DNS with absolute domain
Status: CLOSED ERRATA
Product: OpenShift Container Platform
Classification: Red Hat
Component: Pod (Show other bugs)
2.2.0
x86_64 Linux
high Severity low
: ---
: ---
Assigned To: Luke Meyer
libra bugs
: Upstream
Depends On:
Blocks:
  Show dependency treegraph
 
Reported: 2014-12-10 03:26 EST by gregory.nuyttens
Modified: 2015-02-12 08:09 EST (History)
7 users (show)

See Also:
Fixed In Version: openshift-origin-broker-util-1.33.0.1-1
Doc Type: Bug Fix
Doc Text:
Cause: oo-accept-broker did not test DNS using absolute domain names. (ie: those ending in .) Consequence: Under certain DNS configurations this may have produced incorrect test results. Fix: oo-accept-broker now tests for absolutely domain names avoiding problems associated with relative DNS search results. Result: oo-accept-broker DNS tests are now more accurate.
Story Points: ---
Clone Of:
Environment:
Last Closed: 2015-02-12 08:09:45 EST
Type: Bug
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
Dump of the problem (2.52 KB, application/octet-stream)
2014-12-10 03:26 EST, gregory.nuyttens
no flags Details


External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Product Errata RHBA-2015:0220 normal SHIPPED_LIVE Red Hat OpenShift Enterprise 2.2.4 bug fix and enhancement update 2015-02-12 13:08:20 EST

  None (edit)
Description gregory.nuyttens 2014-12-10 03:26:24 EST
Created attachment 966702 [details]
Dump of the problem

Description of problem:
When we make an oo-accept-broker command we got:
FAIL: txt record testrecord.${DNS_SUFFIX} still resolves on server XX.XX.XX.XX
1 ERRORS

Version-Release number of selected component (if applicable):
All versions

How reproducible:
Need to have a certain configuration at the DNS server side (it's another department which are responsible of the DNS servers in my company)

Steps to Reproduce:
1. Have the certain configuration of the DNS
2. test the command oo-accept-broker or directly make the command which are invoked by oo-accept-broker -> host -t txt testrecord.${DNS_SUFFIX} ${BIND_SERVER}

Actual results:
host -t txt testrecord.${DNS_SUFFIX} ${BIND_SERVER}
-> testrecord.openshift.${DNS_SUFFIX}.${DNS_SUFFIX} has no TXT record

Expected results:
host -t txt testrecord.${DNS_SUFFIX} ${BIND_SERVER}
Host testrecord.openshift.${DNS_SUFFIX} not found: 3(NXDOMAIN)

Additional info:
For me the problem is that into the oo-accept-broker the command host -t txt testrecord.${DNS_SUFFIX} ${BIND_SERVER} will be replaced by host -t txt testrecord.${DNS_SUFFIX}. ${BIND_SERVER} (with a dot after the DNS SUFFIX). We need to verify the absolute record (with a dot at the end) and not the relative record
Comment 2 Luke Meyer 2014-12-11 16:05:51 EST
oo-accept-broker creates and then deletes a test record to prove that the DNS plugin is working. I agree it would make sense to test for the absolute domain, not relative, but what is the particular DNS setup that's leading to the issue?
Comment 4 openshift-github-bot 2014-12-13 15:50:20 EST
Commit pushed to master at https://github.com/openshift/origin-server

https://github.com/openshift/origin-server/commit/29e1fff065a3831ea46d3715eee3bcffa1c57606
oo-accept-broker: testrecord DNS w/absolute domain

Bug 1172495 - oo-accept-broker should look for testrecord DNS with absolute domain
https://bugzilla.redhat.com/show_bug.cgi?id=1172495
Comment 5 gregory.nuyttens 2014-12-15 02:56:23 EST
Thank you for your modification, with this one it seems to be OK at our side.

For your information, I made a internal request for the dns server config and I saw that the bug only occurs when we have allow-query-cache activated.

        allow-query { any; };
        allow-query-cache { any; };
        allow-recursion { none; };

Best regards,
Comment 6 Luke Meyer 2014-12-15 08:14:17 EST
Thanks for the information, so we will know what to look for if someone else reports this... caching queries is probably not the best idea for a DDNS service, would be a good argument for separating out the DNS service used for OSE. I don't think the change made here will address that problem.
Comment 7 gregory.nuyttens 2014-12-15 08:22:44 EST
You are right..

The one thing I can tell is that when we remove the allow-query-cache we don't have any problem. If we don't remove this line but apply your fix from https://github.com/openshift/origin-server/commit/29e1fff065a3831ea46d3715eee3bcffa1c57606 it works also. 
So it's ok for us now.
Comment 10 Ma xiaoqiang 2015-01-25 21:21:57 EST
I can not reproduce this issue.
I check the codes, it has been merged.

I configure  allow-query-cache activated, then run "oo-accept-broker". No error is given out.

So move it to VERIFIED.
Comment 12 errata-xmlrpc 2015-02-12 08:09:45 EST
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHBA-2015-0220.html

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