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

do not collect metadata stuff on client side #4250

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

xiangce
Copy link
Contributor

@xiangce xiangce commented Oct 16, 2024

  • "metadata" was already collected by core collection specs
  • This change needs QE tests for all dependencies
  • Jira: RHINENG-7581

Signed-off-by: Xiangce Liu [email protected]

All Pull Requests:

Check all that apply:

  • Have you followed the guidelines in our Contributing document, including the instructions about commit messages?
  • No Sensitive Data in this change?
  • Is this PR to correct an issue?
  • Is this PR an enhancement?

@codecov-commenter
Copy link

codecov-commenter commented Oct 16, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.02%. Comparing base (dbc93e3) to head (5f98f54).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4250      +/-   ##
==========================================
+ Coverage   76.97%   77.02%   +0.04%     
==========================================
  Files         735      735              
  Lines       41300    41223      -77     
  Branches     8770     8761       -9     
==========================================
- Hits        31791    31751      -40     
+ Misses       8441     8404      -37     
  Partials     1068     1068              
Flag Coverage Δ
unittests 77.00% <100.00%> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@xiangce xiangce added client These issues represent work to be done by the "client" team. collection labels Oct 17, 2024
@xiangce xiangce force-pushed the no_client_metadata branch from af9047d to 429b7ad Compare October 18, 2024 05:24
@xiangce xiangce force-pushed the no_client_metadata branch from 429b7ad to acef0cd Compare December 5, 2024 01:30
@xiangce xiangce changed the title WIP: do not collect metadata stuff on client side do not collect metadata stuff on client side Dec 5, 2024
@xiangce xiangce marked this pull request as ready for review December 5, 2024 01:32
- "metadata" was already collected by core collection specs
- This change needs QE tests for all dependencies
- Jira: RHINENG-7581

Signed-off-by: Xiangce Liu <[email protected]>
@xiangce xiangce force-pushed the no_client_metadata branch from acef0cd to 9a2f634 Compare December 5, 2024 01:57
@jenkins-qa-bot
Copy link
Collaborator

Can one of the admins verify this patch?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client These issues represent work to be done by the "client" team. collection
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants