Skip to content

fix: node watcher should only reconcile on important events #1522

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nddq
Copy link
Member

@nddq nddq commented Apr 7, 2025

Description

Add predicates to filter out node events that are not important for Retina agent.

Related Issue

Fixed #1453

Checklist

  • I have read the contributing documentation.
  • I signed and signed-off the commits (git commit -S -s ...). See this documentation on signing commits.
  • I have correctly attributed the author(s) of the code.
  • I have tested the changes locally.
  • I have followed the project's style guidelines.
  • I have updated the documentation, if necessary.
  • I have added tests, if applicable.

Screenshots (if applicable) or Testing Completed

Please add any relevant screenshots or GIFs to showcase the changes made.

Additional Notes

Add any additional notes or context about the pull request here.


Please refer to the CONTRIBUTING.md file for more information on how to contribute to this project.

@nddq nddq requested a review from a team as a code owner April 7, 2025 22:32
@nddq nddq requested review from aranta-rokade and mereta April 7, 2025 22:32
@nddq nddq self-assigned this Apr 7, 2025
@nddq nddq requested review from anubhabMajumdar and Copilot and removed request for aranta-rokade and mereta April 8, 2025 21:35
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Copy link
Member

@SRodi SRodi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nddq is it possible to add a test or at least capture some screenshot in the PR description with some manual tests?

@nddq
Copy link
Member Author

nddq commented Apr 15, 2025

image
image
@SRodi Retina has been running for 18 hours and only two Added node logs from the node controller

SRodi
SRodi previously approved these changes Apr 16, 2025
Copy link
Member

@SRodi SRodi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM pending review from @anubhabMajumdar

Copy link

github-actions bot commented Jun 2, 2025

This PR will be closed in 7 days due to inactivity.

@github-actions github-actions bot added the meta/waiting-for-author Blocked and waiting on the author label Jun 2, 2025
@nddq nddq removed the meta/waiting-for-author Blocked and waiting on the author label Jun 2, 2025
…of them

Signed-off-by: Quang Nguyen <nguyenquang@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Node reconciler indefinitely logs "Added Node .."
3 participants