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

feat(sentry): Add sentry integration for inventory #2268

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

adonispuente
Copy link
Contributor

@adonispuente adonispuente commented Oct 17, 2024

This adds the ability to upload source maps from inventory into sentry. It also adds the pointer to tell sentry which project to send the error too.

Testing is a bit tricky, as currently you need to run chrome-prod in tandem with inventory to test the actual error.

in inventory, you need to add the script "static" : "fec static" to the package.json file, and then run npm run static
You can create an easy error by adding <button onClick={() => methodDoesNotExist()}> anywhere in inventory and clicking the button.

  • In chrome youd need to have a prod account -> change webpack.config.js file env var too 'prod-stable' -> add
    this to routes ...(true && { '/apps/inventory': { host: 'http://localhost:8003', }, }),
    -> and run npm run dev.

Observer that the error then goes to the inventory-rhel project and has valuable data inside.

Here is an example: https://red-hat-it.sentry.io/issues/5999468439/?project=4507454722211840&referrer=onboarding-first-event-indicator

@adonispuente adonispuente requested a review from a team as a code owner October 17, 2024 18:51
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.19%. Comparing base (aff479f) to head (edb9907).
Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2268   +/-   ##
=======================================
  Coverage   59.19%   59.19%           
=======================================
  Files         202      202           
  Lines        5303     5303           
  Branches     1919     1919           
=======================================
  Hits         3139     3139           
  Misses       2164     2164           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

2 participants