Page MenuHomePhabricator

[terraform] remove unnecessary redeploy
ClosedPublic

Authored by will on Aug 15 2024, 8:13 PM.
Tags
None
Referenced Files
F2766935: D13095.diff
Thu, Sep 19, 3:50 PM
Unknown Object (File)
Sat, Sep 7, 1:02 PM
Unknown Object (File)
Thu, Sep 5, 9:26 AM
Unknown Object (File)
Wed, Sep 4, 1:36 PM
Unknown Object (File)
Tue, Sep 3, 1:04 AM
Unknown Object (File)
Sat, Aug 31, 8:40 PM
Unknown Object (File)
Sat, Aug 31, 3:10 PM
Unknown Object (File)
Fri, Aug 30, 2:09 PM
Subscribers

Details

Summary

These parts of the script are no longer necessary as they are taken care of by
the terraform apply. Originally had the redeployment to take care of redeploying
the primary service after all nodes were taken down, but terraform apply already restores 1 desired node for primary.

Additionally, when the image is updated to a new one (which currently is assumed to cause a migration), local provisioner logic
in the terraform apply will ensure that the primary service is available before the secondary service is launched

Depends on D13094

Test Plan

ran the script and confirmed migration occurred successfully and secondary nodes only applied after primary node was healthy

Diff Detail

Repository
rCOMM Comm
Branch
script_changes
Lint
No Lint Coverage
Unit
No Test Coverage