Closed Bug 1879170 Opened 8 months ago Closed 4 months ago

stand up a LandoAPI container for Lando-Treestatus

Categories

(Cloud Services :: Operations: LandoAPI, task)

task

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: sheehan, Assigned: dlactin)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

After some deliberation on the best way to deploy Lando-Treestatus, it was decided that we should stand up a separate container for the Treestatus API, with a distinct domain, which allows us to avoid needing to namespace API requests under /treestatus/. This makes the migration day tasks a simple change to DNS, and should avoid cross contamination between Lando and Treestatus when either is under high load.

I've completed the work to move the Treestatus API endpoints into their own container. To deploy this to production we will need to deploy the latest Treestatus branch updates to the development branch on each of LandoAPI and LandoUI, and make the following changes to the CloudOps infra configs:

  • Create a second LandoAPI container with the same configuration as the regular LandoAPI container, except with the TREESTATUS_APP environment variable set to 1 on startup. This will load the Treestatus API endpoints instead of the standard LandoAPI endpoints.
  • Add a new domain for the Lando-Treestatus specific endpoint and point it at the new container.
  • Add a TREESTATUS_URL environment variable to the LandoUI container that points to the new domain.

I'd like to deploy these changes to the dev environment first, and then deploy to production once we have confirmation that everything is working as expected.

Assignee: nobody → dlactin
Status: NEW → ASSIGNED

A new deployment was created with the requested parameters. Pending review.

Attached file GitHub Pull Request

PR has been merged, landoapi-dev was deploying from this branch but has been reset back to the master branch now.

Treestatus changes for prod have been merged and deployed.

Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: