Skip to content

build: remove integration k8s + components#2502

Merged
skl merged 20 commits intomainfrom
skl/k8s-test-split
Feb 11, 2026
Merged

build: remove integration k8s + components#2502
skl merged 20 commits intomainfrom
skl/k8s-test-split

Conversation

@skl
Copy link
Member

@skl skl commented Feb 10, 2026 •
edited
Loading

This PR:

  • Deletes internal/test/integration
  • All k8s integration tests (and components) are now sourced from obi directly
  • Beyla-specifics added to internal/test/beyla_extensions
  • Same 17 K8s shards, 5 OATs shards, and 91 integration tests across 15 shards as main

@codecov-commenter
Copy link

codecov-commenter commented Feb 10, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 47.32%. Comparing base (f2966b7) to head (d361b7f).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files 
@@             Coverage Diff             @@
##             main    #2502       +/-   ##
===========================================
+ Coverage   35.58%   47.32%   +11.73%     
===========================================
  Files          86       59       -27     
  Lines        6308     4511     -1797     
===========================================
- Hits         2245     2135      -110     
+ Misses       3893     2212     -1681     
+ Partials      170      164        -6
Flag Coverage Δ
integration-test 12.88% <ø> (-0.01%) ⬇️
integration-test-vm-${ARCH}-${KERNEL_VERSION} 0.00% <ø> (ø)
k8s-integration-test 1.85% <ø> (-0.01%) ⬇️
oats-test 0.00% <ø> (ø)
unittests 43.80% <ø> (+10.73%) ⬆️

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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@skl skl marked this pull request as ready for review February 10, 2026 20:05
@skl skl requested a review from a team as a code owner February 10, 2026 20:05
@mariomac
Copy link
Contributor

mariomac commented Feb 11, 2026

Amazing!! How are these differences handled in the tests?

  • Metric names: obi_network_flow... vs beyla_network_flow...
  • Attribute name: obi_ip vs beyla_ip

I cannot see it in your PR but however the network flow tests are passing 🧚🦄🪄

@skl
Copy link
Member Author

skl commented Feb 11, 2026

@mariomac similar to the drift/sync, there is a list of transforms which are applied when the OBI test suite is imported. It's easier to browse the branch directly, here are the transforms you're looking for:

beyla/scripts/generate-obi-tests.sh

Lines 80 to 86 in 68420b8

# --- Metric name prefixes (exported output) ---
'obi_|beyla_'
# --- Attribute names (exported output) ---
'obi\.ip|beyla.ip'
'obi\.network\.flow|beyla.network.flow'
'obi\.network\.inter\.zone|beyla.network.inter.zone'

The transforms are applied after the files have been copied into internal/obi (which is not in git):

beyla/scripts/generate-obi-tests.sh

Lines 493 to 497 in 68420b8

apply_behavioral_transforms() {
local jobs="$1"
echo " Applying OBI → Beyla behavioral transforms..."
find "$OBI_DEST" -type f \( -name "*.go" -o -name "*.yml" -o -name "*.yaml" \) | run_parallel "$jobs" apply_transforms "${BEHAVIORAL_TRANSFORMS[@]}"
}

There is also a code injection which I needed here, to override the metric prefix:

beyla/scripts/generate-obi-tests.sh

Lines 121 to 123 in 68420b8

# The vendored DefaultOBIConfig() returns MetricPrefix="obi", but the
# Beyla binary exports internal metrics with the "beyla" prefix.
'config := ti\.DefaultOBIConfig()|config.MetricPrefix = "beyla"'

@skl skl merged commit 0791fd7 into main Feb 11, 2026
85 of 87 checks passed
@skl skl deleted the skl/k8s-test-split branch February 11, 2026 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mariomac mariomac mariomac approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@skl @codecov-commenter @mariomac