Build 2024-05-20

[twhetzel]

Overview

New build.

Pre-merge checklist

Documentation

Was the documentation added/updated under docs/?

• [ ] Yes
• [x] No, updates to the docs were not necessary after careful consideration

QC

Was the full pipeline run before submitting this PR using sh run.sh make build-mondo-ingest on this branch (after docker pull obolibrary/odkfull:dev), and no errors occurred?

• [ ] Yes
• [x] No, there are no functional (code-related) changes to the pipeline in the PR, so no re-run is necessary

New Packages

Were any new Python packages added?

• [ ] Yes, requirements.txt.full was updated and an ODK issue submitted
• [x] No

Were any other non-Python packages added?

• [ ] Yes, Dockerfile updated and an ODK issue submitted
• [x] No

PR Review and Conversations Resolved

Has the PR been sufficiently reviewed by at least 1 team member of the Mondo Technical team and all threads resolved?

• [ ] Yes

[twhetzel]

I ran sh run.sh make build-mondo-ingest, was another make command supposed to be run for this?

[matentzn]

It looks like you ran sh run.sh make build-mondo-ingest but you should run: sh run.sh make build-mondo-ingest -B if you run the whole pipeline. That should probably be mentioned in the docs as well.

[joeflack4]

Or, can do what I do and clone a copy of the repo and run the build there. I think this should have the same effect as -B, but probably the reason I started doing this is so I could continue development while the build runs in parallel.

[twhetzel]

Thanks, I'll re-run with -B and created a ticket to update the docs.