Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Publish go tip test report #5082

Merged
merged 10 commits into from
Jan 6, 2024

Conversation

albertteoh
Copy link
Contributor

Which problem is this PR solving?

Description of the changes

  • Publish detailed test report on go tip workflow run completion.

How was this change tested?

The detailed report will be published under the Go Tip unit test run summary:

Screenshot 2024-01-06 at 4 42 57 pm Screenshot 2024-01-06 at 4 43 12 pm

Checklist

Albert Teoh added 9 commits December 26, 2023 19:36
Signed-off-by: Albert Teoh <albert@packsmith.io>
Signed-off-by: Albert Teoh <albert@packsmith.io>
Signed-off-by: Albert Teoh <albert@packsmith.io>
Signed-off-by: Albert Teoh <albert@packsmith.io>
Signed-off-by: Albert Teoh <albert@packsmith.io>
Signed-off-by: Albert Teoh <albert@packsmith.io>
@albertteoh albertteoh requested a review from a team as a code owner January 6, 2024 05:45
@albertteoh albertteoh added the changelog:ci Change related to continuous integration / testing label Jan 6, 2024
@yurishkuro yurishkuro enabled auto-merge (squash) January 6, 2024 05:47
types:
- completed

jobs:
unit-test-results:
name: Unit Test Results
name: ${{ github.event.workflow_run.name }}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ensures the title of the summary report is "Unit Tests on Go Tip summary" instead of just "Unit Tests summary"

@albertteoh albertteoh mentioned this pull request Jan 6, 2024
2 tasks
Copy link

github-actions bot commented Jan 6, 2024

Test Results

2 083 tests  ±0   2 073 ✅ ±0   1m 8s ⏱️ ±0s
  220 suites ±0      10 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit e5e6880. ± Comparison against base commit 370e756.

♻️ This comment has been updated with latest results.

@yurishkuro
Copy link
Member

Flaky test #5083

@yurishkuro
Copy link
Member

Weird, how is FOSSA mixed up in the same workflow as tests... And shows as failed even through it didn't.

image

@yurishkuro yurishkuro merged commit 01b1015 into jaegertracing:main Jan 6, 2024
37 checks passed
@albertteoh albertteoh deleted the publish-go-tip-test-report branch January 6, 2024 06:21
@albertteoh
Copy link
Contributor Author

Weird, how is FOSSA mixed up in the same workflow as tests... And shows as failed even through it didn't.

Yeah, that is a little odd; I noticed it's also mixed in with the build all-in-one workflow too. Not sure why 🤷🏼

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:ci Change related to continuous integration / testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants