Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

handle incorrect lastupdatedAt #1245

Open
adityathebe opened this issue Dec 9, 2024 · 1 comment
Open

handle incorrect lastupdatedAt #1245

adityathebe opened this issue Dec 9, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@adityathebe
Copy link
Member

adityathebe commented Dec 9, 2024

2024-12-09 16:11:12 +0000 UTC 2024-12-09 16:13:42 +0000 UTC
21:58:41.77 WRN (watch.homelab) receive lag 2m30s. receivedAt: 2024-12-09 16:13:42 +0000 UTC lastUpdatedTime: 2024-12-09 16:11:12 +0000 UTC [Deployment/default/ittools]
2024-12-09 16:13:41 +0000 UTC 2024-12-09 16:13:42 +0000 UTC
2024-12-09 16:13:41 +0000 UTC 2024-12-09 16:13:42 +0000 UTC
2024-12-09 16:13:41 +0000 UTC 2024-12-09 16:13:42 +0000 UTC

I scaled down Deployment/default/ittools to 0 replicas.
2.5 minutes later, I scaled it up to 1 replica.

config-db received 4 updates from the shared informer.
The first object gives us an incorrect lastUpdatedAtTime (2m30s ago) (I checked the object and there's no correct time we can extract from it).
If I had scaled it up after 24hrs, the lastupdatetime would have been 24hrs.
However the subsequent updates for the object had correct timestamps.

Due to this, the receive lag is highly skewed.

image

@adityathebe adityathebe added the bug Something isn't working label Dec 9, 2024
@adityathebe
Copy link
Member Author

adityathebe commented Dec 9, 2024

I think - instead of instrumenting the objects as soon as they arrive, we should be buffering.
In the example above, we should be recording 1 lag time for the 4 updates we received.

or we record the receive lag right before we consume it - i.e. after dequeuing.
Because we already dedup the 4 objects into 1 object before we consume.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant