Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Alert] Avoid caching charts when sending as text #19134

Closed
chonyy opened this issue Mar 13, 2022 · 1 comment
Closed

[Alert] Avoid caching charts when sending as text #19134

chonyy opened this issue Mar 13, 2022 · 1 comment
Assignees
Labels
#bug Bug report

Comments

@chonyy
Copy link

chonyy commented Mar 13, 2022

Hi all,

I have set up an alert to send a chart as text. However, the values in the chart, which is the content of the alert, sometimes are cached. It’s sending the chart value of the previous alert.
Is this an expected behavior? Ideally, whenever the alert is triggered, I would like to get the real-time value of the chart. Is there a way to guarantee to get the real-time value for every single alert?

After this PR, I believe we could already bust the cache when alerting the screenshot of the dashboard. I'm looking for a similar feature that will allow the user to bust cache when sending a chart as text when alerting.

CC the contributor of the mentioned PR, @hughhhh @betodealmeida .
Hope someone could help me look into this. Thanks 🙏

@chonyy chonyy added the #bug Bug report label Mar 13, 2022
@chonyy chonyy changed the title [Alert] Avoid caching the content of the alert [Alert] Avoid caching the chart when sending as text Mar 13, 2022
@chonyy chonyy changed the title [Alert] Avoid caching the chart when sending as text [Alert] Avoid caching charts when sending as text Mar 13, 2022
@rusackas
Copy link
Member

@chonyy that PR from @hughhhh does sound like it did exactly what you want... busting the cache for charts in dashboards on alerts (however they're sent) if I'm interpreting correctly. Are you still facing this issue, or is this resolved by now so we can close out the fairly stale bug report (which might be more of a feature request anyway)?

@chonyy chonyy closed this as completed Jan 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
#bug Bug report
Projects
None yet
Development

No branches or pull requests

3 participants