Skip to content

Commit

Permalink
Update rfc-045-consolidate-sidekiq-usage-into-shared-gem.md
Browse files Browse the repository at this point in the history
  • Loading branch information
Tijmen Brommet committed May 17, 2017
1 parent 330527d commit 8b426f2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion rfc-045-consolidate-sidekiq-usage-into-shared-gem.md
@@ -1,6 +1,6 @@
## Problem

The Publishing Platform team is currently working on adding request tracing to all apps that use the Publishing API in a asynchronous way via Sidekiq - [https://github.com/alphagov/collections-publisher/pull/194](https://github.com/alphagov/collections-publisher/pull/194) and [https://github.com/alphagov/whitehall/pull/2567](https://github.com/alphagov/whitehall/pull/2567). This will need repeating across many apps. The implementation is sufficiently complex that a mass-change of all apps in the future is not unlikely.
The Publishing Platform team is currently working on adding request tracing to all apps that use the Publishing API in an asynchronous way via Sidekiq - [https://github.com/alphagov/collections-publisher/pull/194](https://github.com/alphagov/collections-publisher/pull/194) and [https://github.com/alphagov/whitehall/pull/2567](https://github.com/alphagov/whitehall/pull/2567). This will need repeating across many apps. The implementation is sufficiently complex that a mass-change of all apps in the future is not unlikely.

A previous shotgun surgery on apps with Sidekiq was the adding of sidekiq-statsd - [https://trello.com/c/z2aHqwS8/48-add-sidekiq-statsd-to-apps-that-use-sidekiq](https://trello.com/c/z2aHqwS8/48-add-sidekiq-statsd-to-apps-that-use-sidekiq) which needed ~10 PRs.

Expand Down

0 comments on commit 8b426f2

Please sign in to comment.