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

Integration of new StatusCreatedAt GSI in web application #4287

Open
srtalbot opened this issue Sep 18, 2024 · 13 comments
Open

Integration of new StatusCreatedAt GSI in web application #4287

srtalbot opened this issue Sep 18, 2024 · 13 comments
Assignees
Labels

Comments

@srtalbot
Copy link
Contributor

srtalbot commented Sep 18, 2024

closes #3051

Context on why we had to create a new GSI: #4205 (comment)

Deployment plan (for both Staging and Production):

  1. Review, update, merge and release (feat: add CreatedAt field to Vault CONF# items forms-terraform#810)
  2. Write an run migration script to update all existing Vault CONF# items
  3. Review, update, merge and release (feat: update new Status#CreatedAt field when changing Status in DynamoDB Vault entries #4257)
  4. Write and run migration script to update existing Vault NAME# items. This script should be executed in a period where there is the lowest traffic possible on GC Forms (e.g. Saturday at noon) as there is a very very very minimal risk of race condition where both the script and a user of GC Forms would try to update the exact same item in the Vault.
  5. Review, merge and release:
  6. Review, merge and release:
  7. Review, merge and release:
  8. Review, merge and release:
  9. Review, merge and release (make sure all changes above have been running for a couple of days in Production):
@srtalbot srtalbot added the Core label Sep 18, 2024
@craigzour craigzour changed the title Migration index Integration of new StatusCreatedAt GSI in web application Sep 19, 2024
@anikbrazeau
Copy link
Member

Related issue: #3051

@craigzour
Copy link
Contributor

Deployment plan has been updated after I discovered a scenario that was not originally taken into account. @srtalbot @Abi-Nada

@srtalbot
Copy link
Contributor Author

Part 2 is now in prod!

On part 3 of the deployment plan. Testing before merging to staging.

@craigzour
Copy link
Contributor

Part 3 is now released in Production.

Will now be testing part 4 in Staging.

@craigzour
Copy link
Contributor

Part 4 migration script was successfully run in Production on Saturday (Nov 30th)

@craigzour
Copy link
Contributor

Added new step 5 that will need to be released before the old step 5 (which is now step 6).

@craigzour
Copy link
Contributor

Part 6 has been deployed to Production

@craigzour
Copy link
Contributor

craigzour commented Dec 20, 2024

Part 7 is currently running in Staging and should be ready to be released in Production.

@craigzour
Copy link
Contributor

Part 7 > API is now running in Production
Next Step is Part 7 > Infra

@srtalbot
Copy link
Contributor Author

srtalbot commented Jan 7, 2025

Infra release for today so that we can release the app tomorrow!

@craigzour
Copy link
Contributor

Part 7 > Infra is now running in Production

@craigzour
Copy link
Contributor

Part 8 is now running in Staging and awaiting Production release

@craigzour
Copy link
Contributor

Part 8 is now running in Production.

The plan is now to wait for at least a week to see if everything works fine with the full integration of the new Index. After that we can proceed with Part 9 which will result in the old Index being completely deleted from DynamoDB.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants