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

Update New Relic redis monitoring configuration #922

Open
10 of 14 tasks
rioug opened this issue Mar 26, 2024 · 2 comments · May be fixed by #970
Open
10 of 14 tasks

Update New Relic redis monitoring configuration #922

rioug opened this issue Mar 26, 2024 · 2 comments · May be fixed by #970
Assignees

Comments

@rioug
Copy link
Contributor

rioug commented Mar 26, 2024

Description

This came up as part of investigating infinite loading bug : openfoodfoundation/openfoodnetwork#11752 (comment)

New Relic provides monitoring for redis, however it's configured with the wrong port. We need to be monitoring redis on port 6380. Redis monitoring is accessible in New Relic via Infrastructure -> Third party services -> Redis

Server to fix :

TODO

Fix ansible script so the correct configuration is used next time we provision a server

Fix

As an ofn-admin, ssh to the server:

  • cd /etc/newrelic-infra/integrations.d
  • update the PORT entries to 6380 in redis-config.yml
  • restart new relic service : systemctl restart newrelic-infra

Expected Behavior

Relic monitoring reports redis activity :
redis_good

Actual Behaviour

Relic monitoring shows a flat graph :
redis_bad

@dacook
Copy link
Member

dacook commented Mar 26, 2024

I wondered if we can set this up with Ansible?

Even better, it should probably be an .env variable, which both openfoodnetwork and newrelic could refer to.

@rioug rioug self-assigned this Mar 27, 2024
@rioug rioug moved this from All the things to In Dev 💪 in OFN Delivery board Mar 27, 2024
@rioug
Copy link
Contributor Author

rioug commented Mar 27, 2024

All instance be au_prod (new relic not setup for third party monitoring ) and in_prod (new relic not setup) have been manually updated and are now reporting in new relic.

Leaving the ticket open till we fix the ansible script

@rioug rioug moved this from In Dev 💪 to Dev ready 👋 in OFN Delivery board Mar 27, 2024
@rioug rioug moved this from Dev ready 👋 to In Progress ⚙ in OFN Delivery board Dec 11, 2024
@rioug rioug linked a pull request Dec 11, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress ⚙
Development

Successfully merging a pull request may close this issue.

2 participants