Skip to content

Conversation

@josedonizetti
Copy link
Collaborator

@josedonizetti josedonizetti commented Nov 6, 2025

This PR refactor the old flag --install-path to its new format --runtime workdir=.... This refactor also starts to introduce the format we want the flags package to look like.

  • All flags under pkg/corba/flags
  • Only one config representation to flag and config file
  • Test for flags, structured and unstructured config file
  • Constant to avoid repetition

@codecov
Copy link

codecov bot commented Nov 6, 2025

Codecov Report

❌ Patch coverage is 46.55172% with 31 lines in your changes missing coverage. Please review.
✅ Project coverage is 30.78%. Comparing base (034cbab) to head (b5f438a).
⚠️ Report is 69 commits behind head on main.

Files with missing lines Patch % Lines
cmd/tracee/cmd/root.go 0.00% 13 Missing ⚠️
pkg/cmd/cobra/cobra.go 0.00% 10 Missing ⚠️
pkg/cmd/tracee.go 0.00% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5021      +/-   ##
==========================================
+ Coverage   29.72%   30.78%   +1.05%     
==========================================
  Files         234      236       +2     
  Lines       26186    26795     +609     
==========================================
+ Hits         7785     8248     +463     
- Misses      17864    18004     +140     
- Partials      537      543       +6     
Flag Coverage Δ
unit 30.78% <46.55%> (+1.05%) ⬆️
Files with missing lines Coverage Δ
pkg/cmd/flags/config.go 89.47% <100.00%> (+89.47%) ⬆️
pkg/cmd/flags/runtime.go 100.00% <100.00%> (ø)
pkg/cmd/tracee.go 0.00% <0.00%> (ø)
pkg/cmd/cobra/cobra.go 0.00% <0.00%> (ø)
cmd/tracee/cmd/root.go 0.00% <0.00%> (ø)

... and 28 files with indirect coverage changes

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

@josedonizetti josedonizetti force-pushed the newGeneralFlag branch 2 times, most recently from 5516c16 to 67c263c Compare November 8, 2025 13:46
@josedonizetti josedonizetti changed the title flag(general): new flag format flag(runtime): new flag format Nov 15, 2025
@josedonizetti josedonizetti force-pushed the newGeneralFlag branch 2 times, most recently from b673a4c to 048e55d Compare November 15, 2025 19:04
@josedonizetti josedonizetti force-pushed the newGeneralFlag branch 3 times, most recently from cf30996 to dcf5cf5 Compare November 25, 2025 02:11
@josedonizetti josedonizetti marked this pull request as ready for review November 25, 2025 02:26
@josedonizetti josedonizetti requested a review from geyslan December 4, 2025 13:11
Copy link
Member

@geyslan geyslan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

workdir as install-path are misleading to me. we should list what is written/read in such destination to better understand its purpose.

Copy link
Member

@geyslan geyslan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

BREAKING CHANGE: add --runtime flag and remove --install-path
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.

3 participants