Skip to content

Commit

Permalink
Fixed issue with ipa-client-install missing sssd-ipa package dependency
Browse files Browse the repository at this point in the history
  • Loading branch information
nmaludy committed Mar 14, 2022
1 parent 0eee490 commit 9a7fc83
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 1 deletion.
2 changes: 2 additions & 0 deletions manifests/install/client.pp
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@
String $principal_user = $ipa::final_domain_join_principal,
String $sssd_debug_level = $ipa::sssd_debug_level,
String $sssd_package_name = $ipa::params::sssd_package_name,
String $sssd_ipa_package_name = $ipa::params::sssd_ipa_package_name,
Array[String] $sssd_services = $ipa::sssd_services,
Hash[String, Hash[String, Any]] $sssd_config_entries = $ipa::sssd_config_entries,
) inherits ipa {
Expand Down Expand Up @@ -96,6 +97,7 @@
mode => '0600',
require => [
Package[$sssd_package_name],
Package[$sssd_ipa_package_name],
Exec["client_install_${$facts['fqdn']}"],
],
notify => Ipa::Helpers::Flushcache["server_${$facts['fqdn']}"],
Expand Down
3 changes: 2 additions & 1 deletion manifests/install/sssd.pp
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
#
class ipa::install::sssd (
String $sssd_package_name = $ipa::params::sssd_package_name,
String $sssd_ipa_package_name = $ipa::params::sssd_ipa_package_name,
) inherits ipa::params {
package { $sssd_package_name:
package { [$sssd_package_name, $sssd_ipa_package_name]:
ensure => present,
}

Expand Down
1 change: 1 addition & 0 deletions manifests/params.pp
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,7 @@
$ipa_server_package_name = 'ipa-server'
$kstart_package_name = 'kstart'
$sssd_package_name = 'sssd-common'
$sssd_ipa_package_name = 'sssd-ipa'
$sssdtools_package_name = 'sssd-tools'
# In order to avoid this error:
Expand Down

1 comment on commit 9a7fc83

@paxri01
Copy link

Choose a reason for hiding this comment

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

LGTM

Please sign in to comment.