Skip to content

Comments

Migrate PR workflow to inline Docker build with dual registry#1460

Closed
andersondario-er wants to merge 1 commit intodevelopfrom
feat/use-kubernetes-pr-pipeline
Closed

Migrate PR workflow to inline Docker build with dual registry#1460
andersondario-er wants to merge 1 commit intodevelopfrom
feat/use-kubernetes-pr-pipeline

Conversation

@andersondario-er
Copy link
Contributor

Summary

  • Replace gundi-workflows/build_docker.yml with inline Docker build logic (das-web-react is a public repo and cannot call reusable workflows from serca-pipelines which is internal)
  • Remove deploy label gate: build runs on every PR
  • Add dual registry push: serca-artifact-registry (primary) + padas-app (secondary)
  • Remove GitHub environment declaration
  • Add GHA cache (cache-from/cache-to) for faster builds
  • Image tags: {sha} and pr-{sha} pushed to both registries
  • Add concurrency group with cancel-in-progress

Test plan

  • Open a test PR and verify config + build jobs run without deploy label
  • Verify Docker image is pushed to both serca-artifact-registry and padas-app
  • Verify image tags match expected format ({sha} and pr-{sha})
  • Verify GHA cache is populated on first run and used on subsequent pushes

🤖 Generated with Claude Code

Replace gundi-workflows build_docker.yml with inline Docker build
logic. das-web-react is a public repo and cannot call reusable
workflows from serca-pipelines (internal visibility).

Key changes:
- Inline Docker build with dual registry push (serca + padas-app)
- Remove deploy label gate: build runs on every PR
- Remove GitHub environment declaration
- GHA cache for faster builds
- Image tags: {sha} and pr-{sha} pushed to both registries
@andersondario-er andersondario-er self-assigned this Feb 18, 2026
@andersondario-er
Copy link
Contributor Author

Recreating with dash branch name (no slashes)

@andersondario-er andersondario-er deleted the feat/use-kubernetes-pr-pipeline branch February 18, 2026 17:36
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