Skip to content

Commit

Permalink
Remove redundant autorequire
Browse files Browse the repository at this point in the history
We already have a relationship in `manifests/server/instance/service.pp`
that ensure `Service["postgresqld_instance_${name}"]` is realized before
`Anchor["postgresql::server::service::end::${name}"]`.

Fun fact, removing this duplaciate fix the circular dependency reported
by puppet.
  • Loading branch information
smortex committed Apr 15, 2024
1 parent 0ea543e commit d15ac61
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions lib/puppet/type/postgresql_psql.rb
Original file line number Diff line number Diff line change
Expand Up @@ -138,10 +138,6 @@ def matches(value)
["postgresql::server::service::end::#{self[:instance]}"]
end

autorequire(:service) do
["postgresqld_instance_#{self[:instance]}"]
end

def should_run_sql(refreshing = false)
onlyif_param = @parameters[:onlyif]
unless_param = @parameters[:unless]
Expand Down

0 comments on commit d15ac61

Please sign in to comment.