Point72
Setup CI for adapter with service dependencies
ok this is finally working ok, does not appreciably increase CI/CD time and worth it until we can separate adapters.
There is a race condition in one of the tests it seems, wanted to note here in case it comes up. I cannot fully hammer it down, and unclear if its an actual issue with the historical->realtime switch or if the test itself has buggy assumptions.
There is a race condition in one of the tests it seems, wanted to note here in case it comes up. I cannot fully hammer it down, and unclear if its an actual issue with the historical->realtime switch or if the test itself has buggy assumptions.
do you have an example link
https://github.com/Point72/csp/actions/runs/10149403062/job/28067369034
Based on the test and the error message it looks like whats happening is that the sub is missing the first pub. Its possible the daemon is late to register the subscriber before the first publish. One way to try and fix it is to delay the first pub a bit. Another possibility is to compare the last N msgs rather than the first N