feat(GrafanaAlertRuleGroup): add support for Grafana-managed recording rules

[miinsun]

see #1862

I’ve added a new Record type to the GrafanaAlertRuleGroup Spec.
I’ve also updated the controller to handle From and Metric variables within the Record type.

Is there anything else I need to do???

Fixes: #1862

[CLAassistant]

All committers have signed the CLA.

[weisdd]

Thanks for the contribution!

This functionality is not activated by default in self-hosted / Enterprise Grafana and results in a failure to apply changes by the operator (when Record is passed).
Thus, it'd be great to have a full example added to examples/ including the feature toggle and related configuration.
Also, an e2e-test would be handy to make sure we catch failures should the Grafana API change in future.

[miinsun]

Thanks for your feedback;) Looks like I missed adding the test code. I'll update the PR soon with the example and e2e test as you suggested.

[sachinjkattoor]

Thanks a lot @miinsun . @weisdd looking forward to getting this released :) Super excited that this will help to avoid a lot of manual effort

[weisdd]

@miinsun Thanks for the update. I've pushed a few changes to your branch (feature toggle configuration was missing, test data needed to be added to a different file). For the rest, it should be good to be merged.

@theSuess any closing thoughts? (e.g. whether we should keep the recording rule example in a different folder)

[theSuess]

Happy with merging this as-is and updating docs later on

[miinsun]

It was my first time using Chainsaw, and honestly, I struggled with writing test cases. But now I see how it should be done—it looks great! I learned a lot while reviewing your code. Thanks @weisdd

[sachinjkattoor]

@theSuess @weisdd As I understand v5.17.0 was scheduled to release yesterday but I do also see a last open PR (#1818) for this milestone.

Any rough idea on when the v5.17.0 release can be/would be done?

[weisdd]

@sachinjkattoor It's expected to happen today.