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

also provide writable tmp volumes for db init check #170

Merged
merged 2 commits into from
Feb 11, 2025

Conversation

machisuji
Copy link
Member

@machisuji machisuji commented Feb 11, 2025

so used rails runner is guaranteed to work

specifically when appsignal is configured, it will try to write a log file and fail due to the read-only file system (if configured which it is by default)

This ought to fix the issue we are observing when appsignal is configured:

wait-for-db appsignal: Unable to log to '/app/log' or the '/tmp' fallback. Please check the permissions for the application's (log) directory.

It turns out, though, that this is not the main issue. The main issue is not appsignal at all but mini_magick since the latest update:

wait-for-db /usr/local/lib/ruby/3.4.0/tmpdir.rb:44:in 'Dir.tmpdir': could not find a temporary directory (ArgumentError)                                                                    
wait-for-db                                                                                                                                                               
wait-for-db     end or raise ArgumentError, "could not find a temporary directory"
wait-for-db                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^                                                                                                          
wait-for-db     from /app/vendor/bundle/ruby/3.4.0/gems/mini_magick-5.1.2/lib/mini_magick/configuration.rb:71:in 'MiniMagick::Configuration.extended'

Either way, adding the tmp volumes fixes both issues.

…ner is guaranteed to work

specifically when appsignal is configured, it will try to write a log file and fail due to the read-only file system (if configured which it is by default)
Copy link

changeset-bot bot commented Feb 11, 2025

🦋 Changeset detected

Latest commit: 27a51ce

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@openproject/helm-charts Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@oliverguenther oliverguenther merged commit 846b5a7 into main Feb 11, 2025
3 checks passed
@oliverguenther oliverguenther deleted the fix/appsignal-unable-to-log branch February 11, 2025 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants