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

dassie/koppie alignment and added bundle caching #6325

Merged
merged 3 commits into from Sep 19, 2023
Merged

Conversation

dlpierce
Copy link
Contributor

@dlpierce dlpierce commented Sep 19, 2023

Summary

dassie/koppie alignment and added bundle caching

Guidance for testing, such as acceptance criteria or new user interface behaviors:

  • dassie and koppie build and run from clean compose environment. (Clean volumes with docker compose down -v and remove any Gemfile.*.lock files in hyrax root and .dassie/.koppie)

Changes proposed in this pull request:

  • Use the same images for dassie/koppie dependent services.
  • Don't build an image for db_migrate, reuse the app image.
  • Use dev-entrypoint.sh to copy gems from image build to /app/bundle volume to speed up runtime bundle install. Only set in compose to avoid affecting CircleCI build.
  • /bin is volume mounted to allow easier dev of entrypoints, etc.

@samvera/hyrax-code-reviewers

- Use the same images for dassie/koppie dependent services.
- Don't build an image for db_migrate, reuse the app image.
- Use `dev-entrypoint.sh` to copy gems from image build to `/app/bundle` volume to speed up runtime `bundle install`. Only set in compose to avoid affecting CircleCI build.
- `/bin` is volume mounted to allow easier dev of entrypoints, etc.
@dlpierce dlpierce added the notes-container Release Notes: Docker, Helm, etc label Sep 19, 2023
- Use the same images for dassie/koppie dependent services.
- Don't build an image for db_migrate, reuse the app image.
- Use `dev-entrypoint.sh` to copy gems from image build to `/app/bundle` volume to speed up runtime `bundle install`. Only set in compose to avoid affecting CircleCI build.
- `/bin` is volume mounted to allow easier dev of entrypoints, etc.
Copy link
Contributor

@alishaevn alishaevn left a comment

Choose a reason for hiding this comment

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

I tested this locally and it's fixed the issue I was having here.

@dlpierce dlpierce merged commit e4890a8 into main Sep 19, 2023
4 checks passed
@dlpierce dlpierce deleted the dc-cached-bundle branch September 19, 2023 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
notes-container Release Notes: Docker, Helm, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants