Skip to content

Toggle transitive entities for machine get and list#86

Draft
majst01 wants to merge 2 commits into
mainfrom
toggle-transitive-entities
Draft

Toggle transitive entities for machine get and list#86
majst01 wants to merge 2 commits into
mainfrom
toggle-transitive-entities

Conversation

@majst01
Copy link
Copy Markdown
Contributor

@majst01 majst01 commented Oct 16, 2025

Description

This will reduce the processing time and payload size for intensive machine get and list calls

@metal-robot metal-robot Bot added the area: control-plane Affects the metal-stack control-plane area. label Oct 16, 2025
@majst01 majst01 force-pushed the toggle-transitive-entities branch from 1c0e1c9 to bab9a95 Compare October 17, 2025 04:52
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 17, 2025

Codecov Report

❌ Patch coverage is 47.34300% with 109 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.09%. Comparing base (ad1746d) to head (bab9a95).

Files with missing lines Patch % Lines
pkg/repository/machine.go 40.11% 92 Missing and 11 partials ⚠️
pkg/repository/ip.go 50.00% 1 Missing ⚠️
pkg/repository/network.go 50.00% 1 Missing ⚠️
pkg/repository/project-member.go 50.00% 1 Missing ⚠️
pkg/repository/project.go 50.00% 1 Missing ⚠️
pkg/repository/tenant-member.go 50.00% 1 Missing ⚠️
pkg/repository/tenant.go 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #86      +/-   ##
==========================================
+ Coverage   66.05%   66.09%   +0.04%     
==========================================
  Files         108      108              
  Lines       11394    11423      +29     
==========================================
+ Hits         7526     7550      +24     
- Misses       3186     3190       +4     
- Partials      682      683       +1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

Labels

area: control-plane Affects the metal-stack control-plane area.

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant