Skip to content

[logs] Collect /var/lib/rsyslog/ directory listings - #4463

Open
abetkike wants to merge 1 commit into
sosreport:mainfrom
abetkike:collect-varlib-rsyslog-listing
Open

[logs] Collect /var/lib/rsyslog/ directory listings#4463
abetkike wants to merge 1 commit into
sosreport:mainfrom
abetkike:collect-varlib-rsyslog-listing

Conversation

@abetkike

Copy link
Copy Markdown

Add directory listings for /var/lib/rsyslog/ to help diagnose
rsyslog state file issues and verify SELinux contexts on the
path hierarchy. Commands fail gracefully when rsyslog is not
installed or uses a non-default WorkDirectory.

Collects:

  • ls -ldZ /var/ /var/lib/ /var/lib/rsyslog/ — path hierarchy with SELinux labels
  • ls -lZ /var/lib/rsyslog/ — directory contents with SELinux labels

  • Is the commit message split over multiple lines and hard-wrapped at 72 characters?
  • Is the subject and message clear and concise?
  • Does the subject start with [plugin_name] if submitting a plugin patch or a [section_name] if part of the core sosreport code?
  • Does the commit contain a Signed-off-by: First Lastname email@example.com?
  • Are any related Issues or existing PRs properly referenced via a Closes (Issue) or Resolved (PR) line?
  • Are all passwords or private data gathered by this PR obfuscated?

Add directory listings for /var/lib/rsyslog/ to help diagnose
rsyslog state file issues and verify SELinux contexts on the
path hierarchy. Commands fail gracefully when rsyslog is not
installed or uses a non-default WorkDirectory.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Amey Betkiker <abetkike@redhat.com>
@packit-as-a-service

Copy link
Copy Markdown

Congratulations! One of the builds has completed. 🍾

You can install the built RPMs by following these steps:

  • sudo dnf install -y 'dnf*-command(copr)'
  • dnf copr enable packit/sosreport-sos-4463
  • And now you can install the packages.

Please note that the RPMs should be used only in a testing environment.

Comment on lines +99 to +102
self.add_cmd_output([
"ls -ldZ /var/ /var/lib/ /var/lib/rsyslog/",
"ls -lZ /var/lib/rsyslog/",
])

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Plugin.get_dir_listing() exists for standardized directory listing collections. Please use that for these.

@TurboTurtle TurboTurtle added Status/Needs Review This issue still needs a review from project members Kind/Collection New or updated command or file collection Reviewed/Needs Iteration Review has been performed, change needs to be iterated on based on feedback before merge. labels Aug 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Kind/Collection New or updated command or file collection Reviewed/Needs Iteration Review has been performed, change needs to be iterated on based on feedback before merge. Status/Needs Review This issue still needs a review from project members

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants