department-of-veterans-affairs / va.gov-team

Public resources for building on and in support of VA.gov. Visit complete Knowledge Hub:
https://depo-platform-documentation.scrollhelp.site/index.html
281 stars 198 forks source link

[CST] Create new feature toggles for LH migration #64004

Closed jerekshoe closed 1 year ago

jerekshoe commented 1 year ago

Background

Currently there is a single feature toggle used for the CST LH migration. This feature toggle determines whether we use the LH Benefits Claims API or the EVSS equivalent when making requests related to claims. Because of a few different situations that have come up, it makes sense for us to split the feature toggle so that can manage our release in a more fine-grained manner. We will be splitting these based on the specific endpoints that are being called with 3 different feature toggles:

Context

Scenarios that led to this decision: 1) Timeouts on the show endpoint are causing us to delay the entire rollout and we are currently stuck at 5% while waiting for LH to switch their documents provider from EVSS to the LH Benefits Documents API. The index route doesn't have these same issues and other teams have started directing more traffic towards that endpoint (AuthExp is at 50% currently because they only use index) 2) 5103 - There have been multiple issues with the LH implementation of the 5103 endpoint and because of this we have had to switch to using EVSS twice so far. We have had to update the code each time to comment out the LH code and replace it with the existing EVSS code; using a feature toggle instead would make this much easier

Tasks

Acceptance Criteria

jerekshoe commented 1 year ago

Progress: