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

Add Sentry #545

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Add Sentry #545

wants to merge 10 commits into from

Conversation

thaddeus
Copy link
Contributor

@thaddeus thaddeus commented Jul 20, 2024

Adds Sentry for the Express server, Cron jobs, and the Manager interface

Merged in the other currently deployed PR

The index file got auto formatted when I saved it - so most of the changes are just formatting. We should add a prettierrc or prettier.config in this and other projects, along with VSCode suggested extensions IMO.

@thaddeus thaddeus requested a review from a team as a code owner July 20, 2024 18:20
Copy link
Contributor

👋 Thanks for opening a pull request!

If you are new, please check out the trimmed down summary of our deployment process below:

  1. 👀 Observe the CI jobs and tests to ensure they are passing

  2. ✔️ Obtain an approval/review on this pull request

  3. 🚀 Branch deploy your pull request to production

    Comment .deploy on this pull request to trigger a deploy. If anything goes wrong, rollback with .deploy main

  4. 🎉 Merge!

src/tarkov-data-manager/index.mjs Dismissed Show dismissed Hide dismissed
@thaddeus
Copy link
Contributor Author

.deploy

Copy link
Contributor

⚠️ Cannot proceed with deployment

  • reviewDecision: REVIEW_REQUIRED
  • commitStatus: FAILURE

This is usually caused by missing PR approvals or CI checks failing

@Razzmatazzz
Copy link
Member

.unlock production

Copy link
Contributor

🔓 There is currently no production deployment lock set

@Razzmatazzz
Copy link
Member

.deploy

Copy link
Contributor

Deployment Triggered 🚀

Razzmatazzz, started a branch deployment to production

You can watch the progress here 🔗

Branch: add-sentry

Copy link
Contributor

Deployment Results ⚠️

Warning: deployment status is unknown, please use caution

@Razzmatazzz
Copy link
Member

.deploy

Copy link
Contributor

Deployment Triggered 🚀

Razzmatazzz, started a branch deployment to production

You can watch the progress here 🔗

Branch: add-sentry

Copy link
Contributor

Deployment Results ✅

Razzmatazzz successfully deployed branch add-sentry to production

@GrantBirki
Copy link
Member

.unlock

Copy link
Contributor

🔓 Deployment Lock Removed

The production deployment lock has been successfully removed

@GrantBirki
Copy link
Member

.deploy main

Copy link
Contributor

Deployment Triggered 🚀

GrantBirki, started a branch deployment to production

You can watch the progress here 🔗

Branch: main

@GrantBirki
Copy link
Member

GrantBirki commented Jul 25, 2024

Seeing some errors on containers now that this is deployed:

wait-for-it.sh: waiting 15 seconds for database:3306
wait-for-it.sh: database:3306 is available after 0 seconds
[ { 'count(*)': 15 } ]
Database is ready

> tarkov-data-manager@2.0.0 start
> node --max-old-space-size=3000 index.mjs

(node:42) Warning: Accessing non-existent property 'format' of module exports inside circular dependency
(Use `node --trace-warnings ...` to show where the warning was created)
(node:42) Warning: Accessing non-existent property 'format' of module exports inside circular dependency
TypeError: format is not a function
    at getDbStatement (/app/node_modules/@opentelemetry/instrumentation-mysql2/build/src/utils.js:72:15)
    at PoolConnection.query (/app/node_modules/@opentelemetry/instrumentation-mysql2/build/src/instrumentation.js:65:250)
    at /app/node_modules/mysql2/lib/pool.js:154:14
    at /app/node_modules/mysql2/lib/pool.js:49:37
    at process.processTicksAndRejections (node:internal/process/task_queues:77:11)
Error refreshing users TypeError: format is not a function
    at getDbStatement (/app/node_modules/@opentelemetry/instrumentation-mysql2/build/src/utils.js:72:15)
    at PoolConnection.query (/app/node_modules/@opentelemetry/instrumentation-mysql2/build/src/instrumentation.js:65:250)
    at /app/node_modules/mysql2/lib/pool.js:154:14
    at /app/node_modules/mysql2/lib/pool.js:68:16
    at PoolConnection.<anonymous> (/app/node_modules/mysql2/lib/connection.js:818:13)
    at Object.onceWrapper (node:events:634:26)
    at PoolConnection.emit (node:events:519:28)
    at ClientHandshake.<anonymous> (/app/node_modules/mysql2/lib/connection.js:137:14)
    at ClientHandshake.emit (node:events:519:28)
    at ClientHandshake.execute (/app/node_modules/mysql2/lib/commands/command.js:49:10) {
  fatal: true
}

tdm@tdm-vm:~$ sudo docker ps
CONTAINER ID   IMAGE                                     COMMAND                  CREATED          STATUS                          PORTS                                                                                                                                    NAMES
d77e8c2195f2   tarkov-data-manager_caddy                 "sh /app/start.sh"       13 minutes ago   Up 7 minutes                    0.0.0.0:80->80/tcp, :::80->80/tcp, 0.0.0.0:443->443/tcp, :::443->443/tcp, 0.0.0.0:8443->8443/tcp, :::8443->8443/tcp, 443/udp, 2019/tcp   caddy-service
81441efa592a   tarkov-data-manager_backup                "docker-entrypoint.s…"   13 minutes ago   Up 7 minutes                                                                                                                                                             tarkov-data-manager_backup_1
77d9007d90c8   tarkov-data-manager_tarkov_data_manager   "docker-entrypoint.s…"   13 minutes ago   Restarting (1) 19 seconds ago                                                                                                                                            tarkov_data_manager
3859740cdf38   tarkov-data-manager_database              "docker-entrypoint.s…"   13 minutes ago   Up 7 minutes                    0.0.0.0:3306->3306/tcp, :::3306->3306/tcp, 33060/tcp                                                                                     tarkov-data-manager_database_1

Copy link
Contributor

Deployment Results ✅

GrantBirki successfully deployed branch main to production

@GrantBirki
Copy link
Member

.unlock

Copy link
Contributor

🔓 Deployment Lock Removed

The production deployment lock has been successfully removed

@Razzmatazzz
Copy link
Member

.deploy

Copy link
Contributor

github-actions bot commented Aug 5, 2024

Deployment Triggered 🚀

Razzmatazzz, started a branch deployment to production

You can watch the progress here 🔗

Branch: add-sentry

Copy link
Contributor

github-actions bot commented Aug 5, 2024

Deployment Results ✅

Razzmatazzz successfully deployed branch add-sentry to production

@Razzmatazzz
Copy link
Member

.deploy main

Copy link
Contributor

github-actions bot commented Aug 5, 2024

Deployment Triggered 🚀

Razzmatazzz, started a branch deployment to production

You can watch the progress here 🔗

Branch: main

Copy link
Contributor

github-actions bot commented Aug 5, 2024

Deployment Results ✅

Razzmatazzz successfully deployed branch main to production

@Razzmatazzz
Copy link
Member

When I deployed this, it appeared to deploy properly, but the data manager never came back online.

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.

3 participants