Rename :skipped reason to :backoff_in_effect for clarity

What does this MR do?

Renames the :skipped reason to :backoff_in_effect in Geo::Scheduler::SchedulerWorker to make the logs more descriptive.

Why was this MR needed?

As described in issue 8694, the :skipped reason in geo.log was not clear about why a job was skipped. Since the only reason for should_be_skipped? returning true is the 5-minute backoff delay from GeoBackoffDelay, the new name :backoff_in_effect makes the logs more informative for debugging.

What are the relevant issue numbers?

Closes #8694 (closed)

Does this MR meet the acceptance criteria?

  • Added a test for the new behavior
  • Changelog entry added
Edited by Michael Kozono

Merge request reports

Loading