AgentBaker icon indicating copy to clipboard operation
AgentBaker copied to clipboard

test: modify benchmarking workflow and publish build performance artifact

Open zachary-bailey opened this issue 1 year ago • 1 comments

What type of PR is this?

/test

What this PR does / why we need it:

In addition to publishing a pipeline artifact that contains build performance data, this PR changes the workflow for benchmarking the build process. Changes include removing the start_watch function, adding the installJq function, and outputting benchmarking data in JSON instead of formatted strings. At the end of each script, a JSON object will be output in the terminal with performance data for each section.

The installJq function was necessary because jq is not installed until the installDeps function is called in install-dependencies.sh, but it is needed when the build process starts in pre-install-dependencies.sh.

The pipeline artifact that will be published is a JSON file that will contain benchmarking data for the build process. In later changes, github actions will be used to evaluate this artifact and check for build time regression.

The formatting of the artifact can be checked by going to the PR gate build and checking the artifacts list.

Which issue(s) this PR fixes:

This will prevent vhd build times from incrementally increasing due to code changes.

Note: When reviewing "Files changed", please collapse /pkg/agent/testdata to see all relevant file changes

Requirements:

zachary-bailey avatar Apr 26 '24 17:04 zachary-bailey

Pull Request Test Coverage Report for Build 8941515521

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 76.258%

Totals Coverage Status
Change from base Build 8931627929: 0.0%
Covered Lines: 3000
Relevant Lines: 3934

💛 - Coveralls

coveralls avatar Apr 26 '24 17:04 coveralls

closing PR, new PR will be created

zachary-bailey avatar Jun 14 '24 18:06 zachary-bailey