[nvidia] Capture more nvidia commands

[jcastill]

Capture commands related to nvidia container toolkit.

Related: RHEL-58172

---

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

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-3777
• And now you can install the packages.

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

[pmoravec]

Sounds good to me. I re-started timeouted tests that failed.

[arif-ali]

curious, would it good to add journal for all these too. i.e. if one only wants to enable the nvidia plugin and nothing else?

[jcastill]

Good idea. I'll addthe journals as well

[TurboTurtle]

If these are added to the services tuple, it can be used for plugin enablement as well as automatically getting the journal and service status.

[jcastill]

Good idea. Shall I add the three services to the plugin enablement? We have its journal capture a bit different:

self.add_journal(boot=0, identifier='nvidia-persistenced')

But a quick test I did in a RHEL AI image showed that the nvidia-persistenced service is captured correctly. My only doubt is the 'boot=0' option.

[arif-ali]

lgtm

[arif-ali]

nit, do we need the extra line here? not overly precious about it though

[jcastill]

Good point, I think we can remove it.
My only doubt then is what to do with this add_journal() call before it. The service is now in the services tuple, but the boot=0 is what makes me doubt if it's safe or not to remove it. Any thoughts on this?

[TurboTurtle]

At current, this will result in duplicate journal collections for nvidia-persistenced, once for the whole journal and then again for only the current boot. Since that service is now in the services tuple, we automatically get the journal.

[jcastill]

Thank you for clarifying it Jake. I'll remove the extra capture and will push again.