Release/1.5.1 #247
Merged
Release/1.5.1 #247
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
torimcd
approved these changes
Oct 10, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
[1.5.1]
Update to poetry.lock file to fix issue when building Docker container. #246
Description
GitHub Issue #246 - When building the Hydrocron Docker container the
RUN ~/.local/bin/poetry lock --no-updatecommand returnslist index out of range.Overview of work done
After reproducing the issue locally, I found that I could build the container after updating the
poetry.lockfile with thepoetry updatecommand. The updatedpoetry.lockfile is included in this release which will hopefully allow the container to be built when merging tomainand deploying to OPS.Overview of verification done
Existing unit tests pass.
Overview of integration done
Deployed to UAT environment to test build and confirmed no impact to existing functionality by running local tests against the UAT environment and the API benchmark tests.
PR checklist:
See Pull Request Review Checklist for pointers on reviewing this pull request