Skip to content

Testing PR build scan#17130

Closed
mumrah wants to merge 1 commit into
apache:trunkfrom
mumrah:gh-test-pr-build-scan
Closed

Testing PR build scan#17130
mumrah wants to merge 1 commit into
apache:trunkfrom
mumrah:gh-test-pr-build-scan

Conversation

@mumrah

@mumrah mumrah commented Sep 8, 2024

Copy link
Copy Markdown
Member

Just open for testing

@mumrah mumrah added the do-not-merge PRs that are only open temporarily and should not be merged label Sep 8, 2024
@mumrah

mumrah commented Sep 9, 2024

Copy link
Copy Markdown
Member Author

@chia7712 looks like it worked!

Here is one of the reports for this PR (coming from my fork): https://ge.apache.org/s/icjmgcm6xszvy

I see one issue I'll file a fix for shortly.

@chia7712

chia7712 commented Sep 9, 2024

Copy link
Copy Markdown
Member

I see one issue I'll file a fix for shortly.

Did you mean this issue: https://github.com/apache/kafka/actions/runs/10776195209/job/29882369035 ?

Run actions/download-artifact@v4
Downloading single artifact
Error: Unable to download artifact(s): Artifact not found for name: build-scan-test-11
        Please ensure that your artifact is not expired and the artifact was uploaded using a compatible version of toolkit/upload-artifact.
        For more information, visit the GitHub Artifacts FAQ: https://github.com/actions/toolkit/blob/main/packages/artifact/docs/faq.md

@chia7712

chia7712 commented Sep 9, 2024

Copy link
Copy Markdown
Member

Did you mean this issue: https://github.com/apache/kafka/actions/runs/10776195209/job/29882369035 ?

It seems the CI Complete always tries to download the test result even though the PR does not run the test. I guess that can be fixed by merging #17105 which enable test by default.

@mumrah

mumrah commented Sep 9, 2024

Copy link
Copy Markdown
Member Author

@chia7712 #17105 and #17140 will fix the failing "CI Complete" actions

@mumrah mumrah closed this Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge PRs that are only open temporarily and should not be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants