Skip to content

[networking] Capture lldpad.conf file.#4027

Merged
TurboTurtle merged 1 commit into
sosreport:mainfrom
abhishek-rawal:arawal-capture-lldpad-conf
May 27, 2025
Merged

[networking] Capture lldpad.conf file.#4027
TurboTurtle merged 1 commit into
sosreport:mainfrom
abhishek-rawal:arawal-capture-lldpad-conf

Conversation

@abhishek-rawal

Copy link
Copy Markdown
Contributor

At present, no lldpad settings are gathered.
Capture lldpad.conf present under /var/lib/lldpad/.

Related: RHEL-93499


Please place an 'X' inside each '[]' to confirm you adhere to our Contributor Guidelines

  • 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?

@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 yum install -y dnf-plugins-core on RHEL 8
  • sudo dnf install -y dnf-plugins-core on Fedora
  • dnf copr enable packit/sosreport-sos-4027
  • And now you can install the packages.

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

@abhishek-rawal abhishek-rawal force-pushed the arawal-capture-lldpad-conf branch from ada02ae to b8c2806 Compare May 26, 2025 11:04
Comment thread sos/report/plugins/networking.py
At present, no lldpad settings are gathered.
Capture lldpad.conf present under /var/lib/lldpad/.

Related: RHEL-93499

Signed-off-by: Abhishek Rawal <rawal.abhishek92@gmail.com>
@abhishek-rawal abhishek-rawal force-pushed the arawal-capture-lldpad-conf branch from b8c2806 to 0dbd268 Compare May 26, 2025 11:10
@pmoravec pmoravec added Reviewed/Needs 2nd Ack Require a 2nd ack from a maintainer Kind/Good First Issue This is a good issue for a first time contributor Quick Change This is likely a quick/trivial code change Status/Needs Review This issue still needs a review from project members labels May 26, 2025
@TurboTurtle TurboTurtle added Reviewed/Ready for Merge Has been reviewed, ready for merge Kind/Collection New or updated command or file collection and removed Reviewed/Needs 2nd Ack Require a 2nd ack from a maintainer Status/Needs Review This issue still needs a review from project members labels May 27, 2025
@TurboTurtle TurboTurtle merged commit a63f4d4 into sosreport:main May 27, 2025
38 checks passed
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 Kind/Good First Issue This is a good issue for a first time contributor Quick Change This is likely a quick/trivial code change Reviewed/Ready for Merge Has been reviewed, ready for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants