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

KDS Reconciles all DP related insights #3783

Closed
lahabana opened this issue Feb 1, 2022 · 3 comments · Fixed by #3819
Closed

KDS Reconciles all DP related insights #3783

lahabana opened this issue Feb 1, 2022 · 3 comments · Fixed by #3819
Assignees
Labels
kind/bug A bug triage/accepted The issue was reviewed and is complete enough to start working on it

Comments

@lahabana
Copy link
Contributor

lahabana commented Feb 1, 2022

What happened?

I think something is wrong with reconciliation of insights. At the beginning I thought it's only related to Zone{In,E}gressInsights, but it looks like it's for all dataplane related insights.
I went to the doctor and forget to stop my CPs and have a lot of updates (every 2.5 minutes).

image

This shouldn't happen if no change happen on the cluster

@lahabana lahabana added triage/pending This issue will be looked at on the next triage meeting kind/bug A bug triage/accepted The issue was reviewed and is complete enough to start working on it and removed triage/pending This issue will be looked at on the next triage meeting labels Feb 1, 2022
@parkanzky
Copy link
Contributor

These inherit from DataplaneInsights, which has the field -

  // Generation is an integer number which is periodically increased by the
  // status sink
  uint32 generation = 7;

Logging the objects it is evident that this is, indeed, increasing on every sync and appears in the GetSpec() output we compare to decide whether a change occurred.

May need to handle this differently.

@lahabana
Copy link
Contributor Author

lahabana commented Feb 9, 2022

Ho this was introduced with: #2526 which makes sense... Should we remove the generation in the KDS code to avoid syncing things that do not really matter outside of the local zone?
@jakubdyszkiewicz @lobkovilya WDYT?

@lahabana
Copy link
Contributor Author

lahabana commented Feb 9, 2022

Maybe there's something to be done with the ResourceMapper to strip this info?

@lahabana lahabana added this to the Perf milestone Feb 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug A bug triage/accepted The issue was reviewed and is complete enough to start working on it
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants