#9459 Nightly test failure (with healthcheck 0.14) in test_ipahealthcheck.py::TestIpaHealthCheckWithoutDNS::test_ipa_dns_systemrecords_check
Closed: fixed a year ago by rcritten. Opened a year ago by frenaud.

Issue

The test test_ipahealthcheck.py::TestIpaHealthCheckWithoutDNS::test_ipa_dns_systemrecords_check is failing when run with ipa-healthechk 0.14. Example of failure in PR#3024 with the following logs and report:

self = <ipatests.test_integration.test_ipahealthcheck.TestIpaHealthCheckWithoutDNS object at 0x7ff2ba03cf50>

    def test_ipa_dns_systemrecords_check(self):
        """
        Test checks the result of IPADNSSystemRecordsCheck
        when ipa-server is configured without DNS.
        """
        version = tasks.get_healthcheck_version(self.master)
        if (parse_version(version) < parse_version('0.12')):
            expected_msgs = {
                "Expected SRV record missing",
                "Got {count} ipa-ca A records, expected {expected}",
                "Got {count} ipa-ca AAAA records, expected {expected}",
                "Expected URI record missing",
            }
        else:
            expected_msgs = {
                "Expected SRV record missing",
                "Unexpected ipa-ca address {ipaddr}",
                "expected ipa-ca to contain {ipaddr} for {server}",
                "Expected URI record missing",
            }

        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"] in expected_msgs
E           AssertionError: assert 'missing IP address for ipa-ca server {server}' in {'Expected SRV record missing', 'Expected URI record missing', 'Unexpected ipa-ca address {ipaddr}', 'expected ipa-ca to contain {ipaddr} for {server}'}

A new test was added in ipa-healthcheck 0.13 with commit https://github.com/freeipa/freeipa-healthcheck/commit/8ca85127b97b1379e36794c19fe38d372ed07f76
The test needs to be adapted and expect the new error.


master:

  • f9075f9 ipatests: fix healthcheck test without DNS

ipa-4-11:

  • 4c85121 ipatests: fix healthcheck test without DNS

ipa-4-10:

  • d780ff9 ipatests: fix healthcheck test without DNS

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

a year ago

Log in to comment on this ticket.

Metadata