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

Fix pm2-process restart error #11

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Fix pm2-process restart error #11

wants to merge 17 commits into from

Conversation

peterMuriuki
Copy link
Collaborator

@peterMuriuki peterMuriuki commented Sep 7, 2020

  • use weareinteractive.pm2 role to install and configure pm2 processes
  • Test using podman

@peterMuriuki peterMuriuki changed the title Re- adding vagrant as provisioner in test Fix pm2-process restart error Oct 2, 2020
@peterMuriuki peterMuriuki removed the WIP Work in progress label Oct 2, 2020
@peterMuriuki
Copy link
Collaborator Author

created issue to look at idempotence and linting here: #13

@moshthepitt
Copy link
Contributor

what is causing the test to fail?

ignore_errors: true
tags:
- molecule-idempotence-notest # noqa 301
- name: pm2 configure
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of doing this here, can we do it in the meta? Here is are examples: 1, 2.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some of the envs that should be passed are outputs of havin run some tasks for example. pm2 start command needs to be passed the path where the index file is, the index file is an output of running build.
The problem is that I understand dependencies will run before the role actually executes.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool, makes sense

peterMuriuki and others added 3 commits October 2, 2020 16:25
Forced due to reason that ll containers running on github actions run on systemd-unsupported docker containers
Make it possible to deploy several apps to the same host
Copy link
Member

@ukanga ukanga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quite a number of places where code is commented out, could we either remove it or make relevant parts part of the code?

Are we using both Github actions and drone or only one of them is going to be relevant?

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

Successfully merging this pull request may close these issues.

4 participants