action #153859
closedPreview images for backlog status are broken when they shouldn't size:S
0%
Description
Observation¶
The spike solution generates PNG previews with unique names (to workaround client preview image caching). The drawback is that every pipeline run will also remove the previous preview hence making the old links linking a non-existing image. This may result in previews not being correct if we have notifications in Slack (or Matrix, or elsewhere).
Acceptance Criteria¶
- AC1: URL previews for the backlog status work consistently
- AC2: Preview images are updated on a regular basis
Suggestions¶
- Preview images are generated in GitHub CI
- GitHub deletes old results, and by extension images
- Research if there's a different way to use different URL parameters instead of an image with a different name to avoid the issue with previews being removed (
/foo/bar.png?d3adb33f
) - Find another way to store the preview image?
- See if certain headers or meta data can be used to address caching
Updated by livdywan about 1 year ago
- Copied from action #153466: Notification if one of the queries on https://os-autoinst.github.io/qa-tools-backlog-assistant/ is red, e.g. write email to our Slack or o3-admins from backlogger size:M added
Updated by jbaier_cz about 1 year ago
- Status changed from Workable to Feedback
Updated by okurz about 1 year ago
- Status changed from Feedback to Resolved
https://github.com/openSUSE/backlogger/pull/26 is merged and effective. We have older and current pictures still preserved in our Slack room.
Updated by livdywan about 1 year ago
- Status changed from Resolved to Feedback
okurz wrote in #note-4:
https://github.com/openSUSE/backlogger/pull/26 is merged and effective. We have older and current pictures still preserved in our Slack room.
We were not able to confirm that. I see the same preview for a link from Monday. Is this expected?
Updated by okurz about 1 year ago
Yes, I guess. We don't need Slack to archive us older pictures, only the latest status matters. Can we resolve?
Updated by dheidler about 1 year ago
Is this expected?
It should be. The timestamp get parameter is only for cache invalidation. Not for actually selecting the version.
Updated by okurz about 1 year ago
- Status changed from Feedback to Resolved
Ok, so let's resolve again