Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

spec file: bump sssd version #6561

Closed
wants to merge 2 commits into from
Closed

Conversation

flo-renaud
Copy link
Contributor

spec file: bump sssd version

Bump sssd version to 2.8.0 on fedora37+ and RHEL
to ensure the fix for SSSD #6631 is present.

No need to bump the version on fedora 36 as the issue
is not seen on versions < 37.

Fixes: https://pagure.io/freeipa/issue/9234
Signed-off-by: Florence Blanc-Renaud flo@redhat.com

ipatests: remove xfail for tests using sssctl domain-status

The tests calling sssctl domain-status were marked xfail
because of SSSD issue #6331. Now that the issue is fixed
and freeipa bumped sssd required version, remove the xfail
annotation.

Related: https://pagure.io/freeipa/issue/9234

@flo-renaud
Copy link
Contributor Author

Adding ipa-4-10 label as this branch is used to build RHEL 9.2 / Fedora 36+ and the fix is needed there.
No need to backport to ipa-4-9 as 4.9 is used for fedora 35, fedora 36 and RHEL8 and the fix is not needed on these versions.

@flo-renaud flo-renaud added the ipa-4-10 Mark for backport to ipa 4.10 label Nov 18, 2022
@abbra
Copy link
Contributor

abbra commented Nov 21, 2022

LGTM, please remove the temp commit.

Bump sssd version to 2.8.0 on fedora37+ and RHEL
to ensure the fix for SSSD freeipa#6631 is present.

No need to bump the version on fedora 36 as the issue
is not seen on versions < 37.

Fixes: https://pagure.io/freeipa/issue/9234
Signed-off-by: Florence Blanc-Renaud <flo@redhat.com>
The tests calling sssctl domain-status were marked xfail
because of SSSD issue freeipa#6331. Now that the issue is fixed
and freeipa bumped sssd required version, remove the xfail
annotation.

Related: https://pagure.io/freeipa/issue/9234
Signed-off-by: Florence Blanc-Renaud <flo@redhat.com>
@flo-renaud
Copy link
Contributor Author

@abbra thanks for the review, temp commit removed and tests are green.

@abbra abbra added the ack Pull Request approved, can be merged label Nov 22, 2022
@abbra
Copy link
Contributor

abbra commented Nov 22, 2022

Thank you, @flo-renaud. Ack added.

@flo-renaud flo-renaud added the pushed Pull Request has already been pushed label Nov 22, 2022
@flo-renaud
Copy link
Contributor Author

master:

  • 5a23d8e spec file: bump sssd version
  • 29012bb ipatests: remove xfail for tests using sssctl domain-status

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged ipa-4-10 Mark for backport to ipa 4.10 pushed Pull Request has already been pushed
Projects
None yet
2 participants