Skip to content

Introduce 2 service approach#2280

Merged
oleflb merged 16 commits intoHULKs:mainfrom
oleflb:hulk-service
Apr 18, 2026
Merged

Introduce 2 service approach#2280
oleflb merged 16 commits intoHULKs:mainfrom
oleflb:hulk-service

Conversation

@oleflb
Copy link
Copy Markdown
Contributor

@oleflb oleflb commented Mar 9, 2026

Why? What?

The motivation is to be able to easily run commands inside the hulk container, which has special mounts, libraries and environment variables.
With the launchHULK script, this required some shenanigans in order to launch a shell.

This PR introduces a 2 service approach:

  • One hulk-runtime service, which spawns the container and does nothing (sleeps)
  • The hulk service, which has a dependency on hulk-runtime and execs into the container to run the hulk binary.

Therefore it is now very simple for users to run custom commands (or a shell) in the container by just running sudo podman exec -it hulk bash

In this PR I also fixed logging such that the journal can be used again.

Fixes #

ToDo / Known Issues

If this is a WIP describe which problems are to be fixed.

Ideas for Next Iterations (Not This PR)

If there are some improvements that could be done in a next iteration, describe them here.

How to Test

Describe how to test your changes. (For the reviewer)

@github-project-automation github-project-automation Bot moved this to Request for Review in Development Mar 9, 2026
@pejotejo pejotejo added the GO26 label Mar 10, 2026
@schmidma
Copy link
Copy Markdown
Member

This branch has conflicts...

Comment thread tools/k1-setup/launchHULK Outdated
Comment thread tools/k1-setup/launchHULK Outdated
Comment thread tools/k1-setup/launch-hulk Outdated
Comment thread tools/pepsi/src/gammaray.rs
@oleflb oleflb requested a review from knoellle April 17, 2026 19:18
@oleflb oleflb changed the title Update launchHULK to allow running custom binaries Introduce 2 service approach Apr 17, 2026
@oleflb oleflb enabled auto-merge April 17, 2026 19:25
knoellle
knoellle previously approved these changes Apr 18, 2026
Copy link
Copy Markdown
Member

@knoellle knoellle left a comment

Choose a reason for hiding this comment

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

LGTM

Comment thread tools/pepsi/src/gammaray.rs Outdated
Comment thread tools/pepsi/src/gammaray.rs Outdated
Copy link
Copy Markdown
Member

@knoellle knoellle left a comment

Choose a reason for hiding this comment

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

LGTM

@oleflb oleflb added this pull request to the merge queue Apr 18, 2026
Merged via the queue into HULKs:main with commit cb29792 Apr 18, 2026
7 checks passed
@oleflb oleflb deleted the hulk-service branch April 18, 2026 14:15
@github-project-automation github-project-automation Bot moved this from Request for Review to Done in Development Apr 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants