Skip to content

default container image fixes#3217

Merged
ikreymer merged 6 commits intomainfrom
image-fixes
Mar 11, 2026
Merged

default container image fixes#3217
ikreymer merged 6 commits intomainfrom
image-fixes

Conversation

@ikreymer
Copy link
Copy Markdown
Member

  • test: use latest crawler now dedupe now that 1.12 has been released
  • remove unused value
  • chart: use pinned version of redis and kvrocks to avoid version issues, since pull policy is set to IfNotPresent

In general, should only use unpinned (:latest) images if pull policy is Always, to avoid issues where different versions may be present on different nodes.

- test: use latest crawler now dedupe now that 1.12 has been released
- chart: use pinned version of redis and kvrocks to avoid version issues, since pull policy is set to IfNotPresent
@ikreymer ikreymer requested a review from tw4l March 11, 2026 01:18
@tw4l
Copy link
Copy Markdown
Member

tw4l commented Mar 11, 2026

I pushed a commit to simply remove the dedupe crawler channel since we already have 2 channels for testing specifying a different chanel and they all point to the same image now anyway.

Re-running nightly tests from this branch post change just to be safe: https://github.com/webrecorder/browsertrix/actions/runs/22956693705

@ikreymer ikreymer merged commit 1792663 into main Mar 11, 2026
43 of 45 checks passed
@ikreymer ikreymer deleted the image-fixes branch March 11, 2026 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants