Skip to content

Conversation

@qiujian16
Copy link
Member

Summary

This PR adds documentation for the feedbackScrapeType field in ManifestWork, which allows users to choose between Poll and Watch modes for collecting resource feedback.

Changes

  • Added new "Feedback scrape types" subsection in the Fine-grained field values tracking section
  • Documented Poll (default) and Watch modes with their characteristics
  • Provided configuration example showing how to use Watch mode
  • Included important considerations for Watch mode:
    • Resource consumption implications
    • --max-feedback-watch flag for limiting concurrent watches
    • Automatic fallback behavior when limit is reached
    • Automatic informer cleanup on resource deletion
    • Use case recommendations for each mode

Related Links

Fixes #538

🤖 Generated with Claude Code

Add documentation for the feedbackScrapeType field that allows users to
choose between Poll and Watch modes for collecting resource feedback.

- Explain Poll (default) and Watch modes
- Provide configuration example with Watch mode
- Document important considerations for Watch mode including resource
  consumption and automatic fallback behavior
- Include information about --max-feedback-watch flag and informer cleanup

Fixes open-cluster-management-io#538

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
Signed-off-by: Jian Qiu <[email protected]>
@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jan 29, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: qiujian16

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@netlify
Copy link

netlify bot commented Jan 29, 2026

Deploy Preview for open-cluster-management ready!

Name Link
🔨 Latest commit 886a115
🔍 Latest deploy log https://app.netlify.com/projects/open-cluster-management/deploys/697b05a9cbc60d00081d119f
😎 Deploy Preview https://deploy-preview-539--open-cluster-management.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@zhujian7
Copy link
Member

/LGTM

@openshift-ci openshift-ci bot added the lgtm label Jan 29, 2026
@openshift-merge-bot openshift-merge-bot bot merged commit 47d8152 into open-cluster-management-io:main Jan 29, 2026
7 checks passed
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.

Document feedbackScrapeType in manifestwork document

2 participants