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

Deployment tracking #748

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Deployment tracking #748

wants to merge 7 commits into from

Conversation

davidsiaw
Copy link
Contributor

@davidsiaw davidsiaw commented Dec 8, 2022

This PR aims to consolidate tracking to a cron job that will run every several minutes to poll the statuses of all deployments

This is to

  • avoid using the current recursive system
  • avoid getting rate limited by having too many status polls
  • make it easier to maintain deployment errors and tracking

How to test

There are 2 things to test here, migration of the old barcelona stacks and functionality of new stacks.

  1. kaiser up a local barcelona on the master
  2. deploy a district and then deploy the barcelona hello stack
  3. update to this branch
  4. re-deploy the district
  5. we should have a ServiceDeploy object in rails console
  6. we should be able to run bcn:deployment_check and it should say it is completed if the stack is complete or incomplete if it is not yet updated..

For new stacks, simply deploy the hello stack on the current branch, and check that bcn:deployment_check will complete the deployment notification for compleetion.

@davidsiaw davidsiaw changed the title Add deployment tracking models Deployment tracking Dec 8, 2022
@davidsiaw davidsiaw changed the base branch from master to deployment-tracking-models December 8, 2022 09:32
@davidsiaw davidsiaw marked this pull request as ready for review December 12, 2022 04:09
Base automatically changed from deployment-tracking-models to master December 15, 2022 11:00
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