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

(RHEL-40924) Make sure /tmp is not on tmpfs #22

Merged
merged 4 commits into from
Jul 19, 2024
Merged

(RHEL-40924) Make sure /tmp is not on tmpfs #22

merged 4 commits into from
Jul 19, 2024

Conversation

msekletar
Copy link
Member

@msekletar msekletar commented Jul 17, 2024 •
edited by github-actions bot
Loading

No description provided.

Jan Synacek and others added 4 commits July 17, 2024 07:41
@msekletar
Avoid /tmp being mounted as tmpfs without the user's will
997d57f 
rhel-only: policy

Related: RHEL-40924
@lnykryn @msekletar
unit: don't add Requires for tmp.mount
8fb2877 
rhel-only: policy

Related: RHEL-40924
@msekletar
units: add [Install] section to tmp.mount
af78fbf 
rhel-only: policy

Related: RHEL-40924
@msekletar
units: don't enable tmp.mount statically in local-fs.target
68f872f 
rhel-only: policy

Related: RHEL-40924
@msekletar msekletar requested a review from dtardon July 17, 2024 12:59
@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 17, 2024 •
edited
Loading

Commit validation

Tracker - RHEL-40924

The following commits meet all requirements

commit upstream
997d57f - Avoid /tmp being mounted as tmpfs without the user's will rhel-only: policy
8fb2877 - unit: don't add Requires for tmp.mount rhel-only: policy
af78fbf - units: add [Install] section to tmp.mount rhel-only: policy
68f872f - units: don't enable tmp.mount statically in local-fs.target rhel-only: policy

Tracker validation

Success

🟢 Tracker RHEL-40924 has set desired product: CentOS Stream 10
🟢 Tracker RHEL-40924 has set desired component: systemd
🟢 Tracker RHEL-40924 has been approved

Pull Request validation

Success

🟡 CI - Waived
🟢 Review - Reviewed by a member
🟢 Approval - Changes were approved

Auto Merge

Success

🟢 Pull Request is not marked as draft and it's not blocked by dont-merge label
🟢 Pull Request meet requirements, title has correct form
🟢 Pull Request meet requirements, mergeable is true
🟠 Pull Request meet requirements, mergeable_state is unstable
🟢 Pull Request has correct target branch main
🟢 Pull Request was merged

dtardon
dtardon approved these changes Jul 19, 2024
View reviewed changes
Copy link
Member

@dtardon dtardon left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot merged commit 571c902 into redhat-plumbers:main Jul 19, 2024
27 of 30 checks passed
@jamacku jamacku added this to the RHEL-10.0.BETA milestone Jul 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dtardon dtardon dtardon approved these changes

Assignees
No one assigned
Labels
ci-waived released
Projects
None yet
Milestone
RHEL-10.0.BETA
Development

Successfully merging this pull request may close these issues.
4 participants
@msekletar @dtardon @jamacku @lnykryn