You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
delegates new funds which triggets passive accounting claiming accrued rewards to withdrawal acct
sends icq to query withdrawal acct bal (should contain rewards from step 1) and sweep it. now withdrawal acct has balance 0
1 and 2 are basically a race between the ICA that triggers PA, and the ICQ. They don't always land in 1 -> 2 order.
Whether 2 lands before 1 can sometimes depend on the type of epoch. For day epochs, we trigger unbondings first, then delegations, then the ICQ, so usually by the time the ICQ lands, the unbondings have already triggered PA. So 1 before 2 is more likely in day epochs and 2 before 1 is more likely in stride epochs.
Our monitoring stack assumes that 1 lands before 2, so when 2 lands before 1 we get false positive monitoring errors. That's annoying and we should fix it by making the order deterministic.
Proposed change: daisy chain 1 & 2 so that we have ordering and monitoring consistency. Kick off 2 in the delegation callback.
The logical change is: "at the top of each epoch, if our delegations fail don't also reinvest"
note: ideally at some point in the future we'd have ics999-like functionality that could bundle multiple queries and ica msgs into a single packet and execute them atomically on the host.
The text was updated successfully, but these errors were encountered:
right now, each epoch stride:
1 and 2 are basically a race between the ICA that triggers PA, and the ICQ. They don't always land in 1 -> 2 order.
Whether 2 lands before 1 can sometimes depend on the type of epoch. For day epochs, we trigger unbondings first, then delegations, then the ICQ, so usually by the time the ICQ lands, the unbondings have already triggered PA. So 1 before 2 is more likely in day epochs and 2 before 1 is more likely in stride epochs.
Our monitoring stack assumes that 1 lands before 2, so when 2 lands before 1 we get false positive monitoring errors. That's annoying and we should fix it by making the order deterministic.
Proposed change: daisy chain 1 & 2 so that we have ordering and monitoring consistency. Kick off 2 in the delegation callback.
The logical change is: "at the top of each epoch, if our delegations fail don't also reinvest"
note: ideally at some point in the future we'd have ics999-like functionality that could bundle multiple queries and ica msgs into a single packet and execute them atomically on the host.
The text was updated successfully, but these errors were encountered: