Restore LFS files when building images #211
Merged
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
The current build process fails to include model weights in worker images because we are failing to restore LFS files. Fixes #208
rafaspadilha
approved these changes
Nov 28, 2024
lonnes
approved these changes
Nov 28, 2024
rafaspadilha
added a commit
that referenced
this pull request
Nov 29, 2024
renatolfc
added a commit
that referenced
this pull request
Nov 29, 2024
This re-releases #206 and #211 now using the release pipeline, which also updates the prod images in the Container Registry. ---- This PR marks the full release of FarmVibes.AI's source code with an open source license. This PR also introduces several significant updates to the development environment, build workflows, and continuous integration processes. The most important changes include the addition of a development container configuration, updates to GitHub workflows for building and testing, and the introduction of new settings for code linting and formatting. ### Development Environment Updates: * [`.devcontainer/devcontainer.json`](diffhunk://#diff-24ad71c8613ddcf6fd23818cb3bb477a1fb6d83af4550b0bad43099813088686R1-R62): Added a new devcontainer configuration with custom VSCode settings and extensions, Docker mounts, and environment variables. * [`.devcontainer/post-create.sh`](diffhunk://#diff-bcf66a848607b49d2a45edfad434f6808017ee9d18f5a20dde6670745010d89aR1-R71): Added a post-create script to set up the development environment, including SSH configuration, Python virtual environment setup, and installation of necessary tools and dependencies. ### Build and Deployment Workflows: * [`.github/workflows/base-build.yml`](diffhunk://#diff-034a91a8f3709fafd1219e587ce389e3a39338ac6adbe543d524bcb1e84b2cf3R1-R64): Added a new workflow for building and pushing base Docker images, including steps for Azure CLI login and updating image tags. * [`.github/workflows/docker-build.yml`](diffhunk://#diff-3414847e2ad632333f775cabb810f0dc0df61a570365df34750a08b00912fe82R1-R41): Introduced a workflow for building and pushing service Docker images on push events to the `dev` and `main` branches. * [`.github/workflows/release-to-main.yml`](diffhunk://#diff-b55efbebc7173bd35e8aa625b689d11a521f0ea54f064ae7a2be9cbf5c5fa1a4R1-R54): Added a workflow to handle the release process, including retagging Docker images and updating the default image tag in the codebase. ### Continuous Integration: * [`.github/workflows/lint-test.yml`](diffhunk://#diff-1dccad827a8e5157df1dcdb70b73e454524c2586ebc64576af82cb348d478892R1-R197): Created a comprehensive workflow for linting, type checking, and testing the codebase across multiple packages and environments. ### Code Linting and Formatting: * [`.ruff.toml`](diffhunk://#diff-b3236d4cb260ca6acce083493fcdc42816f732fc3d5aefb263a3c3fe9db4f81fR1-R17): Added configuration for the `ruff` linter, specifying linting rules, file exclusions, and formatting preferences. These changes aim to ensure FarmVibes.AI's longevity, to streamline the development process, ensure consistent code quality, and automate the build and deployment pipelines. --------- Co-authored-by: FarmVibes Release Pipeline <farmvibesaicd@microsoft.com> Co-authored-by: Alex Crown <acrown@microsoft.com> Co-authored-by: Bruno Silva <brunosilva@microsoft.com> Co-authored-by: Eduardo Rodrigues <edrodrigues@microsoft.com> Co-authored-by: Jessica Wolk <Jessica.Wolk@microsoft.com> Co-authored-by: Leonardo Nunes <lnunes@microsoft.com> Co-authored-by: Naga Bilwanth Gangarapu <Naga@zensa.co> Co-authored-by: Rafael Padilha <rpadilha@microsoft.com> Co-authored-by: Rick Gutierrez <ricardgu@microsoft.com> Co-authored-by: Roberto de Moura Estevão Filho <robertode@microsoft.com> Co-authored-by: Roberto Santos <robsantos@microsoft.com> Co-authored-by: Sara Malvar <saramalvar@microsoft.com> Co-authored-by: Naga Bilwanth Gangarapu <82965480+v-ngangarapu@users.noreply.github.com>
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.
The current build process fails to include model weights in worker images because we are failing to restore LFS files.
Fixes #208