NGWPC PI-10 Delivery: IaC rework, GHCR image workflows, slim image, docs and MkDocs upgrade - #75
Merged
Merged
Conversation
Add terraform bootstrap layer for remote state backend
create_networking toggle (create VPC or reference existing)
…e, and s3_bucket/s3_output_prefix outputs
…lidation, align CLI flags across submit/report. Update IaC docs with skip-bootstrap path and cross-account setup.
Layered Terraform IaC and batch scripts cleanup
Add GHCR publish and prune workflows
Update docs for IaC rework and GHCR, strip notebook outputs
Slim inference image with inference-only environment
biplovbhandari
changed the title
biplovbhandari
changed the title
Add dark mode, versioned docs, navigation tabs, and PR validation
biplovbhandari
changed the title
Fix mike workflow to not move latest alias on main push
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
2 participants
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.
Summary
IaC rework (Layered Terraform IaC and batch scripts cleanup NGWPC/bridge-classification#1) - Replace monolithic
terraform/with layeredinfra/terraform/(bootstrap, foundation, app). IAM and networking toggles for reusing existing infrastructure. GHCR toggle (create_ecr = false+inference_image_repo). Launch template, updated batch scripts, newsrc/terraform.pyhelper.GHCR workflows (Add GHCR publish and prune workflows NGWPC/bridge-classification#2) -
build-dev-images.ymlpublishesghcr.io/<org>/bridge-classification/inferenceon push to main (sha-<short>+devtags).prune-dev-images.ymlcleans old images weekly..dockerignoretightened.Docs and notebook cleanup (Update docs for IaC rework and GHCR, strip notebook outputs NGWPC/bridge-classification#3) - Updated deployment-runbook and aws-batch-inference docs for the IaC and GHCR changes. Added GHCR note to
build_and_push.sh. Stripped notebook outputs.Slim inference image (Slim inference image with inference-only environment NGWPC/bridge-classification#4) - Split
environment-inference.yamlfrom the full dev env. Dockerfile uses inference-only deps. Image size reduced from ~10.6 GB to ~9.0 GB.mkdocs build --strict), edit page links. Symlinked Terraform README into docs for strict build compatibility. Only tag pushes move thelatestalias.Additional documentation will be submitted in MS Word format by the end of September 2026.