Skip to content

[sunbeam] Add deployment commands#4036

Merged
arif-ali merged 1 commit into
sosreport:mainfrom
arif-ali:sos-arif-sunbeam-deployment
Jun 9, 2025
Merged

[sunbeam] Add deployment commands#4036
arif-ali merged 1 commit into
sosreport:mainfrom
arif-ali:sos-arif-sunbeam-deployment

Conversation

@arif-ali

@arif-ali arif-ali commented Jun 3, 2025

Copy link
Copy Markdown
Member
  • Update some commands that now need a sunbeam_user
  • manifest show doesn't need --id any more

Closes: #4037
Closes: #4038


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

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

@arif-ali arif-ali added Kind/Ubuntu Ubuntu related item Status/Needs Review This issue still needs a review from project members Kind/Collection New or updated command or file collection labels Jun 3, 2025
@arif-ali arif-ali force-pushed the sos-arif-sunbeam-deployment branch from d89ef01 to 0dc934f Compare June 4, 2025 15:49
Comment thread sos/report/plugins/sunbeam.py Outdated
@arif-ali arif-ali force-pushed the sos-arif-sunbeam-deployment branch 2 times, most recently from 45038f5 to ceb371a Compare June 9, 2025 08:44
* Update some commands that now need a sunbeam_user
* manifest show doesn't need --id any more
* Move the cluster commands under the user

Closes: sosreport#4037
Closes: sosreport#4038

Signed-off-by: Arif Ali <arif.ali@canonical.com>
@arif-ali arif-ali force-pushed the sos-arif-sunbeam-deployment branch from ceb371a to 941f00a Compare June 9, 2025 09:10

@pmoravec pmoravec left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

From sos plugin perspective, ack. No idea what is being collected etc :) but that is not needed for such review.

@TurboTurtle

Copy link
Copy Markdown
Member

Restarted the stage two jobs. Will merge afterwards.

@TurboTurtle TurboTurtle added Reviewed/Ready for Merge Has been reviewed, ready for merge and removed Status/Needs Review This issue still needs a review from project members labels Jun 9, 2025
@arif-ali arif-ali merged commit f4eae5a into sosreport:main Jun 9, 2025
39 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/Ubuntu Ubuntu related item Reviewed/Ready for Merge Has been reviewed, ready for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[sunbeam] show manifest uses removed option --id [sunbeam] sos report execute some of the sunbeam commands as root

4 participants