Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

analyze: make systemd-analyze verify/security work for transient units #32850

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

analyze: make systemd-analyze verify/security work for transient units #32850

wants to merge 3 commits into from

Conversation

yuwata
Copy link
Member

@yuwata yuwata commented May 15, 2024 •
edited by github-actions bot

Closes #32824.

@yuwata yuwata added the analyze label May 15, 2024
@github-actions github-actions bot added tests please-review PR is ready for (re-)review by a maintainer labels May 15, 2024
@yuwata yuwata added this to the v257 milestone May 15, 2024
@github-actions GitHub Actions
Copy link

Important

An -rc1 tag has been created and a release is being prepared, so please note that PRs introducing new features and APIs will be held back until the new version has been released.

@yuwata yuwata mentioned this pull request May 15, 2024
Open
@yuwata
Copy link
Member Author

yuwata commented May 15, 2024

@YHNdnzj Thank you for the review.

  • document is extended,
  • fixed typo.

@bluca bluca added ci-fails/needs-rework 🔥 Please rework this, the CI noticed an issue with the PR and removed please-review PR is ready for (re-)review by a maintainer labels May 17, 2024
@yuwata
core,analyze: create temporary directory for generators only when run…
6a6e3d7 
…ning with MANAGER_TEST_RUN_GENERATORS

Otherwise, e.g. systemd-analyze verify or security cannot be used for
transient services.

Fixes systemd#32824.
@yuwata
man/analyze: extend document for --generators option
4e718dc
@yuwata
test: add test case for systemd-analyze verify/security for transient…
0ad9b2e 
… service
@github-actions github-actions bot added please-review PR is ready for (re-)review by a maintainer and removed ci-fails/needs-rework 🔥 Please rework this, the CI noticed an issue with the PR labels May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@YHNdnzj YHNdnzj YHNdnzj left review comments

Assignees
No one assigned
Labels
analyze documentation please-review PR is ready for (re-)review by a maintainer tests
Milestone
v257
Development

Successfully merging this pull request may close these issues.

systemd-analyze verify fails to find transient units
3 participants
@yuwata @YHNdnzj @bluca