Skip to content

Mark ScheduleAggregationWorker as idempotent

What does this MR do?

This marks the Namespaces::ScheduleAggregationWorker as idempotent. It was already idempotent, because it checks the existence of the Namespace::AggregationSchedule before doing anything and it uses safe_find_or_create to avoid race conditions.

This adds specs proving that it is idempotent.

It also adds the worker to the DROPPABLE_QUEUES so we can use it to try out deduplicating jobs.

Needed for: gitlab-com/gl-infra/scalability#218 (closed)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Edited by Bob Van Landuyt

Merge request reports