Skip to content

Conversation

@sradco
Copy link

@sradco sradco commented Aug 11, 2025

VEP Metadata

Tracking issue: https://issues.redhat.com/browse/CNV-66100
SIG label: sig-observability

What this PR does

This VEP proposes separating KubeVirt's monitoring and metrics collection from core components through a hybrid architecture that combines efficient informer-based collection with selective sidecar containers.

Special notes for your reviewer

@kubevirt-bot kubevirt-bot added the dco-signoff: yes Indicates the PR's author has DCO signed all their commits. label Aug 11, 2025
@kubevirt-bot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign vladikr for approval. For more information see the Code Review Process.

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

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

@sradco sradco changed the title VEP - Monitoring Stack Separation via Hybrid Architecture VEP 81- Monitoring Stack Separation via Hybrid Architecture Aug 11, 2025
@sradco sradco force-pushed the monitoring_code_separation branch 4 times, most recently from 20e3f1c to efbfbdf Compare August 11, 2025 14:49
This VEP proposes separating KubeVirt's monitoring and metrics
collection from core components through a hybrid architecture
that combines efficient informer-based collection with selective
sidecar containers.

Signed-off-by: Shirly Radco <[email protected]>
@sradco sradco force-pushed the monitoring_code_separation branch from efbfbdf to e611dba Compare August 11, 2025 14:52
@sradco
Copy link
Author

sradco commented Aug 12, 2025

@vladikr , @lyarwood I would appreciate your review of this proposal.

@sradco
Copy link
Author

sradco commented Oct 27, 2025

@vladikr , @lyarwood Ping. I would appreciate your review of this proposal.

@sradco
Copy link
Author

sradco commented Oct 27, 2025

@enp0s3 Please have a look at this VEP and add it to the list of VEPs for next release. Thank you.

@vladikr
Copy link
Member

vladikr commented Oct 27, 2025

Thanks @sradco. I'll create a tracker for the 1.8 release and this VEP. So far we were mainly focused on the 1.7 release

@xpivarc
Copy link
Member

xpivarc commented Nov 21, 2025

Summarizing the discussion from VEP meeting:

  1. Please open issue so we can add it to the tracking board
  2. Please update diagram (to the latest version) and include old vs new comparison
  3. Please include the discussion that happen before 11.November into issue or PR to include the context for other reviewers
  4. Please talk to sig-network about how to scrap the metrics from the launcher sidecar,
  5. Please include how this enable adding metrics out of tree or eases work for forks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dco-signoff: yes Indicates the PR's author has DCO signed all their commits. size/XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants