-
Notifications
You must be signed in to change notification settings - Fork 8
fix: container build for arm64 and add devcontainer settings for docker in docker #292
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
Conversation
|
This test run generated changes to the mock modules in this PR. Please review the changes and merge if appropriate, see #293. |
|
This test run generated changes to the mock modules in this PR. Please review the changes and merge if appropriate, see #294. |
|
This test run generated changes to the mock modules in this PR. Please review the changes and merge if appropriate, see #295. |
|
This test run generated changes to the mock modules in this PR. Please review the changes and merge if appropriate, see #296. |
…rge) (#296) This PR was created by the AVM governance test workflow after pre-commit was run. Please review if the changes should be made to the mock modules: <https://github.com/Azure/avm-terraform-governance/actions/runs/17795914169> Co-authored-by: Azure Verified Modules [bot] <1049636+azure-verified-modules[bot]@users.noreply.github.com>
…merge) (#295) This PR was created by the AVM governance test workflow after pre-commit was run. Please review if the changes should be made to the mock modules: <https://github.com/Azure/avm-terraform-governance/actions/runs/17795914169> Co-authored-by: Azure Verified Modules [bot] <1049636+azure-verified-modules[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just a minor question
This pull request makes several improvements to container build and test workflows, enhances cross-platform support for ARM64 builds, and updates development environment configurations for better usability and security. The most significant changes are grouped below.
CI/CD Workflow Improvements
.github/workflows/container-release.yml) is now triggered on pull requests to themainbranch, in addition to tags and manual dispatches. This allows container builds and tests to run automatically on PRs..github/workflows/legacy-container-test.yml) has been removed, consolidating container build and test logic into the main workflow.ARM64 Cross-Platform Build Support
container/Dockerfile.azterraform,container/Dockerfile.build) have been updated to install and use the correct cross-compilation toolchain (gcc-aarch64-linux-gnu) when building for ARM64, and to set the appropriate compiler environment variables. This ensures reliable builds for multiple architectures. [1] [2]Dev Container Configuration Enhancements
.devcontainer/devcontainer.jsonfiles inmanaged-files/root,tests/terraform-azure-avm-res-mock, andtests/terraform-azurerm-avm-res-mockhave been updated to add container environment variables, grant additional capabilities (SYS_PTRACE, seccomp unconfined), use host networking, and mount the Docker socket. These changes improve debugging, container management, and compatibility with Docker-in-Docker workflows.Documentation and Editor Settings
./avm pre-commit, ensuring proper workflow for contributors.These updates collectively improve build reliability, developer experience, and CI/CD automation for the project.