lumberjack icon indicating copy to clipboard operation
lumberjack copied to clipboard

ci: use build artifacts to speed up SonarCloud job

Open LayZeeDK opened this issue 2 years ago • 5 comments

PR Checklist

Please check if your PR fulfills the following requirements:

  • [x] The commit message follows our guidelines: CONTRIBUTING.md#commit
  • [ ] Tests for the changes have been added (for bug fixes / features)
  • [ ] Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[x] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

The SonarCloud job takes a long time because we have to set up package dependencies to run Nx commands and use the Nx Cloud cache.

Issue Number: N/A

What is the new behavior?

The DTE orchestrator uploads test coverage and lint reports as build artifacts that are downloaded by the SonarCloud job.

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

The configure Sonar report paths script is converted to a local GitHub action. This is implemented as an Nx Node.js application project which is built in the DTE jobs and used in the SonarCloud job to prevent having to install Nx or other dependencies in order to minimize the time spent in the job.

  • Update SonarCloud GitHub action to v1.9
  • Update Nx DTE to v0.11.x
  • Update Nx Cloud to 15.2.3
  • Remove duplicate nx-cloud start-ci-run command as this causes DTE agent synchronization issues

LayZeeDK avatar Nov 22 '22 18:11 LayZeeDK

Blocked by #149.

LayZeeDK avatar Feb 15 '23 19:02 LayZeeDK

@LayZeeDK should we close this PR? It has been stale for a while.

NachoVazquez avatar Jan 13 '24 12:01 NachoVazquez