Skip to content

Conversation

@Symorglass
Copy link
Contributor

@Symorglass Symorglass commented Jul 7, 2025

What does this PR change?

This PR integrates the containerized Salt Event Processor service. It allows user to deploy and manage the Salt Event Processor as a separate service container alongside the Uyuni server.

  • Registered event processor service
  • Container Lifecycle Management and CLI Integration: implemented event processor mgradm install/uninstall/upgrade/status/start/restart/stop podman command and flags
  • Integrated --debug-java flag to expose java debug port

Codespace

Check if you already have a running container clicking on Running CodeSpace

Create CodeSpace About billing for Github Codespaces CodeSpace Billing Summary CodeSpace Limit

Test coverage

  • Unit tests (pending)

  • Integration Test:

    • Tested install and upgrade Uyuni server with event processor image
    • Verified event processor integrates with mgradm start/stop/restart/status commands successfully
    • Confirmed event processor runs as systemd service [email protected] with proper database connectivity and network configuration
    • Tested --debug-java flag
  • DONE

Links

Issue(s): #210

  • DONE

Changelogs

Make sure the changelogs entries you are adding are compliant with https://github.com/uyuni-project/uyuni/wiki/Contributing#changelogs and https://github.com/uyuni-project/uyuni/wiki/Contributing#uyuni-projectuyuni-repository

If you don't need a changelog check, please mark this checkbox:

  • No changelog needed

If you uncheck the checkbox after the PR is created, you will need to re-run changelog_test (see below)

Before you merge

Check How to branch and merge properly!

@Symorglass Symorglass force-pushed the salt-event-processor-container branch from 1cac240 to 2096578 Compare July 8, 2025 05:19
@Symorglass Symorglass force-pushed the salt-event-processor-container branch from 6c2802a to 429f91e Compare July 30, 2025 16:20
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
2 New issues

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@Symorglass Symorglass force-pushed the salt-event-processor-container branch from 90b63fe to 81c9af8 Compare September 10, 2025 21:28
@sonarqubecloud
Copy link

@Symorglass Symorglass marked this pull request as ready for review September 11, 2025 13:58
@Symorglass Symorglass requested a review from cbosdo September 11, 2025 13:58
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.

2 participants