Skip to content

[Policy] Add os_release_id check - #4306

Merged
arif-ali merged 1 commit into
sosreport:mainfrom
eric-desrochers:azurelinux
Apr 22, 2026
Merged

[Policy] Add os_release_id check#4306
arif-ali merged 1 commit into
sosreport:mainfrom
eric-desrochers:azurelinux

Conversation

@eric-desrochers

Copy link
Copy Markdown
Contributor

Add additional check introducing os_release_id in azure.py policy.


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?

Add additional check introducing os_release_id in azure.py policy.

Signed-off-by: Eric Desrochers <edesrochers@microsoft.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-4306
  • And now you can install the packages.

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

@TurboTurtle TurboTurtle added Kind/Enhancement Reviewed/Needs 2nd Ack Require a 2nd ack from a maintainer labels Apr 21, 2026
@arif-ali arif-ali added Reviewed/Ready for Merge Has been reviewed, ready for merge and removed Reviewed/Needs 2nd Ack Require a 2nd ack from a maintainer labels Apr 22, 2026
@arif-ali
arif-ali merged commit fcc6a5f into sosreport:main Apr 22, 2026
44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Kind/Enhancement 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