SSSD / sssd-ci-containers

Setup containerized environment for testing and developing SSSD.
GNU General Public License v3.0
10 stars 21 forks source link

Conflicting names for IPA keytab file #96

Closed abrad3 closed 5 months ago

abrad3 commented 5 months ago

In this repo, the IPA test keytab file is /enrollment/ipa.test.keytab. However, in the tests it seems to expect it to be called /enrollment/ipa.keytab. I couldn't run tests on IPA until I renamed /enrollment/ipa.test.keytab to /enrollment/ipa.keytab. I tested the fix by running the solution to task 16 at https://tests.sssd.io/en/latest/course/course.html#take-the-course

Which name is the one that should be used? Once I know, I can make a PR to fix this.

abrad3 commented 5 months ago

I've noticed the same issue happens with samba- which makes sense, it uses the same naming convention

jakub-vavra-cz commented 5 months ago

Hi, this was changed as preparation for setups that might have multiple ipa, ad servers/domains. The course needs fixing to reflect the updated mhc.yaml.

See: https://github.com/SSSD/sssd/blob/master/src/tests/system/mhc.yaml

Regards, Jakub

cc: @pbrezina

abrad3 commented 5 months ago

I'm no longer seeing this issue- not sure if it was fixed by something on your end or mine- I hadn't pulled from master in a while so it might have been me. So I'm going to close this!