feat(post_process): Schedule less tasks to derive code mappings

[armenzg]

This adds two one-second locks to prevent scheduling many tasks that use locking internally.

This also makes some other minor changes.

[codecov]

❌ 7 Tests Failed:

Tests completed	Failed	Passed	Skipped
23574	7	23567	276

View the top 3 failed tests by shortest run time

tests.sentry.tasks.test_post_process.PostProcessGroupErrorTest::test_only_maps_a_given_issue_once_per_day

Stack Traces | 5.98s run time

#x1B[1m#x1B[.../sentry/tasks/test_post_process.py#x1B[0m:324: in test_only_maps_a_given_issue_once_per_day
    assert mock_derive_code_mappings.delay.call_count == 1
#x1B[1m#x1B[31mE   AssertionError: assert 0 == 1#x1B[0m
#x1B[1m#x1B[31mE    +  where 0 = <MagicMock name='derive_code_mappings.delay' id='140399592045408'>.call_count#x1B[0m
#x1B[1m#x1B[31mE    +    where <MagicMock name='derive_code_mappings.delay' id='140399592045408'> = <MagicMock name='derive_code_mappings' id='140399590118400'>.delay#x1B[0m

tests.sentry.api.endpoints.test_organization_events_trends_v2.OrganizationEventsTrendsStatsV2EndpointTest::test_two_projects_same_transaction_split_queries

Stack Traces | 5.99s run time

#x1B[1m#x1B[.../api/endpoints/test_organization_events_trends_v2.py#x1B[0m:414: in test_two_projects_same_transaction_split_queries
    assert len(trends_call_args_data_1[f"{project1.id},foo bar*"]) > 0
#x1B[1m#x1B[31mE   KeyError: '4555400802205700,foo bar*'#x1B[0m

tests.sentry.tasks.test_post_process.PostProcessGroupErrorTest::test_skipping_an_issue_doesnt_mark_it_processed

Stack Traces | 6.02s run time

#x1B[1m#x1B[.../sentry/tasks/test_post_process.py#x1B[0m:366: in test_skipping_an_issue_doesnt_mark_it_processed
    assert mock_derive_code_mappings.delay.call_count == 1
#x1B[1m#x1B[31mE   AssertionError: assert 0 == 1#x1B[0m
#x1B[1m#x1B[31mE    +  where 0 = <MagicMock name='derive_code_mappings.delay' id='140399386429152'>.call_count#x1B[0m
#x1B[1m#x1B[31mE    +    where <MagicMock name='derive_code_mappings.delay' id='140399386429152'> = <MagicMock name='derive_code_mappings' id='140399386439232'>.delay#x1B[0m

To view more test analytics, go to the Test Analytics Dashboard
📢 Thoughts on this report? Let us know!

[getsantry]

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!

---

"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀