whatyouhide / xandra

Fast, simple, and robust Cassandra/ScyllaDB driver for Elixir.
ISC License
406 stars 54 forks source link

`startup establishes a control connection and a pool` test is flaky #332

Closed harunzengin closed 1 year ago

harunzengin commented 1 year ago

In https://github.com/lexhide/xandra/actions/runs/6100320853/job/16554384322?pr=329

test startup establishes a control connection and a pool (Xandra.Cluster.PoolTest)
Error:      test/xandra/cluster/pool_test.exs:94
     match (=) failed
     The following variables were pinned:
       pid = #PID<0.1288.0>
     code:  assert %{cluster_pid: ^pid, host: %Host{address: {127, 0, 0, 1}, port: @port}} =
              assert_telemetry(telemetry_ref, [:control_connection, :connected])
     left:  %{cluster_pid: ^pid, host: %Xandra.Cluster.Host{address: {127, 0, 0, 1}, port: 9052}}
     right: %{
              cluster_name: nil,
              cluster_pid: #PID<0.1273.0>,
              host: %Xandra.Cluster.Host{
                address: {127, 0, 0, 1},
                port: 9052,
                data_center: nil,
                host_id: nil,
                rack: nil,
                release_version: nil,
                schema_version: nil,
                tokens: nil
              }
            }
     stacktrace:
       test/xandra/cluster/pool_test.exs:106: (test)
whatyouhide commented 1 year ago

I'm fairly sure this was fixed by https://github.com/lexhide/xandra/commit/c43f80c7b9f807a65aab53425e74addb3b4e0a5e#diff-1755284d08091fbd525f17baa740124d2ca89ac682b9c4fc82a19b678c5c2ad7R4. 🙃 Closing for now, I'll reopen this if it shows up again!