Skip to content

Conversation

@richardkiene
Copy link
Contributor

@richardkiene richardkiene commented Aug 22, 2025

Summary

  • Add multi-architecture support for Docker image builds
  • Simplify 1Password CLI setup by removing automatic service account creation
  • Improve documentation for manual 1Password configuration

Changes

Multi-architecture Build Support

  • Added platforms: linux/amd64,linux/arm64 to the Docker build-push-action
  • QEMU and Docker Buildx were already configured in the workflow

1Password Setup Simplification

  • Removed automatic service account creation logic from setup-1password.sh
  • Script now only verifies authentication status and provides clear manual setup instructions
  • Updated .env.example with step-by-step guide for creating service accounts
  • Enhanced README documentation with detailed 1Password configuration instructions

Impact

  • The devcontainer can now run natively on Apple Silicon Macs without Rosetta emulation
  • 1Password setup is more predictable and secure with manual token configuration
  • Clearer documentation helps users understand the authentication options

Testing

The multi-architecture build will be tested when this PR triggers the GitHub Actions workflow.

Add platforms specification to Docker build-push-action to build for both linux/amd64 and linux/arm64 architectures. This allows the devcontainer to run natively on Apple Silicon Macs without Rosetta emulation.
- Remove automatic service account creation from setup-1password.sh
- Update script to only verify authentication status and provide manual instructions
- Improve documentation in .env.example with clear step-by-step instructions
- Update README with detailed service account creation guide
- Focus on manual token configuration for better security and predictability
@richardkiene richardkiene merged commit 23d916d into main Aug 22, 2025
1 check passed
@richardkiene richardkiene deleted the feature/multi-arch-docker-builds branch August 22, 2025 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants