fzhao99

Results 14 comments of fzhao99

> Based on my digging around... Looks like [v3 of `applicationinsights`](https://www.npmjs.com/package/applicationinsights) uses [Azure Monitor OpenTelemetry](https://learn.microsoft.com/en-us/azure/azure-monitor/app/opentelemetry-nodejs-migrate?tabs=upgrade). > > Our function tests [are failing](https://github.com/CDCgov/prime-simplereport/actions/runs/8992793122/job/24703361748?pr=7624#step:4:16) because `tagOverrides` does not seem to be supported...

Smoke tested on dev4 and things look good, but @alismx and @shanice-skylight wanted to double check that the comment about devops reviewing the package pins here for security reasons isn't...

> @fzhao99 @shanice-skylight @emyl3 Looks like removing the pick pushes this version back down to 3.0 which is a problematic version. > > Our options are: > > * merge...

if anyone's reading this in the future [this wiki page](https://github.com/CDCgov/prime-simplereport/wiki/How-to-Review-and-Test-Pull-Requests-with-Terraform-Changes) was made as a testing reference

Got a false positive run here: https://github.com/CDCgov/prime-simplereport/actions/runs/7588051271/job/20669739705#step:14:26

Set of symptoms in the syphilis MMG: https://phinvads.cdc.gov/vads/ViewValueSet.action?oid=2.16.840.1.114222.4.11.7225

Writeup for investigation / last working hypothesis is available [here](https://docs.google.com/document/d/1PgU25kIP5iNMmW5GVAVyVRxeEip2iBs1H5iu95OGGEM/edit) for whoever eventually picks up this ticket. Godspeed brave engineer.

Related ticket: https://github.com/orgs/CDCgov/projects/15/views/1?pane=issue&itemId=59021834

Updated to a new MSW version [here](https://github.com/CDCgov/prime-simplereport/pull/7720 )

Found these old tickets while doing some other digging: https://github.com/CDCgov/prime-simplereport/issues/2668 https://github.com/CDCgov/prime-simplereport/issues/4788