Skip to content
This repository was archived by the owner on Nov 14, 2022. It is now read-only.

Conversation

@james0209
Copy link
Contributor

@james0209 james0209 commented Jul 25, 2022

Summary of work

  • Create 2 main Dockerfiles: one for queue-processor; one for queue-processor with Mantid in the environment

  • Version of Mantid required (latest stable or latest nightly) can be passed as a build-arg (defaults to latest stable)

  • Dockerfile uses multi-stage builds to reduce final image size

  • Make use of conda-pack to reduce Image size. A tool that let’s you package a Conda environment into a standalone environment, with no need for the Conda toolchain. Once environment has been packaged that way, it can be copied into a new smaller image that only contains the self-contained environment

  • Remove unused function generating fake test data

@codecov
Copy link

codecov bot commented Jul 25, 2022

Codecov Report

Merging #1463 (4d5bf0a) into main (b6f467e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1463   +/-   ##
=======================================
  Coverage   89.84%   89.84%           
=======================================
  Files          17       17           
  Lines         827      827           
=======================================
  Hits          743      743           
  Misses         84       84           
Flag Coverage Δ
backend 89.84% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@james0209 james0209 changed the title 1631 mantid latest nightly ar 1631 Mantid versions: latest & nightly Jul 26, 2022
@james0209 james0209 requested a review from Pasarus August 2, 2022 08:33
@Pasarus Pasarus merged commit 05d4481 into main Aug 2, 2022
@Pasarus Pasarus deleted the 1631-mantid-latest-nightly-AR-1631 branch August 2, 2022 10:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants