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

Enable SSSD authentication by default in containers: uyuni-tools changes #224

Merged
merged 3 commits into from
Apr 17, 2024

Conversation

mbussolotto
Copy link
Member

@mbussolotto mbussolotto commented Mar 29, 2024

What does this PR change?

  • Enable SSSD authentication by default
  • Persists /etc/sssd folder: all authentication mechanism should be configured using sssd

Test coverage

  • No tests
  • DONE

Links

Issue(s): https://github.com/SUSE/spacewalk/issues/22545

  • DONE

Changelogs

Make sure the changelogs entries you are adding are compliant with https://github.com/uyuni-project/uyuni/wiki/Contributing#changelogs and https://github.com/uyuni-project/uyuni/wiki/Contributing#uyuni-projectuyuni-repository

If you don't need a changelog check, please mark this checkbox:

  • No changelog needed

If you uncheck the checkbox after the PR is created, you will need to re-run changelog_test (see below)

Before you merge

Check How to branch and merge properly!

@mbussolotto mbussolotto marked this pull request as ready for review April 9, 2024 09:10
@mbussolotto mbussolotto requested review from cbosdo and rjmateus April 10, 2024 08:58
@mbussolotto mbussolotto changed the title PAM authentication in Containers enable SSSD authentication by default in containers Apr 10, 2024
@mbussolotto mbussolotto changed the title enable SSSD authentication by default in containers Enable SSSD authentication by default in containers Apr 10, 2024
@mbussolotto mbussolotto changed the title Enable SSSD authentication by default in containers Enable SSSD authentication by default in containers: uyuni-tools changes Apr 10, 2024
Copy link
Member

@rjmateus rjmateus left a comment

Choose a reason for hiding this comment

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

Just one comment about the migration. Looks good from my side

@mbussolotto mbussolotto merged commit 61cd3ef into uyuni-project:main Apr 17, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants