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

set tzdata data_dir in config #453

Closed
wants to merge 1 commit into from
Closed

Conversation

eumel8
Copy link

@eumel8 eumel8 commented Oct 15, 2024

tzdata has a auto-update mechanism as described here If you operate asciinema in a Pod of a Kuberntes cluster with readOnlyRootFilesystem this will cause CrashLoopBackOff due the missing write permissions in the depth of the application directory. So move out this directory to /tmp/tzdata, so it can be mount by an emptyDir. At startup tzdata will fill this directory automatically:

06:43:43.238 [info] No tzdata release files found in custom data dir. Copying release file from tzdata priv dir.

Another option would be disable the tzdata update feature

Signed-off-by: Frank Kloeker <[email protected]>
@lmarti1505
Copy link

Auto update of tzdata can also be disabled via Advanced Configuration of asciinema-server described here.

In the custom.exs file you can configure the tzdata module autoupdate function.
Example in custom.exs:

import Config

# Custom configuration for tzdata module
config :tzdata, :autoupdate, :disabled

@ku1ik
Copy link
Contributor

ku1ik commented Oct 17, 2024

I don't think having auto update enabled by default is really necessary (we don't do much if at all with timezones in this project), so I believe we should just put the line @lmarti1505 suggested in the default config.exs file. And we'll just always update the tzdata package for each new tagged release of the server. Does this sound good?

@eumel8
Copy link
Author

eumel8 commented Oct 17, 2024

@ku1ik sounds good, thanks!

ku1ik added a commit that referenced this pull request Nov 3, 2024
This solves the problem described in #453
@ku1ik
Copy link
Contributor

ku1ik commented Nov 3, 2024

I've just released a new version with the auto-update disabled.

https://github.com/asciinema/asciinema-server/releases/tag/v20241103

@ku1ik ku1ik closed this Nov 3, 2024
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.

3 participants