Skip to content
New issue

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

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

Already on GitHub? # to your account

v2.7.6 passenger module and nginx.conf issue #420

Closed
dedicatted opened this issue Jul 12, 2016 · 3 comments
Closed

v2.7.6 passenger module and nginx.conf issue #420

dedicatted opened this issue Jul 12, 2016 · 3 comments

Comments

@dedicatted
Copy link

dedicatted commented Jul 12, 2016

here is my conf:

    "nginx": {
      "repo_source" : "passenger",
      "package_name" : "nginx-extras",
      "user": "webserver",
      "init_style": "init",
      "source": {
        "modules": [
          "nginx::passenger"
        ],
        "use_existing_user" : true
      },
      "passenger": {
        "install_method" : "package",
        "ruby": "/home/webserver/.rvm/gems/ruby-2.2.2@my_rails/wrappers/ruby"
      }

while installation I do see:

STDERR: Configuration file '/etc/init.d/nginx'
 ==> File on system created by you or by a script.
 ==> File also in package provided by package maintainer.
   What would you like to do about it ?  Your options are:
    Y or I  : install the package maintainer's version
    N or O  : keep your currently-installed version
      D     : show the differences between the versions
      Z     : start a shell to examine the situation

I've checked out both version of a file, and new version that goes with passenger not correct, it has default user 'user www-data;' directive.

@mikelax
Copy link

mikelax commented Oct 26, 2016

@dedicatted I ran into the same issue and just solved it this am. I think you need to remove the source attribute as you want only the nginx-extras package. What tripped me up was that I had to remove both ['nginx']['install_method'] and ['nginx']['source'].

There isn't an actual bug with the cookbook, I think the documentation just needs to be improved in the passenger section of the README.

Here is my config

  "nginx" => {
    "init_style" => "upstart",
    "default_site_enabled" => false,
    "version" => "1.10.2",
    "keepalive_timeout" => "301",
    "status" => {
      "port" => "8090"
    },
    "repo_source" => "passenger",
    "package_name" => "nginx-extras",
    "passenger" => {
      "version" => "5.0.30",
      "install_method" => "package"
    }
  }

@tas50
Copy link
Contributor

tas50 commented Sep 17, 2017

Thanks for opening this issue. Just today we merged the changes from the chef_nginx fork of this cookbook back to master here. It’s a pretty huge change set and includes over a year of active development that resulted in several major releases. I think there’s a very high chance that this issue has been resolved with that work and at this point I’m going to close this issue out. I’d encourage you to checkout the master branch and please open this issue back up if you’re still having the problem there.

Here’s the changes that were just merged in: #435

@tas50 tas50 closed this as completed Sep 17, 2017
@lock
Copy link

lock bot commented Apr 25, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Apr 25, 2020
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants