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

Remove SidekiqStatsJob #19990

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Remove SidekiqStatsJob #19990

wants to merge 2 commits into from

Conversation

rmtolmach
Copy link
Contributor

Summary

  • This work is behind a feature toggle (flipper): NO

This job doesn't run anymore. We still get Sidekiq data and it is accessible in Datadog, but this job is not how it's generated. I'm removing it as part of department-of-veterans-affairs/va.gov-team#99587. See that issue for more detailed info.

Related issue(s)

Testing done

None.

Screenshots

Sidekiq metrics:
image

What areas of the site does it impact?

Sidekiq jobs. But really nothing, because this doesn't run anymore.

Acceptance criteria

  • No error nor warning in the console.

Copy link

Error: A file (or its parent directories) was deleted but its reference still exists in CODEOWNERS. Please update the .github/CODEOWNERS file and delete the entry for the Offending file: spec/sidekiq/sidekiq_stats_job_spec.rb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove SidekiqStatsJob
2 participants