Skip to content

[plugins] Document that sizelimit=0 means no limit - #4457

Merged
pmoravec merged 1 commit into
sosreport:mainfrom
AdityaIn2404:doc-sizelimit-zero-unlimited
Aug 20, 2026
Merged

[plugins] Document that sizelimit=0 means no limit#4457
pmoravec merged 1 commit into
sosreport:mainfrom
AdityaIn2404:doc-sizelimit-zero-unlimited

Conversation

@AdityaIn2404

@AdityaIn2404 AdityaIn2404 commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

The sizelimit parameter in add_copy_spec, add_cmd_output, add_device_cmd, and _collect_cmd_output accepts 0 to indicate no size limit, but this was not documented in the docstrings.

Closes: #4205


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?

  The sizelimit parameter in add_copy_spec, add_cmd_output,
  add_device_cmd, and _collect_cmd_output accepts 0 to
  indicate no size limit, but this was not documented in
  the docstrings.

  Closes: sosreport#4205

Signed-off-by: Aditya Indulkar <aindulka@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-4457
  • And now you can install the packages.

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

@pmoravec
pmoravec merged commit 3e76602 into sosreport:main Aug 20, 2026
61 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.

[report] mention sizelimit=0 means no limit

3 participants