Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/component breakout #221

Open
wants to merge 34 commits into
base: ci/code-pipeline
Choose a base branch
from

Conversation

joe-lipson
Copy link

Break out the "containers" component into:

  • base: base resources used by alerts, includes the ECR repo and secrets
  • build: build the container image, tag it and push to the ECR repo
  • deploy: deploy the container to the EKS cluster

Fixed the build error. It was failing on the docker tag command because the docker tag doesnt understand ARNs, updated the repo format

Updated all the build containers to that latest Ubuntu 22.04

Cleaned up some unused VARs that were being passed to CodeBuild in the Pipeline

There's still an error in the deploy component that needs to be looked into

[Container] 2024/06/13 06:21:57.948379 Running command helm upgrade --install ala-alerts . --set image.tag=$CLEAN_BRANCH
Error: Kubernetes cluster unreachable: the server has asked for the client to provide credentials

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.

None yet

2 participants