Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixed apache group for SUSE/SLES Systems (checked for SLES11/12) #1613

Merged
merged 1 commit into from
Jul 13, 2017
Merged

fixed apache group for SUSE/SLES Systems (checked for SLES11/12) #1613

merged 1 commit into from
Jul 13, 2017

Conversation

pseiler
Copy link
Contributor

@pseiler pseiler commented May 2, 2017

I checked SLES11 and SLES12 (and openSUSE 42.2) and the group for apache is "www" instead of "wwwrun" on all SUSE systems.

The user is wwwrun and the group www
$ id wwwrun
uid=30(wwwrun) gid=8(www) groups=8(www)

@pseiler
Copy link
Contributor Author

pseiler commented May 3, 2017

The CI check failed because an error regarding Ubuntu 14.04 and not SUSE hosts.

These are the errors:

export BEAKER_set=docker/ubuntu-14.04
[...]
rspec ./spec/acceptance/vhost_spec.rb:1569 # apache::vhost define fastcgi applies cleanly
rspec ./spec/acceptance/vhost_spec.rb:1629 # apache::vhost define fastcgi File "/etc/apache2/sites-enabled/25-test.server.conf" should contain "FastCgiExternalServer localhost -socket /tmp/fast/1234"
rspec ./spec/acceptance/vhost_spec.rb:1630 # apache::vhost define fastcgi File "/etc/apache2/sites-enabled/25-test.server.conf" should contain "<Directory \"/tmp/fast\">"

@hunner hunner merged commit e26cf35 into puppetlabs:master Jul 13, 2017
hunner added a commit that referenced this pull request Jul 13, 2017
fixed apache group for SUSE/SLES Systems (checked for SLES11/12)
cegeka-jenkins pushed a commit to cegeka/puppet-apache that referenced this pull request Jul 15, 2020
fixed apache group for SUSE/SLES Systems (checked for SLES11/12)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants