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.

Sign up for GitHub

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

Jump to bottom

Symlink happening after service refresh #541

Closed
gwagner opened this issue Dec 31, 2014 · 4 comments
Closed

Symlink happening after service refresh #541

gwagner opened this issue Dec 31, 2014 · 4 comments

Comments

@gwagner
Copy link

gwagner commented Dec 31, 2014

It appears that the symlink from sites-available to sites-enabled is happening after the service gets refreshed. This is problematic as we need to bounce nginx manually (unless we make another subsequent change to the configs and re-run puppet)
@skull-squadron
Copy link
Contributor

@gwagner Try

sed -i '' -e "s/Service\['nginx'\]/Class\['::nginx::service'\]/" manifests/resource/vhost.pp in the root of puppet-nginx

@gwagner
Copy link
Author

gwagner commented Jan 28, 2015

Should i update to the latest master first?

@skull-squadron
Copy link
Contributor

(updated below)

On Wed, Jan 28, 2015 at 12:31 PM, Geoffrey Wagner
notifications@github.com wrote:

Should i update to the latest master first?

Reply to this email directly or view it on GitHub:
#541 (comment)

@skull-squadron
Copy link
Contributor

Could give that a go. If you're using librarian, just add a , git: 'https://github.com/jfryman/puppet-nginx' option to the end of the dependency and run librarian-puppet update.

(Email gateway comments are always plaintext.)

@jfryman jfryman closed this as completed in 4d0a2dd Feb 5, 2015
jfryman added a commit that referenced this issue Feb 5, 2015
@jfryman
Merge pull request #553 from steakknife/fix__541
6b4f35f 
closes #541 (maybe: needs user feedback)
cegeka-jenkins pushed a commit to cegeka/puppet-nginx that referenced this issue Oct 23, 2017
@skull-squadron
maybe closes voxpupuli#541
8b5c58b
cegeka-jenkins pushed a commit to cegeka/puppet-nginx that referenced this issue Oct 23, 2017
@jfryman
Merge pull request voxpupuli#553 from steakknife/fix__541
dc02ad1 
closes voxpupuli#541 (maybe: needs user feedback)
Slm0n87 pushed a commit to Slm0n87/puppet-nginx that referenced this issue Mar 7, 2019
@skull-squadron
maybe closes voxpupuli#541
a5e736b
Slm0n87 pushed a commit to Slm0n87/puppet-nginx that referenced this issue Mar 7, 2019
@jfryman
Merge pull request voxpupuli#553 from steakknife/fix__541
be23151 
closes voxpupuli#541 (maybe: needs user feedback)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@skull-squadron @gwagner