Page MenuHomePhabricator

[terraform] Deploy Tunnelbroker 0.14 to staging and prod
ClosedPublic

Authored by kamil on Jul 31 2024, 7:19 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Sep 5, 4:21 PM
Unknown Object (File)
Thu, Sep 5, 4:21 PM
Unknown Object (File)
Thu, Sep 5, 4:21 PM
Unknown Object (File)
Thu, Sep 5, 4:21 PM
Unknown Object (File)
Sun, Sep 1, 10:12 AM
Unknown Object (File)
Fri, Aug 30, 9:58 PM
Unknown Object (File)
Fri, Aug 30, 2:30 AM
Unknown Object (File)
Wed, Aug 28, 12:39 PM
Subscribers

Details

Summary

Deploying newest Tunnelbroker image.

For staging this is invalid device token work.
For prod this is invalid device work and APNs, FCM and Web Push notifs.

Test Plan

Run terraform plan -target aws_ecs_service.tunnelbroker on staging and prod.

Also, tested logic end-to-end: ENG-8508.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil held this revision as a draft.
kamil published this revision for review.Jul 31 2024, 7:19 AM
This revision is now accepted and ready to land.Jul 31 2024, 7:30 AM

Didn't we conclude in a recent retro that we always want to deploy updates images to staging first, before production? Did we make an exception here because Tunnelbroker is barely live in production?

Didn't we conclude in a recent retro that we always want to deploy updates images to staging first, before production? Did we make an exception here because Tunnelbroker is barely live in production?

Most of the logic I deployed on prod (notifs work) was on the staging for some time and was tested by my and @marcin,
For the invalid device token work, I deployed this on staging and spent most of the day testing this before releasing, additionally, this is notifs- related error handling code, it's an additional feature and is not affecting existing logic which is live, and for some time it won't be executed so decided to deploy both.

But overall, I agree with you - I should've bump staging to 0.14 and prod to 0.12 (not 0.14) to keep our deploying process consent, thanks for pointing that out.