Skip to content

Ubuntu Noble warden-specific systemd hardening#506

Closed
mike-jc wants to merge 6 commits intocloudfoundry:ubuntu-noblefrom
sap-contributions:ubuntu-noble-warden-systemd-hardening
Closed

Ubuntu Noble warden-specific systemd hardening#506
mike-jc wants to merge 6 commits intocloudfoundry:ubuntu-noblefrom
sap-contributions:ubuntu-noble-warden-systemd-hardening

Conversation

@mike-jc
Copy link
Copy Markdown

@mike-jc mike-jc commented Apr 1, 2026

This is for Ubuntu Noble stemcell.

These changes are related to issues:

Added a new stage base_systemd_clean to the warden_stages collection, which is applied during stemcell build, and removes all custom and optional systemd services, keeping only the necessary minimum set.

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla bot commented Apr 1, 2026

CLA Signed

The committers listed above are authorized under a signed CLA.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 1, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 39f5badd-d547-467a-acee-9db505adba15

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mike-jc mike-jc changed the title Ubuntu noble warden systemd hardening Ubuntu Noble warden-specific systemd hardening Apr 1, 2026
@mike-jc mike-jc force-pushed the ubuntu-noble-warden-systemd-hardening branch 2 times, most recently from 4a9c207 to 7cd21d2 Compare April 1, 2026 19:47
@mkocher
Copy link
Copy Markdown
Member

mkocher commented Apr 2, 2026

Given that we use our containerized stemcells for a lot of testing, I'd be inclined to have it mimic production as much as conveniently possible. I'd prefer to see this an explicit list of services to disable in a container instead of an allow list of services. We recently did some of this in #486 (noble) and #500 (jammy).

@mike-jc mike-jc force-pushed the ubuntu-noble-warden-systemd-hardening branch from d16d45f to 0319bcc Compare April 2, 2026 08:39
@mike-jc
Copy link
Copy Markdown
Author

mike-jc commented Apr 2, 2026

Given that we use our containerized stemcells for a lot of testing, I'd be inclined to have it mimic production as much as conveniently possible. I'd prefer to see this an explicit list of services to disable in a container instead of an allow list of services. We recently did some of this in #486 (noble) and #500 (jammy).

Thank you @mkocher, that's a good point.

I removed the allowlisting of systemd services and tested the mask list approach. The {{systemd-binfmt}} service is already masked, and testing confirms this is sufficient for warden/boshlite on both Jammy and Noble: during testing with the latest official Noble stemcell and the Jammy stemcell built on PR 500, along with a dev release containing PR 60 for Docker CPI, all HAProxy acceptance tests passed successfully.

So I close this PR and the PR for Jammy. We need only Docker CPI changes to have stemcells work there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants