Skip to content

[GHA] Update backend and CodeQL endpoints#3522

Merged
imnasnainaec merged 5 commits intomasterfrom
dotnet-endpoint
Jan 15, 2025
Merged

[GHA] Update backend and CodeQL endpoints#3522
imnasnainaec merged 5 commits intomasterfrom
dotnet-endpoint

Conversation

@imnasnainaec
Copy link
Collaborator

@imnasnainaec imnasnainaec commented Jan 15, 2025

Our CodeQL results were failing to upload (e.g., https://github.com/sillsdev/TheCombine/actions/runs/12777064526/job/35617067435#step:7:486 and https://app.stepsecurity.io/github/sillsdev/TheCombine/actions/runs/12777064526?jobid=35617067435&tab=network-events). Added uploads.github.com:443 to .github/workflows/codeql.yml to fix that (e.g., https://github.com/sillsdev/TheCombine/actions/runs/12790261713/job/35655616598?pr=3522#step:7:481).

Harden Runner is only now handling all the backend/CodeQL_build network events (and still doesn't report the network events of backend/test_build -- see https://app.stepsecurity.io/github/sillsdev/TheCombine/actions/runs/12777064538?jobid=35617067433&tab=network-events). Added the missing endpoint builds.dotnet.microsoft.com:443 to .github/workflows/backend.yml. Also removed superfluous endpoint ts-crl.ws.symantec.com:80.

This change is Reviewable

@imnasnainaec imnasnainaec added the github_actions Pull requests that update GitHub Actions code label Jan 15, 2025
@imnasnainaec imnasnainaec self-assigned this Jan 15, 2025
@codecov
Copy link

codecov bot commented Jan 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.62%. Comparing base (a4d814b) to head (76bc5fe).
Report is 48 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3522      +/-   ##
==========================================
- Coverage   74.66%   74.62%   -0.05%     
==========================================
  Files         286      286              
  Lines       11002    11002              
  Branches     1339     1339              
==========================================
- Hits         8215     8210       -5     
- Misses       2405     2409       +4     
- Partials      382      383       +1     
Flag Coverage Δ
backend 83.62% <ø> (-0.10%) ⬇️
frontend 66.62% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@imnasnainaec imnasnainaec changed the title [GHA] Add builds.dotnet.microsoft.com endpoint to CodeQL workflow [GHA] Update backend and CodeQL endpoints Jan 15, 2025
Copy link
Contributor

@jasonleenaylor jasonleenaylor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 2 of 2 files at r2, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @imnasnainaec)

@imnasnainaec imnasnainaec merged commit b454730 into master Jan 15, 2025
18 checks passed
@imnasnainaec imnasnainaec deleted the dotnet-endpoint branch January 15, 2025 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants