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

Updated init script for Amazon and CentOS #1156

Closed
wants to merge 2 commits into from
Closed

Updated init script for Amazon and CentOS #1156

wants to merge 2 commits into from

Conversation

@ItalyPaleAle
Copy link

@ItalyPaleAle ItalyPaleAle commented Apr 8, 2015

This fixes #1100

The script contains the fixes from the "centos" one to change user.
In addition, contains @ObjectIsAdvantag 's fix for PM2_HOME

EgoAleSum added 2 commits Apr 8, 2015
This fixes #1100
The script contains the fixes from the "centos" one to change user.
In addition, contains @ObjectIsAdvantag 's fix for PM2_HOME
EgoAleSum EgoAleSum
@soyuka
Copy link
Collaborator

@soyuka soyuka commented Apr 9, 2015

Could you please take a look at #1120 (centos startup fix) to see if it corrects the same issue you had regarding centos? If not, please comment on there so we get along with those pull requests, thanks!

@claflico
Copy link

@claflico claflico commented Apr 15, 2015

What version of CentOS would this be for? I'm trying to puppetize the install and configuration of pm2 as a service on CentOS 7 and the deployment fails when I use the pm2-init-centos.sh file as a template.

@ItalyPaleAle
Copy link
Author

@ItalyPaleAle ItalyPaleAle commented Apr 16, 2015

@claflico what error do you get?

@claflico
Copy link

@claflico claflico commented Apr 16, 2015

Since CentOS 7 uses systemd Puppet wants to use "systemctl enable name" to define a service and that fails because it can't find the /usr/lib/systemd/system/pm2.service file. I was able to resolve it by doing away with using the chckonfig/pm2-init-centos.sh method and using the pm2.service file method instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

4 participants