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.

Sign up for GitHub

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

Jump to bottom

Dockerfile revisions #5994

Merged
merged 5 commits into from
Mar 29, 2023
Merged

Dockerfile revisions #5994

merged 5 commits into from
Mar 29, 2023

Conversation

dlpierce
Copy link
Contributor

Changes proposed in this pull request:

  • Install and enable jemalloc in the Dockerfile as suggested by @orangewolf
  • Update Alpine, Ruby, Rubygems and FITS in the Dockerfile
  • Configure FITS to use Alpine's mediainfo as suggested in the FITS release notes
  • Disable Tika in FITS for better characterization performance

@samvera/hyrax-code-reviewers

@dlpierce
Build and enable jemalloc
c1f9631 
Use of jemalloc is expected to improve performance, particularly for multithreaded sidekiq workers.

Alpine 3.17 has an official package, but we're stuck on older versions of Alpine until we are compatible with Ruby 3.1.
@dlpierce
Update Alpine and Ruby
5a2a105
@dlpierce
Unpin default rubygems/bundler version
0bb4ace
@dlpierce
Update FITS, use system mediainfo, disable Tika for performance
b92b85c
@dlpierce
Update ruby in dassie and koppie
3d9d78e
@dlpierce dlpierce merged commit d74df8c into main Mar 29, 2023
@dlpierce dlpierce deleted the dockerfile-rev branch March 29, 2023 19:13
@dlpierce dlpierce added the notes-container Release Notes: Docker, Helm, etc label Mar 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jeremyf jeremyf jeremyf approved these changes

Assignees
No one assigned
Labels
notes-container Release Notes: Docker, Helm, etc
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dlpierce @jeremyf