Skip to content

[autobackport: sssd-2-9] Tests: multihost change passwd setting for STIG - #9346

Merged
spoore1 merged 1 commit into
SSSD:sssd-2-9from
sssd-bot:SSSD-sssd-backport-pr9295-to-sssd-2-9
Sep 29, 2026
Merged

spoore1 merged 1 commit into
SSSD:sssd-2-9from
sssd-bot:SSSD-sssd-backport-pr9295-to-sssd-2-9

Conversation

@sssd-bot

Copy link
Copy Markdown
Contributor

This is an automatic backport of PR#9295 Tests: multihost change passwd setting for STIG to branch sssd-2-9, created by @spoore1.

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 git@github.com:sssd-bot/sssd.git
git fetch sssd-bot refs/heads/SSSD-sssd-backport-pr9295-to-sssd-2-9
git checkout SSSD-sssd-backport-pr9295-to-sssd-2-9
git push sssd-bot SSSD-sssd-backport-pr9295-to-sssd-2-9 --force

Original commits
18a9902 - Tests: multihost change passwd setting for STIG

Backported commits

  • 2561802 - Tests: multihost change passwd setting for STIG

Original Pull Request Body

The STIG settings are hindering user passwd setting in multihost/alltests. Changing from using passwd --stdin to an
echo|chpasswd.

@spoore1
spoore1 force-pushed the SSSD-sssd-backport-pr9295-to-sssd-2-9 branch from d542f23 to 41fbb12 Compare September 28, 2026 12:33
@spoore1 spoore1 added Tests Trivial A single reviewer is sufficient to review the Pull Request no-backport This should go to target branch only. labels Sep 28, 2026
@spoore1

spoore1 commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

Test failures are unrelated since this change only affects the multihost tests and the failures are from the system tests:

FAILED tests/test_authentication.py::test_authentication__user_login_then_changes_password (ipa) - AssertionError: 'user1' password change failed!
3114
assert False
3115
 +  where False = password('user1', 'Secret123', 'New_Secret123')
3116
 +    where password = <sssd_test_framework.utils.authentication.PasswdUtils object at 0x7231de774210>.password
3117
 +      where <sssd_test_framework.utils.authentication.PasswdUtils object at 0x7231de774210> = <sssd_test_framework.utils.authentication.AuthenticationUtils object at 0x7231de9b7610>.passwd
3118
 +        where <sssd_test_framework.utils.authentication.AuthenticationUtils object at 0x7231de9b7610> = <sssd_test_framework.roles.client.Client object at 0x7231de7a4e10>.auth
3119
FAILED tests/test_authentication.py::test_authentication__user_is_forced_to_change_expired_password_before_login[root-su] (ipa) - sssd_test_framework.misc.errors.ExpectScriptError: Unexpected end of file
3120
FAILED tests/test_authentication.py::test_authentication__user_is_forced_to_change_expired_password_before_login[root-ssh] (ipa) - sssd_test_framework.misc.errors.ExpectScriptError: Unexpected end of file
3121
FAILED tests/test_access_control_simple.py::test_access_control_simple__deny_group_supersedes_allow_group (ldap) - sssd_test_framework.misc.errors.ExpectScriptError: Unexpected end of file
3122
FAILED tests/test_authentication.py::test_authentication__user_login[root-ssh] (ldap) - sssd_test_framework.misc.errors.ExpectScriptError: Unexpected end of file
3123
FAILED tests/test_authentication.py::test_authentication__user_login_then_changes_password (ldap) - sssd_test_framework.misc.errors.ExpectScriptError: Unexpected end of file
3124
= 6 failed, 496 passed, 52 skipped, 224 deselected, 1 xfailed, 2 warnings in 5482.35s (1:31:22) =
3125

Also, I believe some other PRs in the works currently will resolve those issues once merged.

The STIG settings are hindering user passwd setting in
multihost/alltests.   Changing from using passwd --stdin to an
echo|chpasswd.

Also, removed commented code from original change.

Reviewed-by: Madhuri Upadhye <mupadhye@redhat.com>
(cherry picked from commit 18a9902)
@sssd-bot

Copy link
Copy Markdown
Contributor Author

The pull request was accepted by @madhuriupadhye with the following PR CI status:


🟢 CodeQL (success)
🟢 rpm-build:centos-stream-9-x86_64:upstream (success)
🟢 testing-farm:centos-stream-9-x86_64:upstream (success)
🟢 Build / make-distcheck (success)
🟢 ci / prepare (success)
🔴 ci / system (centos-9) (failure)
🟢 Static code analysis / codeql (success)
🟢 Static code analysis / files-changed (success)
🟢 Static code analysis / pre-commit (success)
➖ Static code analysis / python-system-tests (skipped)


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-pr9295-to-sssd-2-9 branch from 41fbb12 to e8a80d1 Compare September 29, 2026 10:38
@spoore1
spoore1 merged commit 40bc4a8 into SSSD:sssd-2-9 Sep 29, 2026
11 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. Tests Trivial A single reviewer is sufficient to review the Pull Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants