Skip to content

ROX-33564: Add gzip, less, and tar packages to collector ubi-micro image#3220

Closed
janisz wants to merge 1 commit intomasterfrom
ROX-33564/common-packages
Closed

ROX-33564: Add gzip, less, and tar packages to collector ubi-micro image#3220
janisz wants to merge 1 commit intomasterfrom
ROX-33564/common-packages

Conversation

@janisz
Copy link
Copy Markdown
Contributor

@janisz janisz commented Apr 14, 2026

Checklist

  • Investigated and inspected CI test results
  • Updated documentation accordingly

Automated testing

  • Added unit tests
  • Added integration tests
  • Added regression tests

If any of these don't apply, please comment below.

Testing Performed

TODO(replace-me)
Use this space to explain how you tested your PR, or, if you didn't test it, why you did not do so. (Valid reasons include "CI is sufficient" or "No testable changes")
In addition to reviewing your code, reviewers must also review your testing instructions, and make sure they are sufficient.

For more details, ref the Confluence page about this section.

@janisz janisz requested review from a team and rhacs-bot as code owners April 14, 2026 13:06
@janisz janisz force-pushed the ROX-33564/common-packages branch from 78f5357 to 2ba7509 Compare April 14, 2026 13:09
These common Unix utilities are useful for debugging, log inspection, and
file operations within containers. Adding them to the collector image
provides consistent tooling across the StackRox platform.

Changes:
- Updated rpms.in.yaml with gzip, less, tar (alphabetically sorted)
- Regenerated rpms.lock.yaml for all architectures
- Updated konflux.Dockerfile to install packages (one per line)
- Updated regular Dockerfile to install packages

Code partially generated with AI assistance.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 14, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 27.38%. Comparing base (e98bedf) to head (a7120a0).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3220   +/-   ##
=======================================
  Coverage   27.38%   27.38%           
=======================================
  Files          95       95           
  Lines        5427     5427           
  Branches     2548     2548           
=======================================
  Hits         1486     1486           
  Misses       3214     3214           
  Partials      727      727           
Flag Coverage Δ
collector-unit-tests 27.38% <ø> (ø)

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.

--releasever=10 \
--setopt=install_weak_deps=False \
--nodocs \
ca-certificates curl-minimal elfutils-libelf gzip less libcap-ng libstdc++ libuuid openssl tar tbb && \
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Each of gzip, less and tar is already here. Do you think we still need this PR?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Cool, closing it then

@janisz janisz closed this Apr 15, 2026
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.

3 participants