Add "include image" option into Notification settings. ngalert

[tuhnu9089]

Why is this component needed:
ngalert is a good continuation of the alerting in grafana, but getting alerts without images is inconvenient.
In my opinion "include image" is a very useful function.

---

• Is/could it be used in more than one place in Grafana?
  This feature is already available for simple alerts (without ngalert)

Where is/could it be used?:
The include image should be shown in the "contact point" -> "Notification settings" configuration.
/alerting/notifications/receivers/[NAME]/edit?alertmanager=[NAME]

---

• Post screenshots possible.
• It has a single use case.
• It is/could be used in multiple places.

Implementation (Checklist meant for the person implementing the component)

• Component has a story in Storybook.
• Props and naming follows our style guide.
• It is extendable (rest props are spread, styles with className work, and so on).
• Uses theme for spacing, colors, and so on.
• Works with both light and dark theme.

Documentation

• Properties are documented.
• Use cases are described.
• Code examples for the different use cases.
• Dos and don'ts.
• Styling guidelines, specific color usage (if applicable).

[gotjosh]

I'm turning this into a discussion as it is a feature request. For context, we've completely deprecated the support for images in Alerting.

With enough support, I'm happy to start digging up more context on why we removed it and what kind of solutions we can propose moving forward.

But I don't think this is needed anymore. We've included a way to customise your notifications via templating and you get a direct link to the queries (including graphs) that generated this alert. With those two, the alert itself should have enough context to help you dig what you need.

[alsotop]

There appears to be three other discussion threads started about the same topic:

• Add "include image" option into Grafana Alerting #38030
• Grafana Alerting - Include image of visualized query with the alert notification #41015
• add "include image" option into ngalert #38003

I would say that it makes sense to try and bring these topics together if we want to see that there is a growing demand for this feature to return.

I'd like to put out there that I agree with other posters that images in alerts (even if they are optional) can be a very powerful thing and something that I find has been stopping me from moving from legacy alerts to ngalert. The images provided a great way to really quickly get a lot of context about an issue that may be occurring, including some previous history of how the panel got to that point. I can appreciate that "Grafana 8 alerts are no longer directly tied to panel queries." (ref) but for where they are or can be, the image can be very useful, especially in situations such as Slack notifications.

I'd like to discover how likely it is that Image Rendering will return to ngalerts. Given that Grafana 8.3.0 has now defaulted to ngalerts, I'd rather not stay stuck on the legacy alert system if image notifications are not anywhere near in the pipeline/future of Grafana development.

[gotjosh]

@alsotop thanks for digging up. Given #38030 appears to be the done with the most upvotes. I'm going to close all the others in favour of this one to keep the discussion centralised.