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

Update redis chart to avoid issues with ARM #101

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

yesid-lopez
Copy link

@yesid-lopez yesid-lopez commented Jun 14, 2024

Hey there!

I was running the helm for my raspberry but when it tried to deploy redis it fails with the following error:

<jemalloc>: Unsupported system page size   
│ <jemalloc>: Unsupported system page size                                                                                                                                                                                                                                                                                                                                                                                                                                                                                             

After I bumped the redis chart version it worked:

│ 1:C 14 Jun 2024 11:50:56.448 * oO0OoO0OoO0Oo Redis is starting oO0OoO0OoO0Oo                                                              │
│ 1:C 14 Jun 2024 11:50:56.448 * Redis version=7.2.5, bits=64, commit=00000000, modified=0, pid=1, just started                             ││ 1:C 14 Jun 2024 11:50:56.448 * Configuration loaded                                                                                       ││ 1:M 14 Jun 2024 11:50:56.448 * monotonic clock: POSIX clock_gettime                                                                       │
│ 1:M 14 Jun 2024 11:50:56.449 * Running mode=standalone, port=6379.                                                                        ││ 1:M 14 Jun 2024 11:50:56.449 * Server initialized                                                                                         ││ 1:M 14 Jun 2024 11:50:56.452 * Reading RDB base file on AOF loading...                                                                    │
│ 1:M 14 Jun 2024 11:50:56.452 * Loading RDB produced by version 7.2.5                                                                      ││ 1:M 14 Jun 2024 11:50:56.452 * RDB age 59 seconds                                                                                         ││ 1:M 14 Jun 2024 11:50:56.452 * RDB memory usage when created 0.86 Mb                                                                      │
│ 1:M 14 Jun 2024 11:50:56.452 * RDB is base AOF                                                                                            ││ 1:M 14 Jun 2024 11:50:56.452 * Done loading RDB, keys loaded: 0, keys expired: 0.                                                         ││ 1:M 14 Jun 2024 11:50:56.536 * DB loaded from base file appendonly.aof.1.base.rdb: 0.085 seconds                                          │
│ 1:M 14 Jun 2024 11:50:56.536 * DB loaded from append only file: 0.085 seconds                                                             ││ 1:M 14 Jun 2024 11:50:56.536 * Opening AOF incr file appendonly.aof.1.incr.aof on server start                                            ││ 1:M 14 Jun 2024 11:50:56.536 * Ready to accept connections tcp                                                                            │
│ Stream closed EOF for immich/immich-redis-master-0 (redis)

Copy link
Member

@bo0tzz bo0tzz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I was initially going to reject this because I thought it would bring the Redis major version out of line with what we use in the official compose deployment, but it turns out that's already the case 😅

Can you bump the Immich chart version to 0.7.1?

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.

None yet

2 participants