Skip to content

Conversation

abstractmj
Copy link
Contributor

@abstractmj abstractmj commented Jun 15, 2023

What type of PR is this?

kind/bug

fix feed leak when agent is in pull mode

What this PR does / why we need it:

  • feed may leak when agent is in pull mode
  • there is no warning or error log when hub failed to prune feeds for description

Which issue(s) this PR fixes:

Fixes #710

Special notes for your reviewer:

@abstractmj abstractmj requested a review from a team as a code owner June 15, 2023 02:18
@abstractmj abstractmj requested a review from dixudx June 15, 2023 02:18
@dixudx dixudx added this to the v0.16.0 milestone Jun 15, 2023
@dixudx dixudx added the kind/bug Something isn't working label Jun 15, 2023
Copy link
Member

@dixudx dixudx left a comment

Choose a reason for hiding this comment

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

Nice catch. Good work.

}

// creating credentials to parent cluster is also required in the pull mode,
// because the cleaning of redundant objects in the description is performed in the hub
Copy link
Member

Choose a reason for hiding this comment

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

s/hub/clusternet-controller-manager

@dixudx dixudx changed the title fix: fix feed leak when agent is in pull mode; issue #710 fix pruning obsolete feeds when running in pull mode Jun 15, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #711 (e4b09bd) into main (e2ffa5f) will decrease coverage by 0.64%.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##             main     #711      +/-   ##
==========================================
- Coverage   13.60%   12.96%   -0.64%     
==========================================
  Files          66       76      +10     
  Lines        7492     8710    +1218     
==========================================
+ Hits         1019     1129     +110     
- Misses       6391     7497    +1106     
- Partials       82       84       +2     

see 48 files with indirect coverage changes

@dixudx dixudx merged commit 03e174a into clusternet:main Jun 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

obsolete feeds will not be pruned in child clusters when running as pull mode
3 participants