Skip to content

Comments

Fix issue when using apache2_mod_ssl resource instead a wrapper resource#800

Merged
bmhughes merged 1 commit intomainfrom
mod_ssl-fix
Oct 20, 2023
Merged

Fix issue when using apache2_mod_ssl resource instead a wrapper resource#800
bmhughes merged 1 commit intomainfrom
mod_ssl-fix

Conversation

@ramereth
Copy link
Contributor

When doing apache2_module 'ssl' instead of another custom resource, the
mod_ssl package does not get installed properly due to the fact it's set to run
in the root context of the chef run. However, it never seems to get installed,
or at least installed in time for it to work leaving a system in a broken state.

I cannot see why this was needed in the first place as it seems to work without
it.

Signed-off-by: Lance Albertson lance@osuosl.org

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.

When doing `apache2_module 'ssl'` instead of another custom resource, the
mod_ssl package does not get installed properly due to the fact it's set to run
in the root context of the chef run. However, it never seems to get installed,
or at least installed in time for it to work leaving a system in a broken state.

I cannot see why this was needed in the first place as it seems to work without
it.

Signed-off-by: Lance Albertson <lance@osuosl.org>
@ramereth ramereth requested a review from a team as a code owner October 18, 2023 23:29
@ramereth ramereth added the Release: Patch Release to Chef Supermarket as a version patch when merged label Oct 18, 2023
@bmhughes bmhughes merged commit fe92b83 into main Oct 20, 2023
@bmhughes bmhughes deleted the mod_ssl-fix branch October 20, 2023 12:45
@kitchen-porter
Copy link
Contributor

Released as: 9.0.6

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.

4 participants