URL: https://github.com/SSSD/sssd/pull/5755
Title: #5755: Tests: support subid ranges managed by FreeIPA

aborah-sudo commented:
"""
> ```
> ./src/tests/multihost/ipa/test_subid_ranges.py:126:1: W391 blank line at end 
> of file      ---- This is fixed now 
> ./src/tests/multihost/ipa/conftest.py:180:80: E501 line too long (82 > 79 
> characters)
> ```
> 
>     1. Why `tier1` was changed to `tier2`?   ----   Its steeves idea to keep 
> IPA tests in tire2 
> 
>     2. why `exit_status` check was changed to:    ----  IDM ci does not work 
> as we expect .  
exit_status was catching some weird errors which is not related to this test at 
all:

Check test run with empty list:  
https://ci-jenkins-csb-idmops.apps.ocp-c1.prod.psi.redhat.com/job/trigger-test-suite-tool/556/console
    ----   ['Could not chdir to home directory /home/admin: No such file or 
directory\n'] which not relevant and we need to check newuidmap relate errors 
which is not 


The error we are looking for is :  "write to uid_map failed"   that is the 
exact error now configured with the test script . And after puting the exact 
error which is not occurring at the run time . The test run in IDM CI passed 

With exact error code:  
https://ci-jenkins-csb-idmops.apps.ocp-c1.prod.psi.redhat.com/job/trigger-test-suite-tool/558/console
 

> 
> 
> ```
>         for i in exit_status.readlines():
>             assert "write to uid_map failed" not in i
> ```
> 
> ?


"""

See the full comment at 
https://github.com/SSSD/sssd/pull/5755#issuecomment-915682896
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

Reply via email to