feat: migrate to serca-pipelines and mirror registry#1443
Closed
andersondario-er wants to merge 16 commits intodevelopfrom
Closed
feat: migrate to serca-pipelines and mirror registry#1443andersondario-er wants to merge 16 commits intodevelopfrom
andersondario-er wants to merge 16 commits intodevelopfrom
Conversation
- Update Dockerfile.mt to use serca-artifact-registry mirror - Replace gundi-workflows with serca-pipelines for CI/CD - Add lint and test jobs before build - Use new WIF for authentication
- Remove /library/ from docker-mirror paths - Push to both padas-app/er-mt and serca-artifact-registry/earthranger - Use @main branch of serca-pipelines
- Switch to serca-pipelines reusable pipeline - Removes custom lint/test jobs (handled by pipeline) - Uses dual registry push for transition period - ArgoCD deployment disabled for now
- Fix branch reference from @feature/argocd-sync-workflow to @main - Add dual registry push (padas-app + serca-artifact-registry) - Update base-image-registry to docker-mirror path
- Replace hybrid inline/reusable approach with full pipeline - Use pipeline-kubernetes-node-pr.yml for consistency - Removes inline lint/test jobs (handled by pipeline) - Add 'deploy' label condition to build job
Simplify workflow by hardcoding build-args instead of using separate config job with outputs
andersondario-er
had a problem deploying
to
padas-app
GitHub Actions
Failure
- Remove develop-workflow.yml (legacy kubectl-based deployment) - Update main.yml with full ArgoCD logic inlined: - Build and push Docker image to GAR - Update image tag in serca-argocd-apps - Trigger ArgoCD sync and wait for health - Update pr-envs.yml with PR environment deployment: - Build Docker image for PRs with 'deploy' label - Sync PR environment via ArgoCD ApplicationSet Note: ArgoCD logic is inlined because das-web-react is public and cannot call serca-pipelines (internal). Once serca-pipelines is public, these workflows can be simplified to use the reusable pipelines like das-react-admin and report-form-builder.
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.
Summary
Changes
Dockerfile.mt: Base images now pull fromeurope-west3-docker.pkg.dev/serca-artifact-registry/docker-mirror/.github/workflows/main.yml: Usesserca-pipelines/_docker-build.yml.github/workflows/pr-envs.yml: Usesserca-pipelines/_docker-build.ymlTest plan
🤖 Generated with Claude Code