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.

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

(RHEL-60896) socket: fix socket activation of stopped services with pinned FD store #45

Merged
merged 1 commit into from
Oct 22, 2024

Conversation

lnykryn
Copy link
Member

@lnykryn lnykryn commented Oct 14, 2024

(cherry picked from commit 941a12dcba57f6673230a9c413738c51374d2998)

Resolves: RHEL-60896

(cherry picked from commit 941a12dcba57f6673230a9c413738c51374d2998)

Resolves: RHEL-60896
@lnykryn
Copy link
Member Author

lnykryn commented Oct 14, 2024

This is not a complete fix; this part of the code is later reworked, but... meh

@github-actions github-actions bot changed the title socket: fix socket activation of stopped services with pinned FD store (RHEL-60896) socket: fix socket activation of stopped services with pinned FD store Oct 14, 2024
Copy link

github-actions bot commented Oct 14, 2024

Commit validation

Tracker - RHEL-60896

The following commits meet all requirements

commit upstream
ba744ed - socket: fix socket activation of stopped services with pinned FD store… systemd/systemd@941a12d

Follow-up detection

Commit mentions

commit mention
ba744ed - socket: fix socket activation of stopped services with pinned FD store… systemd/systemd@4c523d6

Success

🟠 Mentions, Follow-ups and Revert commits - Waived


Tracker validation

Success

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


Pull Request validation

Success

🟢 CI - All checks have passed
🟢 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 clean
🟢 Pull Request has correct target branch main
🟢 Pull Request was merged

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 e20fddc into redhat-plumbers:main Oct 22, 2024
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants