Skip to content

Conversation

@ramereth
Copy link
Contributor

If you use the postgresql_access resource and have it trigger a reload of the
postgresql_service resource, it will not do it in the correct order. Currently,
it will reload the server, and then the config file gets updated which means the
new config is never reloaded.

This ensures that it happens at the end of the resource run so it works in the
proper order.

Signed-off-by: Lance Albertson [email protected]

Description

Describe what this change achieves

Issues Resolved

List any existing issues this PR resolves

Check List

  • A summary of changes made is included in the CHANGELOG under ## Unreleased
  • New functionality includes testing.
  • New functionality has been documented in the README if applicable.

@ramereth ramereth requested a review from a team as a code owner September 19, 2025 16:53
@ramereth ramereth added the Release: Patch Release to Chef Supermarket as a version patch when merged label Sep 19, 2025
If you use the postgresql_access resource and have it trigger a reload of the
postgresql_service resource, it will not do it in the correct order. Currently,
it will reload the server, and then the config file gets updated which means the
new config is never reloaded.

This ensures that it happens at the end of the resource run so it works in the
proper order.

Signed-off-by: Lance Albertson <[email protected]>
@ramereth ramereth force-pushed the access-notification-fix branch from bf44a21 to d4c854c Compare September 19, 2025 18:15
@github-actions
Copy link

Slowest examples

Top 10 slowest examples (0.01 seconds, 49.1% of total time)
Example Description Time in seconds
spec/libraries/helpers_spec.rb:19 PostgreSQL::Cookbook::Helpers#data_dir(version) with rhel family and Postgres 11 from repo returns the correct path 0.01284
spec/libraries/helpers_spec.rb:27 PostgreSQL::Cookbook::Helpers#data_dir(version) with rhel family and Postgres 11 from os returns the correct path 0.00048
spec/libraries/helpers_spec.rb:119 PostgreSQL::Cookbook::Helpers#default_platform_service_name(version) with debian family and Postgres 12 from os returns the correct service name 0.00041
spec/libraries/helpers_spec.rb:119 PostgreSQL::Cookbook::Helpers#default_platform_service_name(version) with debian family and Postgres 15 from os returns the correct service name 0.0004
spec/libraries/helpers_spec.rb:43 PostgreSQL::Cookbook::Helpers#data_dir(version) with debian family and Postgres 14 from os returns the correct path 0.00039
spec/libraries/helpers_spec.rb:111 PostgreSQL::Cookbook::Helpers#default_platform_service_name(version) with debian family and Postgres 13 from repo returns the correct service name 0.00039
spec/libraries/helpers_spec.rb:119 PostgreSQL::Cookbook::Helpers#default_platform_service_name(version) with debian family and Postgres 14 from os returns the correct service name 0.00039
spec/libraries/helpers_spec.rb:73 PostgreSQL::Cookbook::Helpers#conf_dir(version) with debian family and Postgres 13 from repo returns the correct path 0.00038
spec/libraries/helpers_spec.rb:119 PostgreSQL::Cookbook::Helpers#default_platform_service_name(version) with debian family and Postgres 11 from os returns the correct service name 0.00038
spec/libraries/helpers_spec.rb:111 PostgreSQL::Cookbook::Helpers#default_platform_service_name(version) with debian family and Postgres 12 from repo returns the correct service name 0.00038

@ramereth ramereth merged commit 37ef336 into main Sep 19, 2025
123 of 201 checks passed
@ramereth ramereth deleted the access-notification-fix branch September 19, 2025 18:51
@kitchen-porter
Copy link
Contributor

Released as: 12.3.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Release: Patch Release to Chef Supermarket as a version patch when merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants