log noisy AnimationBucketWarning to firehose, not honeybadger

[davidsbailey]

The following HB error sometimes happens > 15K times / day: https://app.honeybadger.io/projects/3240/faults/37290310#notice-summary

We want to add a pagerduty alert for anything happening more than 1K/day. So that we don't get paged every day with AnimationBucketWarnings, I'm proposing we move it to Firehose instead.

To verify before merging:

• verify locally the the firehose event is getting logged.

[davidsbailey]

cc @joshlory

[epeach]

Looks good! Thanks for migrating this logging.

[davidsbailey]

I can't quite get a local repro that hits this path, so i've triple-checked the syntax to make sure I think it will meet the requirements needed to make it through to redshift. The repro steps i'm trying are to delete the old versions of an animation via S3 in AWS console, and then using Version History to revert a gamelab project to a version which was using older versions of that animation.

[islemaster]

While you're at it, we should probably move these two as well:

code-dot-org/shared/middleware/helpers/bucket_helper.rb

Lines 395 to 397 in 2f68873

	Honeybadger.notify(
	error_class: "#{self.class.name}Warning",
	error_message: "Restore at Specified Version Failed. Restored most recent.",

code-dot-org/shared/middleware/helpers/bucket_helper.rb

Lines 407 to 409 in 2f68873

	Honeybadger.notify(
	error_class: "#{self.class.name}Warning",
	error_message: "Restore at Specified Version Failed on deleted object. No action taken.",

That could always be a followup PR though.