Add dependency-graph-file as step output

Fixes #804
This commit is contained in:
daz 2023-07-24 08:34:10 -06:00
parent 632e888003
commit 9e58f8b1de
No known key found for this signature in database
3 changed files with 28 additions and 18 deletions

View file

@ -78,21 +78,19 @@ jobs:
uses: ./ uses: ./
with: with:
dependency-graph: generate dependency-graph: generate
- name: Run assemble - id: gradle-assemble
run: ./gradlew assemble run: ./gradlew assemble
working-directory: .github/workflow-samples/groovy-dsl working-directory: .github/workflow-samples/groovy-dsl
env: - id: gradle-build
GITHUB_JOB_CORRELATOR: job-correlator
- name: Run build
run: ./gradlew build run: ./gradlew build
working-directory: .github/workflow-samples/groovy-dsl working-directory: .github/workflow-samples/groovy-dsl
env:
GITHUB_JOB_CORRELATOR: job-correlator
- name: Check generated dependency graphs - name: Check generated dependency graphs
run: | run: |
echo "gradle-assemble report file: ${{ steps.gradle-assemble.outputs.dependency-graph-file }}"
echo "gradle-build report file: ${{ steps.gradle-build.outputs.dependency-graph-file }}"
ls -l dependency-graph-reports ls -l dependency-graph-reports
if ([ ! -e dependency-graph-reports/job-correlator.json ] || [ ! -e dependency-graph-reports/job-correlator-1.json ]) if ([ ! -e ${{ steps.gradle-assemble.outputs.dependency-graph-file }} ] || [ ! -e ${{ steps.gradle-build.outputs.dependency-graph-file }} ])
then then
echo "Did not find expected dependency graph files" echo "Did not find expected dependency graph files"
exit 1 exit 1
fi fi

View file

@ -15,14 +15,20 @@ if (GradleVersion.current().baseVersion < GradleVersion.version("5.0")) {
// This is only required for top-level builds // This is only required for top-level builds
def isTopLevelBuild = gradle.getParent() == null def isTopLevelBuild = gradle.getParent() == null
if (isTopLevelBuild) { if (isTopLevelBuild) {
def jobCorrelator = ensureUniqueJobCorrelator(System.env.GITHUB_JOB_CORRELATOR) def reportFile = getUniqueReportFile(System.env.GITHUB_JOB_CORRELATOR)
if (jobCorrelator == null) { if (reportFile == null) {
println "::warning::No dependency snapshot generated for step: report file for '${jobCorrelator}' created in earlier step. Each build invocation requires a unique job correlator: specify GITHUB_JOB_CORRELATOR var for this step." println "::warning::No dependency snapshot generated for step. Could not determine unique job correlator - specify GITHUB_JOB_CORRELATOR var for this step."
return return
} }
println "Generating dependency graph for '${jobCorrelator}'" def githubOutput = System.getenv("GITHUB_OUTPUT")
if (githubOutput) {
new File(githubOutput) << "dependency-graph-file=${reportFile.absolutePath}\n"
}
println "Generating dependency graph into '${reportFile}'"
} }
apply from: 'github-dependency-graph-gradle-plugin-apply.groovy' apply from: 'github-dependency-graph-gradle-plugin-apply.groovy'
@ -33,10 +39,10 @@ apply from: 'github-dependency-graph-gradle-plugin-apply.groovy'
* - If so, tries to find a unique value that does not yet have a corresponding report file. * - If so, tries to find a unique value that does not yet have a corresponding report file.
* - When found, this value is set as a System property override. * - When found, this value is set as a System property override.
*/ */
String ensureUniqueJobCorrelator(String jobCorrelator) { File getUniqueReportFile(String jobCorrelator) {
def reportDir = System.env.DEPENDENCY_GRAPH_REPORT_DIR def reportDir = System.env.DEPENDENCY_GRAPH_REPORT_DIR
def reportFile = new File(reportDir, jobCorrelator + ".json") def reportFile = new File(reportDir, jobCorrelator + ".json")
if (!reportFile.exists()) return jobCorrelator if (!reportFile.exists()) return reportFile
// Try at most 100 suffixes // Try at most 100 suffixes
for (int i = 1; i < 100; i++) { for (int i = 1; i < 100; i++) {
@ -44,7 +50,7 @@ String ensureUniqueJobCorrelator(String jobCorrelator) {
def candidateFile = new File(reportDir, candidateCorrelator + ".json") def candidateFile = new File(reportDir, candidateCorrelator + ".json")
if (!candidateFile.exists()) { if (!candidateFile.exists()) {
System.properties['GITHUB_JOB_CORRELATOR'] = candidateCorrelator System.properties['GITHUB_JOB_CORRELATOR'] = candidateCorrelator
return candidateCorrelator return candidateFile
} }
} }

View file

@ -29,9 +29,10 @@ class TestDependencyGraph extends BaseInitScriptTest {
then: then:
assert reportFile.exists() assert reportFile.exists()
assert gitHubOutputFile.text == "dependency-graph-file=${reportFile.absolutePath}\n"
where: where:
testGradleVersion << DEPENDENCY_GRAPH_VERSIONS testGradleVersion << GRADLE_8_X
} }
// Dependency-graph plugin doesn't support config-cache for 8.0 of Gradle // Dependency-graph plugin doesn't support config-cache for 8.0 of Gradle
@ -114,7 +115,8 @@ class TestDependencyGraph extends BaseInitScriptTest {
GITHUB_REF: "main", GITHUB_REF: "main",
GITHUB_SHA: "123456", GITHUB_SHA: "123456",
GITHUB_WORKSPACE: testProjectDir.absolutePath, GITHUB_WORKSPACE: testProjectDir.absolutePath,
DEPENDENCY_GRAPH_REPORT_DIR: reportsDir.absolutePath DEPENDENCY_GRAPH_REPORT_DIR: reportsDir.absolutePath,
GITHUB_OUTPUT: gitHubOutputFile.absolutePath
] ]
} }
@ -125,4 +127,8 @@ class TestDependencyGraph extends BaseInitScriptTest {
def getReportFile() { def getReportFile() {
return new File(reportsDir, "CORRELATOR.json") return new File(reportsDir, "CORRELATOR.json")
} }
def getGitHubOutputFile() {
return new File(testProjectDir, "GITHUB_OUTPUT")
}
} }