Skip to content

core/unit: re-find dropin paths of a unit after writting settings#40355

Merged
yuwata merged 1 commit intosystemd:mainfrom
HeRaNO:update-dropin-path
Jan 18, 2026
Merged

core/unit: re-find dropin paths of a unit after writting settings#40355
yuwata merged 1 commit intosystemd:mainfrom
HeRaNO:update-dropin-path

Conversation

@HeRaNO
Copy link
Copy Markdown
Contributor

@HeRaNO HeRaNO commented Jan 16, 2026

Fixes: ab932a6 ("core: simplify unit_need_daemon_reload() a bit")
Fixes #35710
Carry and replace #35711
Close: #35711

@github-actions github-actions bot added tests please-review PR is ready for (re-)review by a maintainer labels Jan 16, 2026
@HeRaNO HeRaNO force-pushed the update-dropin-path branch from d93de8e to 16a82bd Compare January 16, 2026 13:45
@yuwata yuwata added reviewed/needs-rework 🔨 PR has been reviewed and needs another round of reworks needs-stable-backport and removed please-review PR is ready for (re-)review by a maintainer labels Jan 16, 2026
@HeRaNO HeRaNO force-pushed the update-dropin-path branch from 16a82bd to 4ca2168 Compare January 16, 2026 15:06
Copy link
Copy Markdown
Member

@yuwata yuwata left a comment

Choose a reason for hiding this comment

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

LGTM.

@yuwata yuwata added good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed pid1 and removed reviewed/needs-rework 🔨 PR has been reviewed and needs another round of reworks labels Jan 16, 2026
@YHNdnzj YHNdnzj added good-to-merge/with-minor-suggestions and removed good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed labels Jan 16, 2026
@HeRaNO HeRaNO force-pushed the update-dropin-path branch from 4ca2168 to 4782613 Compare January 17, 2026 00:35
@yuwata yuwata added good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed and removed good-to-merge/with-minor-suggestions labels Jan 17, 2026
Fixes: ab932a6 ("core: simplify unit_need_daemon_reload() a bit")
Fixes #35710

Co-authored-by: Jian Wen <wenjianhn@gmail.com>
@HeRaNO HeRaNO force-pushed the update-dropin-path branch from 4782613 to 077ac66 Compare January 18, 2026 03:10
@yuwata yuwata merged commit c55aadc into systemd:main Jan 18, 2026
53 of 57 checks passed
@github-actions github-actions bot removed the good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed label Jan 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

Every unit created by runc need daemon reload since systemd v230

4 participants