Skip to content

8 background service #8

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
Open
wants to merge 39 commits into
base: master
Choose a base branch
from
Open

8 background service #8

wants to merge 39 commits into from

Conversation

@mtomsu
Copy link

@mtomsu mtomsu commented Sep 23, 2018 •
edited
Loading

To address Issue #6 in services/client/server.js:

var io = require('socket.io')(server, {
  pingInterval: config.HEARTBEAT_INTERVAL,
  pingTimeout: config.HEARTBEAT_TIMEOUT
});

nathanpeck added 30 commits July 6, 2018 11:40
@nathanpeck
Using localstack/localstack to mimic multiple AWS services locally
ea6f6fe
@nathanpeck
Another service behind an SNS topic and SQS queue
e1a0ca0
@nathanpeck
Adding deployment for new background service
d554b03
@nathanpeck
Fixing a duped name in the code builds
5404864
@nathanpeck
Fixing missing bits in the pipeline
558af34
@nathanpeck
Removing an uneeded resource dependency in the message cards service …
db2fdaa 
…template
@nathanpeck
Adding some missing environment variables
61d2774
@nathanpeck
Trying moving all the templates into a subfolder
cec5994
@nathanpeck
Fixing service name reference in message cards
4e8b0ec
@nathanpeck
Adding SNS topic, SQS queue, subscription, and env variable plumbing
b17fea6
@nathanpeck
Fixing references in the resources template
58cac2f
@nathanpeck
Fixing queue URL which should have been a queue ARN
cf820e8
@nathanpeck
The SNS to SQS subscription also needs to be an ARN
30ff748
@nathanpeck
Fixing reference to get the env variable value
47f38aa
@nathanpeck
Adding missing SSM IAM permission to the pipeline role
a6069d0
@nathanpeck
Fixing silly issue where both builds were sending images to the same ECR
b7aaf5b
@nathanpeck
Updating the pipeline again
270ba8f
@nathanpeck
Fixing up the Datadog agent integration
42cd907
@nathanpeck
Test out adding some spans for custom tracing
9dbc593
@nathanpeck
Trying to fix up the custom tracing
90eb40f
@nathanpeck
Fixing span tagging
7776a6c
@nathanpeck
Adding analyzed spans to the tracing agent
28026b1
@nathanpeck
Trying to fix the span details in Datadog
025bb9a
@nathanpeck
Woops
a26f4cb
@nathanpeck
Maybe I need to make use of the scope manager
b5e6a97
@nathanpeck
Adding tracking to all the client side handlers
93ec6d1
@nathanpeck
Trying to turn on the analyzed spans
6f2ead6
@nathanpeck
More attempts
7b80948
@nathanpeck
Modifying the Datadog settings some more
eb074cb
@nathanpeck
Modifying the environment variables and adding a debug line
e578849
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mtomsu @nathanpeck