Skip to content

Commit

Permalink
Move initial coverage reporting to later in startup to capture read-o…
Browse files Browse the repository at this point in the history
…nly inputs (microsoft#3605)

* add

* Move intial coverage report farther into task start.

* Keeping both calls.
  • Loading branch information
nharper285 committed Oct 26, 2023
1 parent 909b130 commit 5cf619e
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions src/agent/onefuzz-task/src/tasks/coverage/generic.rs
Original file line number Diff line number Diff line change
Expand Up @@ -146,11 +146,11 @@ impl CoverageTask {
bail!("input is not specified on the command line or arguments for the target");
}

context.heartbeat.alive();

info!("report initial coverage");
context.report_coverage_stats().await;

context.heartbeat.alive();

for dir in &self.config.readonly_inputs {
debug!("recording coverage for {}", dir.local_path.display());

Expand All @@ -174,6 +174,9 @@ impl CoverageTask {
context.save_and_sync_coverage().await?;
}

info!("report coverage");
context.report_coverage_stats().await;

context.heartbeat.alive();

if let Some(queue) = &self.config.input_queue {
Expand Down

0 comments on commit 5cf619e

Please sign in to comment.