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

Temporarily update which special issue the EP Merge App removes when some steps fail #2642

Closed
amylai-va opened this issue Feb 20, 2024 · 1 comment
Assignees
Labels

Comments

@amylai-va
Copy link

amylai-va commented Feb 20, 2024

Typically, when certain steps within the merge process fails, we need to remove the EMP special issue. This ticket tracks work to temporarily change the special issue that we remove from EMP to RRD so that we can proceed with EP400 e2e testing in va.gov staging / VBMS UAT.

Acceptance Criteria

  • when a step fails in the merge process and the special issue needs to be removed from the EP400 contention, the EP Merge App removes the RRD special issue from the contention

Notes about work

  • At the moment, EP400 claims are unable to be established in va.gov staging with the EMP special issue since it's not a valid special issue identifier according to EVSS
  • EVSS is planning to add EMP as a valid special issue and estimates that it will be available in PINT/staging on February 29th
  • This ticket will enable us to proceed with EP400 e2e testing in the meantime
@amylai-va amylai-va changed the title Temporarily update EP Merge App to change which special issue is removed when some steps fail Temporarily update which special issue the EP Merge App removes when some steps fail Feb 20, 2024
@amylai-va amylai-va added this to the EP400 Merge MVP milestone Feb 20, 2024
@dfitchett dfitchett self-assigned this Feb 21, 2024
@dfitchett
Copy link
Contributor

This implementation that was completed in the workaround allows us to change the special issue code via environment variable or just default to EMP. So when we do the deployment, we can just have VRO set an env variable for us to change it to RRD. This also renders #2647 unnecessary.

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

2 participants