Skip to content

sudo: don't warn about search base when it was set by provider - #9031

Merged
justin-stephenson merged 1 commit into
SSSD:masterfrom
alexey-tikhonov:issue_9030
Aug 18, 2026
Merged

sudo: don't warn about search base when it was set by provider#9031
justin-stephenson merged 1 commit into
SSSD:masterfrom
alexey-tikhonov:issue_9030

Conversation

@alexey-tikhonov

Copy link
Copy Markdown
Member

Assisted-By: Claude Code (Opus 4.6)

Resolves: #9030

@opoplawski

Copy link
Copy Markdown

I've got it running locally. So far I haven't see the message, though I honestly am not entirely sure what triggers the message. It seemed to usually come fairly soon after startup, but perhaps not always.

@alexey-tikhonov

Copy link
Copy Markdown
Member Author

though I honestly am not entirely sure what triggers the message

IIUC, every re-connection to LDAP server.

@opoplawski

Copy link
Copy Markdown

Yeah, I haven't seen the message since applying the patch

@alexey-tikhonov
alexey-tikhonov marked this pull request as ready for review August 17, 2026 18:38
@alexey-tikhonov

Copy link
Copy Markdown
Member Author

(rebased)

@sumit-bose sumit-bose left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hi,

thank you for the fix, the log message is gone for the IPA case but still present for the others, ACK.

bye,
Sumit

Assisted-By: Claude Code (Opus 4.6)

Resolves: SSSD#9030
Reviewed-by: Justin Stephenson <jstephen@redhat.com>
Reviewed-by: Sumit Bose <sbose@redhat.com>
@sssd-bot

Copy link
Copy Markdown
Contributor

The pull request was accepted by @justin-stephenson with the following PR CI status:


🟢 CodeQL (success)
🟢 rpm-build:centos-stream-10-x86_64:upstream (success)
🟢 rpm-build:fedora-43-x86_64:upstream (success)
🟢 rpm-build:fedora-44-x86_64:upstream (success)
🔴 rpm-build:fedora-rawhide-x86_64:upstream (failure)
🟢 testing-farm:centos-stream-10-x86_64:upstream (success)
🔴 testing-farm:fedora-43-x86_64:upstream (failure)
🔴 testing-farm:fedora-44-x86_64:upstream (failure)
🔴 testing-farm:fedora-rawhide-x86_64:upstream (failure)
🟢 Build / freebsd (success)
🟢 Build / make-distcheck (success)
⚪ ci / intgcheck (centos-10) (cancelled)
⚪ ci / intgcheck (fedora-43) (cancelled)
⚪ ci / intgcheck (fedora-44) (cancelled)
⚪ ci / intgcheck (fedora-45) (cancelled)
🔴 ci / intgcheck (fedora-46) (failure)
🟢 ci / prepare (success)
⚪ ci / system (centos-10) (cancelled)
⚪ ci / system (fedora-43) (cancelled)
⚪ ci / system (fedora-44) (cancelled)
⚪ ci / system (fedora-45) (cancelled)
🔴 ci / system (fedora-46) (failure)
➖ Coverity scan / coverity (skipped)
🟢 Static code analysis / codeql (success)
🟢 Static code analysis / pre-commit (success)
🟢 Static code analysis / python-system-tests (success)


There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging.

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.

Possibly spurious warning about ldap_sudo_search_base

5 participants