Configure/run Sidekiq for Akkounts #304

Merged
greg merged 2 commits from feature/303-akkounts_sidekiq into master 2021-02-08 21:17:55 +00:00
Owner

Added with some help from https://github.com/mperham/sidekiq/blob/master/examples/systemd/sidekiq.service#L20

Tested/running on akkounts-1.

closes #303

Added with some help from https://github.com/mperham/sidekiq/blob/master/examples/systemd/sidekiq.service#L20 Tested/running on `akkounts-1`. closes #303
raucao added the
kredits-1
label 2021-02-03 18:54:47 +00:00
raucao added 1 commit 2021-02-03 18:54:47 +00:00
raucao requested review from greg 2021-02-03 18:54:51 +00:00
raucao reviewed 2021-02-07 11:07:52 +00:00
@ -42,6 +21,7 @@ end
package "libpq-dev"
include_recipe 'kosmos-nodejs'
include_recipe "kosmos-redis"
Author
Owner

I'm actually unsure about this. Wouldn't it be better to define this in the role runlist of the service?

I'm actually unsure about this. Wouldn't it be better to define this in the role runlist of the service?
Owner

I would leave it like this, if we want to change the recipe to not depend on redis we will need to add code to get the IP of the Redis server anyway

I would leave it like this, if we want to change the recipe to not depend on redis we will need to add code to get the IP of the Redis server anyway
Author
Owner

OK, sure. Does the rest look good to you?

OK, sure. Does the rest look good to you?
greg approved these changes 2021-02-08 21:17:39 +00:00
greg left a comment
Owner

👍

👍
greg added 1 commit 2021-02-08 21:17:44 +00:00
greg merged commit bac66d4dba into master 2021-02-08 21:17:55 +00:00
greg deleted branch feature/303-akkounts_sidekiq 2021-02-08 21:18:01 +00:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: kosmos/chef#304
No description provided.