ci: fix relative paths for verify e2e test (#3249)

* Use absolute paths to avoid issues with bazel run changing working directory * Fix error message in attestationconfig cli --------- Signed-off-by: Daniel Weiße <[email protected]>
edgelesssys · Jul 12, 2024 · e08ed38 · e08ed38
1 parent 65ed286
commit e08ed38
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/.github/actions/e2e_verify/action.yml b/.github/actions/e2e_verify/action.yml
@@ -96,7 +96,14 @@ runs:
       run: |
         reports=attestation-report-*.json
 
-        report=$(bazel run //internal/api/attestationconfigapi/cli -- compare ${{ inputs.attestationVariant }} ${reports})
+        # bazel run changes the working directory
+        # convert the relative paths to absolute paths to avoid issues
+        absolute_reports=""
+        for report in ${reports}; do
+          absolute_reports="${absolute_reports} $(realpath "${report}")"
+        done
+
+        report=$(bazel run //internal/api/attestationconfigapi/cli -- compare ${{ inputs.attestationVariant }} ${absolute_reports})
 
         path=$(realpath "${report}")
         cat "${path}"

diff --git a/internal/api/attestationconfigapi/cli/compare.go b/internal/api/attestationconfigapi/cli/compare.go
@@ -66,13 +66,13 @@ func compareVersions(attestationVariant variant.Variant, files []string, fs file
 	lowestVersion := files[0]
 	lowestReport, err := readReport(files[0], fs)
 	if err != nil {
-		return "", fmt.Errorf("reading tdx report: %w", err)
+		return "", fmt.Errorf("reading report: %w", err)
 	}
 
 	for _, file := range files[1:] {
 		report, err := readReport(file, fs)
 		if err != nil {
-			return "", fmt.Errorf("reading tdx report: %w", err)
+			return "", fmt.Errorf("reading report: %w", err)
 		}
 
 		if client.IsInputNewerThanOtherVersion(attestationVariant, lowestReport, report) {