Ensure switching of thread module works on Debian 10 / Ubuntu 20.04 #2034
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
apache::mpm is a typethat may have no external impact to Forge modules. This module is declared in 174 of 575 indexed public
|
tuxmea
referenced
this pull request
May 25, 2020
Usual thread modules are: prefork, event and worker. When switching from one thread module to another we must ensure that the old, no longer to use, thread module gets disabled. Puppet Apache module will always use the new mpm base thread modules and place them in files without mpm_ prefix. The a2dismod utility only checks for file names, not the real thread module.
Codecov Report
@@ Coverage Diff @@
## master #2034 +/- ##
==========================================
+ Coverage 57.87% 58.49% +0.61%
==========================================
Files 13 12 -1
Lines 235 212 -23
==========================================
- Hits 136 124 -12
+ Misses 99 88 -11
Continue to review full report at Codecov.
|
tuxmea
changed the title
|
Disabling class uses another filename: |
Extends #1961 to also work correctly on Debian 10 and Ubuntu 20
tuxmea
changed the title
|
Hi @tuxmea, |
michaeltlombardi
changed the title
cegeka-jenkins
pushed a commit
to cegeka/puppet-apache
that referenced
this pull request
Jul 15, 2020
Allow switching of thread module
bastelfreak
added a commit
to bastelfreak/puppetlabs-apache
that referenced
this pull request
Dec 6, 2022
there were a few things broken: * a2dismod was called with the wrong module name. at least on apache 2.4.54 on Debian 11 it is named mpm_event, not event * the paths to the files were wrong * a2dismod already purges the .load and .conf file, the exec resources are not required (even then a file resource would probably be cleaner to purge them) * Apache needs to be reloaded, so just running this priot to apache2::service isn't enough, it needs to be a notify This initially worked but was changed from mpm_event to event in puppetlabs#1961. Then it was fixed in puppetlabs#2034 by adding the exec resources, but the actual a2dismod was still broken.
bastelfreak
added a commit
to bastelfreak/puppetlabs-apache
that referenced
this pull request
Dec 6, 2022
there were a few things broken: * a2dismod was called with the wrong module name. at least on apache 2.4.54 on Debian 11 it is named mpm_event, not event * the paths to the files were wrong * a2dismod already purges the .load and .conf file, the exec resources are not required (even then a file resource would probably be cleaner to purge them) * Apache needs to be reloaded, so just running this priot to apache2::service isn't enough, it needs to be a notify This initially worked but was changed from mpm_event to event in puppetlabs#1961. Then it was fixed in puppetlabs#2034 by adding the exec resources, but the actual a2dismod was still broken.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Extends #1961 to also work correctly on Debian 10 and Ubuntu 20