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

Setup CICD pipeline #100

Open
wants to merge 79 commits into
base: develop
Choose a base branch
from
Open

Setup CICD pipeline #100

wants to merge 79 commits into from

Conversation

hamzajaved-csiro
Copy link

@hamzajaved-csiro hamzajaved-csiro commented Sep 3, 2024

A breakdown of this PR so its easier to review:

The CICD projects are fairly similar to what we have in existing projects so not much has changed there apart from a few things in the buildspecs.

  • Base Project: Provision ECR Repo and create DocumentDb secret in Secrets Manager
  • Database Project: Provision DocumentDB Cluster
  • Build Project: Build Docker image and push to ECR repo
  • Deploy Project: Deploy the Helm chart in the lists namespace in testing cluster

Helm Chart:

  • Similar to alerts repo

CICD Project in Lists-UI folder:

  • Frontend Project: Similar approach to the seedbank repo

Confluence Doco:
https://confluence.csiro.au/display/ALASD/Developer+Pipeline+-+New+Species+Lists

@hamzajaved-csiro
Copy link
Author

The remaining issue on this branch is once the pod starts in the cluster, it attempts to connect to documentdb.
The config file has the mongodb parameters and it is stored in kube secret lists-service-config-secret in "lists" namespace.

The password for the documentdb database is stored in aws secrets manager under lists-{branch-name} i.e. lists-ci-cd in this case. username adminuser.

Debugging tips:

  • If you are just making changes to the config file, you can just delete the pod e.g kubectl delete pod ala-lists-service-xxxxx and it should pick up the new changes.
  • If the helm chart needs to be updated, make the changes in the git repo and rerun the pipeline.
  • Alternatively, if you want to deploy the helm chart from your local machine, update the values.yaml file with the image tag and ecr repo and deploy it in the lists namespace, just a matter of preference.

@brucehyslop
Copy link

I've updated the pipeline to enable deployment of feature branches (tested on feature/cicd-pipeline branch).
I would not suggest triggering pipeline on the cicd branch because it will change the hostname of deployed resource, which is currently being used by the lists UI deployed to https://lists.test.ala.org.au

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.

4 participants