Skip to content

CI: fix the listing of the test results directory - #2112

Open
tuhaihe wants to merge 1 commit into
apache:mainfrom
tuhaihe:fix-ci-results-listing
Open

tuhaihe wants to merge 1 commit into
apache:mainfrom
tuhaihe:fix-ci-results-listing

Conversation

@tuhaihe

@tuhaihe tuhaihe commented Oct 10, 2026

Copy link
Copy Markdown
Member

The test jobs of the build workflows list the files of the results directory with:

find "$results_dir" -type f -ls >> "$log_file" 2>&1 | tee -a ...

$log_file is not set in that step, so the redirection fails and the job log gets an error instead of the list:

line 102: : No such file or directory

The listing is lost, and the error appears in every test job that has a results directory, on all Ubuntu versions.

Remove the redirection, so that the list goes to tee, which shows it and appends it to the results log of the configuration, as the other echo lines of the step already do.

Assisted-by: Claude Code
Backpatch-through: REL_2_STABLE

Fixes #ISSUE_Number

What does this PR do?

Type of Change

  • Bug fix (non-breaking change)
  • New feature (non-breaking change)
  • Breaking change (fix or feature with breaking changes)
  • Documentation update

Breaking Changes

Test Plan

  • Unit tests added/updated
  • Integration tests added/updated
  • Passed make installcheck
  • Passed make -C src/test installcheck-cbdb-parallel

Impact

Performance:

User-facing changes:

Dependencies:

Checklist

Additional Context

CI Skip Instructions


The test jobs of the build workflows list the files of the results
directory with:

  find "$results_dir" -type f -ls >> "$log_file" 2>&1 | tee -a ...

$log_file is not set in that step, so the redirection fails and the
job log gets an error instead of the list:

  line 102: : No such file or directory

The listing is lost, and the error appears in every test job that has a
results directory, on all Ubuntu versions.

Remove the redirection, so that the list goes to tee, which shows it
and appends it to the results log of the configuration, as the other
echo lines of the step already do.

Assisted-by: Claude Code
Backpatch-through: REL_2_STABLE
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant