Beta testing fixes for first-time AWS deployment (ArcGIS/DC deployment)#1
Closed
cstirry wants to merge 4 commits into
Closed
Beta testing fixes for first-time AWS deployment (ArcGIS/DC deployment)#1cstirry wants to merge 4 commits into
cstirry wants to merge 4 commits into
Conversation
…g with lambda zip)
cstirry
changed the title
Author
|
Met with Srihari on 4/21, he was aware of the two main bugs and believed they had been fixed in the CLI tools, but it was unclear if those fixes were in the repo yet |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Bugs Fixed
S3 terraform state bucket name hardcoded and caused an issue because S3 bucket names are globally unique across all AWS accounts (confirm). The default name
opencontext-terraform-state(cli/commands/configure.py, terraform/bootstrap/variables.tf, terraform/aws/main.tf) gave me an error that already claimed, got a 403 and 409 error. Suggested fix to add an interactive prompt in opencontext configure that suggests a unique name based on the city slug and creates the bucket automatically if it doesn't exist.terraform validate fails before Lambda zip exists. Pre-deploy validation runs terraform validate which references
lambda-deployment.zip, but that file isn't created until the packaging step inside deploy so can't deploy on a first-time run. Suggested fix to remove terraform validate from pre-deploy checks (or edit it), terraform plan (which runs after packaging) catches the same configuration errors (cli/commands/validate.py).Other Potential Issues (Not Fixed Yet)
opencontext test incorrectly skips API Gateway URL testing, citing auth requirement even when no custom domain is configured and the URL works fine via curl
opencontext test marks 404 responses as passing
pyenv rehash lock on Mac blocks opencontext command after install (workaround: rm ~/.pyenv/shims/.pyenv-shim && pyenv rehash).
Getting Started, README, and Quickstart docs need some updates. Recommend deleting the Quickstart doc to avoid confusion.
Potential Nice to Haves
Deployment engineers might find uv-based install instructions useful, new deployers could be guided to set up a uv virtual environment from the start
Deployment engineers might find some sort of evaluation module useful, its easy to confirm that MCP set up and connected in Claude, but hard to know if results are as expected.
Testing
With a few fixes, deployed successfully to AWS staging environment with ArcGIS plugin pointed at https://opendata.dc.gov/. Verified with curl and Claude Connector.