#8573 Nightly failure in test_ipahealthcheck.py::TestIpaHealthCheckWithoutDNS::test_ipa_dns_systemrecords_check
Closed: fixed by rcritten. Opened by frenaud.

Issue

The nightly test test_ipahealthcheck.py::TestIpaHealthCheckWithoutDNS::test_ipa_dns_systemrecords_check is failing in the master branch with updates-testing enabled, see PR #521.

Logs and report:

self = <ipatests.test_integration.test_ipahealthcheck.TestIpaHealthCheckWithoutDNS object at 0x7f29c6253850>
    def test_ipa_dns_systemrecords_check(self):
        """
        Test checks the result of IPADNSSystemRecordsCheck
        when ipa-server is configured without DNS.
        """
        msg1 = "Expected SRV record missing"
        msg2 = "Got {count} ipa-ca A records, expected {expected}"
        tasks.install_packages(self.master, HEALTHCHECK_PKG)
        returncode, data = run_healthcheck(
            self.master,
            "ipahealthcheck.ipa.idns",
            "IPADNSSystemRecordsCheck",
        )
        assert returncode == 1
        for check in data:
            assert check["result"] == "WARNING"
>           assert check["kw"]["msg"] == msg1 or check["kw"]["msg"] == msg2
E           AssertionError: assert ('Got {count} ...ed {expected}' == 'Expected SRV record missing'
E             - Expected SRV record missing
E             + Got {count} ipa-ca AAAA records, expected {expected} or 'Got {count} ...ed {expected}' == 'Got {count} ...ed {expected}'
E             - Got {count} ipa-ca A records, expected {expected}
E             + Got {count} ipa-ca AAAA records, expected {expected}
E             ?                     +++)

The test is executing ipa-healthcheck's IPADNSSystemRecordsCheckcheck but is not prepared for a missing AAAA record.
ipa-healthcheck has been checking for AAAA records since commit 02c3b27 that has been included in ipa-healthcheck 0.7 available in updates-testing since 2020-11-05.

The test needs to be adapted.


Metadata Update from @frenaud:
- Issue assigned to frenaud

Metadata Update from @frenaud:
- Custom field on_review adjusted to https://github.com/freeipa/freeipa/pull/5241

master:

  • ab9ef13f278bf02fc018e20a4aefb20976049e71 ipatests: IPADNSSystemRecordsCheck also checks for AAAA records

ipa-4-8:

  • 4d8403bf9dfc4c02fdbad6492b377efaa0ce5f2c ipatests: IPADNSSystemRecordsCheck also checks for AAAA records

Metadata Update from @rcritten:
- Issue close_status updated to: fixed
- Issue status updated to: Closed (was: Open)

Metadata