Skip to content

ci(hotrod): Print HotROD container logs in case of test failure#4845

Merged
yurishkuro merged 1 commit intojaegertracing:mainfrom
yurishkuro:print-hotrod-logs
Oct 15, 2023
Merged

ci(hotrod): Print HotROD container logs in case of test failure#4845
yurishkuro merged 1 commit intojaegertracing:mainfrom
yurishkuro:print-hotrod-logs

Conversation

@yurishkuro
Copy link
Copy Markdown
Member

@yurishkuro yurishkuro commented Oct 15, 2023

Which problem is this PR solving?

Description of the changes

  • Run container with --name example-hotrod
  • Add docker logs example-hotrod to CIT workflow in case of failure

@yurishkuro
ci(hotrod): Print HotROD container logs in case of test failure
baf0a1b 
Signed-off-by: Yuri Shkuro <github@ysh.us>
@yurishkuro yurishkuro marked this pull request as ready for review October 15, 2023 20:02
@yurishkuro yurishkuro requested a review from a team as a code owner October 15, 2023 20:02
@yurishkuro yurishkuro mentioned this pull request Oct 15, 2023
Merged
4 tasks
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 15, 2023

Codecov Report

All modified lines are covered by tests ✅

📢 Thoughts on this report? Let us know!.

@yurishkuro yurishkuro merged commit 065cc55 into jaegertracing:main Oct 15, 2023
@yurishkuro yurishkuro deleted the print-hotrod-logs branch October 15, 2023 20:15
@yurishkuro yurishkuro added the changelog:ci Change related to continuous integration / testing label Oct 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pavolloffay pavolloffay Awaiting requested review from pavolloffay pavolloffay is a code owner automatically assigned from jaegertracing/jaeger-maintainers

Assignees

No one assigned

Labels

changelog:ci Change related to continuous integration / testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@yurishkuro