-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add aarch64 runner #16
Merged
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
* Add a new GitHub Actions workflow for AArch64 builds using ArchLinux example from https://github.com/marketplace/actions/arm-runner. This prepares an ArchLinux image, sets up the nix environment, and builds the project for AArch64.
* Fix the matrix.machine.platform with aarch64-linux in Nix build commands.
* Move the 'Build system' task to correctly follow the cache setup.
* Updated the CI workflows to include `contents: read` and `id-token: write` permissions. Also adjusted build commands to focus on `otb-dev` target and included the use of `magic-nix-cache-action`.
* The aarch64 GitHub Actions workflow file (.github/workflows/aarch64.yml) was deleted. * This action is no more needed as the ci.yml works fine with Nix-Magic-Cache * Rather than building 2 derivation i.e. otb-dev (all modules) and otb (minimal modules) which takes a lot of time for ARM64 build and gets timed out so we build only one derivation i.e otb-dev which contains all the modules.
daspk04
added a commit
that referenced
this pull request
Aug 31, 2024
* [ADD] Add support for multiple new OTB remote modules * Enabled optional integration of various OTB remote modules like MLUtils, Prefetch, and Phenology. * [ADD] Add otb-dev and otb-dev-docker packages * Add new `otb-dev` package build options for all the remote modules. * Also added `otb-dev-docker` for image based on all the remote modules. * [ADD] Add OTB_APPLICATION_PATH to environment variables * Add OTB_APPLICATION_PATH so that applications are importable via native python api of OTB * Additionally add a USER as `otbuser` and HOME directory for docker images * [FIX] Update the Makefile command for the updated docker package * [CI] Update CI build commands to include both the builds for otb modules * Builds OTB with all the remote modules and also build with minimal modules * [REFAC] Make all remote modules to be not activated by default * [DOC] Update Docker build instructions and handle remote modules * Add documentation for examples on remote modules, * Add some note on OTBTF related to TF library * [FIX] Fix typos in OTB default options * [FIX] Fix the wrong package name in CI build for OTB * [DOC] Correct typo in README.md * Add aarch64 runner (#16) * [CI] Add AArch64 GitHub Action Workflow * Add a new GitHub Actions workflow for AArch64 builds using ArchLinux example from https://github.com/marketplace/actions/arm-runner. This prepares an ArchLinux image, sets up the nix environment, and builds the project for AArch64. * [FIX] Fix Aarch64 workflow build system * Fix the matrix.machine.platform with aarch64-linux in Nix build commands. * [FIX] Fix build steps in aarch64 workflow * Move the 'Build system' task to correctly follow the cache setup. * [CI] Add permissions to CI workflows and update build steps * Updated the CI workflows to include `contents: read` and `id-token: write` permissions. Also adjusted build commands to focus on `otb-dev` target and included the use of `magic-nix-cache-action`. * [CI] Remove deprecated aarch64 GitHub Actions workflow * The aarch64 GitHub Actions workflow file (.github/workflows/aarch64.yml) was deleted. * This action is no more needed as the ci.yml works fine with Nix-Magic-Cache * Rather than building 2 derivation i.e. otb-dev (all modules) and otb (minimal modules) which takes a lot of time for ARM64 build and gets timed out so we build only one derivation i.e otb-dev which contains all the modules. * [REFAC] Move remote module to their own separate nix file for better readability * Move the remote module to its own separate file for better readability, also add information regarding remote module code LICENSE and copyright information * Updated related configurations in `flake.nix` and `docker.nix` to include licensing information and appropriate fetchGit instructions.
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.
Test Aarch64 Linux Runner for OTB based on https://github.com/marketplace/actions/arm-runner