Backport of check error in TestDNSCycleRecursorCheckAllFail before asserting response to stop panic in CI. into release/1.17.x (#20235)

* no-op commit due to failed cherry-picking

* check error in TestDNSCycleRecursorCheckAllFail before asserting response to stop panic in CI. (#20231)

---------

Co-authored-by: temp <temp@hashicorp.com>
Co-authored-by: John Murret <john.murret@hashicorp.com>
pull/20265/head sdk/v0.15.1
hc-github-team-consul-core 2024-01-17 11:11:54 -06:00 committed by GitHub
parent ecb6ed02d6
commit 0e7c7e24ed
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 2 additions and 1 deletions

View File

@ -513,7 +513,8 @@ func TestDNSCycleRecursorCheckAllFail(t *testing.T) {
m.SetQuestion("google.com.", dns.TypeA) m.SetQuestion("google.com.", dns.TypeA)
// Agent request // Agent request
client := new(dns.Client) client := new(dns.Client)
in, _, _ := client.Exchange(m, agent.DNSAddr()) in, _, err := client.Exchange(m, agent.DNSAddr())
require.NoError(t, err)
// Verify if we hit SERVFAIL from Consul // Verify if we hit SERVFAIL from Consul
require.Equal(t, dns.RcodeServerFailure, in.Rcode) require.Equal(t, dns.RcodeServerFailure, in.Rcode)
} }