Skip to content

(RHEL-109832) pam_systemd: honor session class provided via PAM environment #93

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
Merged
github-actions merged 1 commit into from
Oct 14, 2025
Merged

(RHEL-109832) pam_systemd: honor session class provided via PAM environment #93

github-actions merged 1 commit into from
Oct 14, 2025

Conversation

@msekletar
Copy link
Member

@msekletar msekletar commented Oct 3, 2025 •
edited by github-actions bot
Loading

Replaces #38638

Co-authored-by: Lennart Poettering [email protected]
(cherry picked from commit cf2630acaa87ded5ad99ea30ed4bd895e71ca503)

Resolves: RHEL-109832

@msekletar
pam_systemd: honor session class provided via PAM environment
c659f2f 
Replaces #38638

Co-authored-by: Lennart Poettering <[email protected]>
(cherry picked from commit cf2630acaa87ded5ad99ea30ed4bd895e71ca503)

Resolves: RHEL-109832
@msekletar msekletar requested a review from dtardon October 3, 2025 14:12
@github-actions
Copy link

github-actions bot commented Oct 3, 2025 •
edited
Loading

Commit validation

Tracker - RHEL-109832

The following commits meet all requirements

commit upstream
c659f2f - pam_systemd: honor session class provided via PAM environment systemd/systemd@cf2630a

Follow-up detection

Follow-ups

commit follow-up
c659f2f - pam_systemd: honor session class provided via PAM environment systemd/systemd@cb19dbb

Success

🟠 Mentions, Follow-ups and Revert commits - Waived

Tracker validation

Success

🟢 Tracker RHEL-109832 has set desired product: rhel-10.2
🟢 Tracker RHEL-109832 has set desired component: systemd
🟢 Tracker RHEL-109832 has been approved
🟢 Tracker RHEL-109832 has set severity

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 Oct 7, 2025
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

@mrc0mmand
Copy link
Member

mrc0mmand commented Oct 7, 2025

The three failing tests are a known issue (https://issues.redhat.com/browse/RHEL-119135).

@github-actions github-actions bot merged commit f3a6a5a into redhat-plumbers:main Oct 14, 2025
29 of 30 checks passed
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 follow-up-waived released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@msekletar @mrc0mmand @dtardon @jamacku