Skip to content

feat: dual-push to ECR and ACR, deploy to Azure Container App (PINA-8941 WP2)#133

Closed
ClaraTschamon wants to merge 1 commit into
mainfrom
feat/pina-8941-acr-dual-push
Closed

feat: dual-push to ECR and ACR, deploy to Azure Container App (PINA-8941 WP2)#133
ClaraTschamon wants to merge 1 commit into
mainfrom
feat/pina-8941-acr-dual-push

Conversation

@ClaraTschamon

Copy link
Copy Markdown

Summary

Part of PINA-8941 WP2 — adds ACR push and Azure Container App deployment alongside the existing ECR push.

  • Builds the image once, tags it for both ECR and ACR
  • Pushes to ECR (unchanged)
  • Logs in to Azure via service principal, pushes to treelyyieldtablesstaging.azurecr.io
  • Runs az containerapp update to deploy the new image to the staging Container App

Prerequisites

The terraform PR for the apps workspace must be applied first — it creates the service principal and stores AZURE_CLIENT_ID, AZURE_CLIENT_SECRET, AZURE_TENANT_ID, AZURE_SUBSCRIPTION_ID as secrets in the ecr-push GitHub environment.

Test plan

  • Terraform apps workspace applied (secrets visible in ecr-push environment)
  • Push to main triggers this workflow
  • Image appears in both ECR and treelyyieldtablesstaging.azurecr.io
  • az containerapp update succeeds, Container App serves the new image

🤖 Generated with Claude Code

@github-actions

Copy link
Copy Markdown

Coverage

Coverage Report
FileStmtsMissCoverMissing
src/openyieldtables
   __init__.py00100% 
   yieldtables.py730100% 
src/openyieldtables/classes
   __init__.py00100% 
   data_management.py42588%40, 47, 76, 89, 95
src/openyieldtables/models
   __init__.py10100% 
   yieldtable.py370100% 
src/openyieldtables/utils
   __init__.py10100% 
   utils.py150100% 
TOTAL169597% 

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.

1 participant