Conversation
- Introduced the `amazon-q` module, which installs and configures Amazon's AI coding assistant. - Added `main.tf` for Terraform resource definitions, including variables and local configurations. - Created `main.test.ts` for testing the module's functionality, ensuring required resources are created. - Included a comprehensive `README.md` with usage instructions, prerequisites, and examples for running Amazon Q in various configurations.
…iable - Updated the `experiment_auth_tarball` variable in `main.tf` to remove the validation condition, allowing it to default to null. This change allows the module to build and for the validation to be done on the template side
- Changed the default value of the `experiment_auth_tarball` variable in `main.tf` from null to "tarball" to provide a more meaningful default for users, and to allow for validation at the template level
|
@hugodutka Can you Review this PR for the Amazon-Q Module? I have tested it and its working perfectly, with some minor changes. |
| <details> | ||
| <summary><strong>How to generate the Amazon Q auth tarball (step-by-step)</strong></summary> | ||
|
|
||
| **1. Install and authenticate Amazon Q on your local machine:** |
There was a problem hiding this comment.
Don't they support an easier more user friendly auth method?
There was a problem hiding this comment.
I wish - that’s what’s recommended in the community forums. https://community.aws/content/2uZYCp6BNJJgBaRnw3Nie6i8r0l/putting-amazon-q-developer-in-a-docker-container
amazon-q/README.md
Outdated
| - If neither is set, Amazon Q runs in the foreground. | ||
| - For more details, see the [main.tf](./main.tf) source. | ||
|
|
||
| <!-- TODO: Add Screenshot of Amazon-Q In Action --> |
| fi | ||
|
|
||
| # Run with screen if enabled | ||
| if [ "${var.experiment_use_screen}" = "true" ]; then |
There was a problem hiding this comment.
Have you tested the module with screen? I only ever ran it with tmux.
There was a problem hiding this comment.
Yeah I tested it with screen as well it worked fine in my test but I'll double check when I get to my workstation in about an hour
amazon-q/main.tf
Outdated
| PREV_DIR="$PWD" | ||
| TMP_DIR="$(mktemp -d)" | ||
| cd "$TMP_DIR" | ||
| curl --proto '=https' --tlsv1.2 -sSf "https://desktop-release.q.us-east-1.amazonaws.com/${var.amazon_q_version}/q-x86_64-linux.zip" -o "q.zip" |
There was a problem hiding this comment.
We should probably do an architecture check here since we only support x64 Linux
There was a problem hiding this comment.
If they have an arm64 asset, let's do that too. I have seen recent PRs of users adding arm64 support to other modules.
|
We are moving to https://github.com/coder/regitsry. Please move your PR there. Thanks |
- Added `system_prompt` and `task_prompt` variables - Updated the script to detect system architecture and download the appropriate version of Amazon Q for x86_64 and arm64. - Refactored the prompt handling to use a local variable combining system and task prompt
- Added a screenshot of Amazon Q in action to enhance visual understanding. - Removed unverified instructions regarding `zstd` installation for macOS. - Add variables for System and Task Prompts
|
Did this get migrated? |
|
yes to coder/registry#95 |
4 participants
Description
Adds the
amazon-qmodule for automated installation and management of Amazon Q in Coder workspaces.tmuxorscreen.Credit: @hugodutka (amazon-q module)