Remove WireGuard step - Githubissues

cmu-delphi / delphi-epidata

An open API for epidemiological data.

https://cmu-delphi.github.io/delphi-epidata/

MIT License

102 stars 68 forks source link

Remove WireGuard step #1557

Open korlaxxalrok opened 2 weeks ago

korlaxxalrok commented 2 weeks ago

addresses issue(s) #1556

Summary:

Removes the step to install/set up WireGuard from the performance testing Actions workflows.

Reason:

The WireGuard setup, though pretty stable overall, recently failed in our test environment and resulted in a lot of spurious monitoring alerts. We don't need it anymore because we now run these workflows via local runner that has more direct access to the test environment.

NOTE: There is also an Actions secret used in these workflows of DB_CONN_STRING that has already been updated and tested.

Prerequisites:

[x] Unless it is a documentation hotfix it should be merged against the dev branch
[x] Branch is up-to-date with the branch to be merged with, i.e. dev
[x] Build is successful
[x] Code is cleaned up and formatted

sonarcloud[bot] commented 2 weeks ago

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud