Skip to content

[autobackport: sssd-2-9] KRB5: log level adjusted#8550

Merged
alexey-tikhonov merged 1 commit into
SSSD:sssd-2-9from
sssd-bot:SSSD-sssd-backport-pr8548-to-sssd-2-9
Mar 25, 2026
Merged

[autobackport: sssd-2-9] KRB5: log level adjusted#8550
alexey-tikhonov merged 1 commit into
SSSD:sssd-2-9from
sssd-bot:SSSD-sssd-backport-pr8548-to-sssd-2-9

Conversation

@sssd-bot

Copy link
Copy Markdown
Contributor

This is an automatic backport of PR#8548 KRB5: log level adjusted to branch sssd-2-9, created by @alexey-tikhonov.

Please make sure this backport is correct.

Note

The commits were cherry-picked without conflicts.

You can push changes to this pull request

git remote add sssd-bot [email protected]:sssd-bot/sssd.git
git fetch sssd-bot refs/heads/SSSD-sssd-backport-pr8548-to-sssd-2-9
git checkout SSSD-sssd-backport-pr8548-to-sssd-2-9
git push sssd-bot SSSD-sssd-backport-pr8548-to-sssd-2-9 --force

Original commits
8631c02 - KRB5: log level adjusted

Backported commits

  • 23b3542 - KRB5: log level adjusted

Original Pull Request Body

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request adjusts the debug logging level in src/providers/krb5/krb5_init.c. Specifically, it changes a debug message related to missing krb5_kpasswd option and explicit KDC settings from SSSDBG_FATAL_FAILURE to SSSDBG_CONF_SETTINGS. This is an appropriate change, as it correctly categorizes a configuration-related informational message, which describes a fallback mechanism, rather than a fatal error. There is no feedback to provide on the changes.

@alexey-tikhonov alexey-tikhonov added the no-backport This should go to target branch only. label Mar 25, 2026

@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,

backport matches the original patch, ACK.

bye,
Sumit

Resolves: SSSD#8531
Reviewed-by: Sumit Bose <[email protected]>
(cherry picked from commit 8631c02)
@sssd-bot

Copy link
Copy Markdown
Contributor Author

The pull request was accepted by @alexey-tikhonov with the following PR CI status:


🟢 CodeQL (success)
🟢 rpm-build:centos-stream-9-x86_64:upstream (success)
🟢 Build / make-distcheck (success)
🟢 ci / prepare (success)
🟢 ci / system (centos-9) (success)
🟢 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.

@sssd-bot
sssd-bot force-pushed the SSSD-sssd-backport-pr8548-to-sssd-2-9 branch from 23b3542 to 98754d3 Compare March 25, 2026 17:04
@alexey-tikhonov
alexey-tikhonov merged commit 330f845 into SSSD:sssd-2-9 Mar 25, 2026
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Accepted no-backport This should go to target branch only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants