Skip to content

Capture cog load times using sentry performance tracking#2976

Merged
wookie184 merged 4 commits into
mainfrom
capture-cog-load-times
Mar 26, 2024
Merged

Capture cog load times using sentry performance tracking#2976
wookie184 merged 4 commits into
mainfrom
capture-cog-load-times

Conversation

@ChrisLovering
Copy link
Copy Markdown
Member

I ran this once locally to make sure it all worked, and it looks great https://python-discord.sentry.io/performance/?project=2724195&statsPeriod=14d

@ChrisLovering ChrisLovering force-pushed the capture-cog-load-times branch from e96413f to f9328d2 Compare March 26, 2024 17:35
Copy link
Copy Markdown
Contributor

@wookie184 wookie184 left a comment

Choose a reason for hiding this comment

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

Nice, I'd thought about using this for commands, but figured it would be too noisy given some commands wait for user responses etc.

This seems good though.

@ChrisLovering ChrisLovering force-pushed the capture-cog-load-times branch from 0e0d5e9 to 059fb00 Compare March 26, 2024 18:01
Copy link
Copy Markdown
Contributor

@wookie184 wookie184 left a comment

Choose a reason for hiding this comment

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

Looks good

@wookie184 wookie184 merged commit 726c16c into main Mar 26, 2024
@wookie184 wookie184 deleted the capture-cog-load-times branch March 26, 2024 18:13
@sentry
Copy link
Copy Markdown

sentry Bot commented Mar 28, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ **ExtensionFailed: Extension 'bot.exts.info.python_news' raised an error: ResponseCodeError: Status: 500 Response: ** bot.exts.info.python_news View Issue
  • ‼️ ExtensionFailed: Extension 'bot.exts.backend.sync' raised an error: ServerDisconnectedError: Server disconnected bot.exts.backend.sync View Issue
  • ‼️ ExtensionFailed: Extension 'bot.exts.moderation.infraction.superstarify' raised an error: ResponseCodeError: Statu... bot.bot in load_extension View Issue

Did you find this useful? React with a 👍 or 👎

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