Skip to content

Conversation

@arosenkranz
Copy link
Collaborator

Description

  • Adds ddtrace to Gemfile
  • Adds datadog-tracer.rb initializer file set with env vars
  • Replaces restore database file with updated one
  • Replaces storage dir to match db restore
  • Removes mini rack profiler due to conflict with ddtrace during runtime

Note: Also created a new Storedog admin login (email: admin@storedog.com / password: password)

Comment on lines 36 to 40
# gem 'bullet'
# gem 'rack-mini-profiler', require: false
# gem 'flamegraph'
# gem 'stackprof'
# gem 'memory_profiler'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you should remove the comments, and move the ddtrace gem under # monitoring

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done, i kept it outside of the development/test block just in case we ever run this in "production"

@stephen-w-pracy stephen-w-pracy merged commit 8b190d7 into main Sep 20, 2022
@stephen-w-pracy stephen-w-pracy deleted the WEB-2688-ddtrace-ruby branch September 20, 2022 21:47
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.

4 participants