Skip to content

autotailor: Add --local-path option for layered product compatibility#2339

Merged
jan-cerny merged 2 commits intoOpenSCAP:maint-1.3from
ggbecker:fix-autotailor-relative-path-maint-1.3
Apr 17, 2026
Merged

autotailor: Add --local-path option for layered product compatibility#2339
jan-cerny merged 2 commits intoOpenSCAP:maint-1.3from
ggbecker:fix-autotailor-relative-path-maint-1.3

Conversation

@ggbecker
Copy link
Copy Markdown
Member

Backport of #2337

Add a new --local-path flag that generates benchmark references using
local paths instead of absolute file:// URIs. This enables compatibility
with Red Hat Satellite and other layered products that cannot resolve
local filesystem paths.

When --local-path is specified:
- Absolute paths are converted to basename only
- Relative paths are preserved as provided by the user

The default behavior remains unchanged for backward compatibility.

Changes based on code review feedback:
- Renamed option from --relative-path to --local-path for clarity
  (the name better reflects that absolute paths become basenames)
- Extracted URI determination logic to _get_datastream_uri() method
  to improve code organization and readability
- Added comprehensive documentation to autotailor.8 man page
- Added unit tests for _get_datastream_uri() method
- Added integration tests for --local-path functionality

Resolves: RHEL-143568
@ggbecker ggbecker force-pushed the fix-autotailor-relative-path-maint-1.3 branch from e3d556f to b670ad0 Compare April 16, 2026 13:47
@ggbecker ggbecker force-pushed the fix-autotailor-relative-path-maint-1.3 branch from b670ad0 to 3ac1b5a Compare April 16, 2026 14:30
@sonarqubecloud
Copy link
Copy Markdown

@jan-cerny jan-cerny self-assigned this Apr 17, 2026
@jan-cerny jan-cerny added this to the 1.3.15 milestone Apr 17, 2026
@jan-cerny jan-cerny merged commit f0e1788 into OpenSCAP:maint-1.3 Apr 17, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants