Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Dockerfile: build psycopg2 in separate container #561

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

pjonsson
Copy link
Contributor

@pjonsson pjonsson commented Dec 3, 2023

This avoids having to install development libraries in the production container, which makes the image ~300M smaller.


📚 Documentation preview 📚: https://datacube-explorer--561.org.readthedocs.build/en/561/

This avoids having to install development libraries
in the production container, which makes the image
~300M smaller.
Copy link

codecov bot commented Dec 3, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (d7fa351) 85.88% compared to head (1188536) 85.88%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #561   +/-   ##
========================================
  Coverage    85.88%   85.88%           
========================================
  Files           26       26           
  Lines         3457     3457           
========================================
  Hits          2969     2969           
  Misses         488      488           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@omad omad left a comment

Choose a reason for hiding this comment

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

Nice!

@omad omad merged commit 88780f7 into opendatacube:develop Dec 4, 2023
5 checks passed
@pjonsson pjonsson deleted the dockerfile-builder branch December 4, 2023 08:18
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants