Add Aggregate Report Pipeline#9964
Conversation
chidozieononiwu
requested review from
danieljurek,
mitchdenny and
weshaggard
as code owners
chidozieononiwu
force-pushed
the
AgregateReportPipeline
branch
from
7ab9f34 to
9cd98ef
Compare
There was a problem hiding this comment.
Nit: IIRC, this shows up as alternative text, so maybe spell out "Dependency Graph".
There was a problem hiding this comment.
Not a big issue, but if there's nothing Windows-specific, ubuntu-18.04 is much faster.
There was a problem hiding this comment.
Ok, AzureFileCopy is windows specific.
There was a problem hiding this comment.
succeeded() is the default condition, so these instances here and below aren't necessarily needed.
There was a problem hiding this comment.
Oh, I see thanks
chidozieononiwu
force-pushed
the
AgregateReportPipeline
branch
from
9cd98ef to
2d5b1c1
Compare
| displayName: 'Download Pipeline Artifact' | ||
| inputs: | ||
| buildType: specific | ||
| project: $(TargetPipelineProjectId) |
There was a problem hiding this comment.
Which pipeline is this pulling? Do you know if it only pulls latest successful build?
There was a problem hiding this comment.
It pulls the latest successful master branch build of the net-core and downloads the artifacts called 'packages'.
I didn't use allowPartiallySucceededBuilds or allowFailedBuilds so there is not case where it should download an unsuccessful run. (Assuming tool works as advertised)
| @@ -0,0 +1,48 @@ | |||
| trigger: none | |||
There was a problem hiding this comment.
Both java and python have file and pipeline called aggregate-reports can you please update to match?
There was a problem hiding this comment.
I will update the name.
3 participants
Move dependency report into aggregate report pipeline.
Run dependency report using downloaded artifacts from latest successful scheduled run of the net - core pipeline.