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
Added spring cloud task support #5
Conversation
name: task | ||
|
||
# For tests | ||
logging.level.org.springframework.cloud.sleuth: DEBUG |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
probably want to add logging.level.org.springframework.cloud.task: DEBUG
|
||
@Bean | ||
ApplicationRunner myApplicationRunner(Tracer tracer) { | ||
return args -> log.info("<ACCEPTANCE_TEST> <TRACE:{}> Hello from producer", tracer.currentSpan().context().traceId()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably want to rename producer to myApplicationRunner
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's fixed for producer
cause I require this text to be present in the acceptance_tests
module 😬
|
||
@Bean | ||
CommandLineRunner myCommandLineRunner(Tracer tracer) { | ||
return args -> log.info("<ACCEPTANCE_TEST> <TRACE:{}> Hello from consumer", tracer.currentSpan().context().traceId()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably want to rename consumer to myCommandLineRunner
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's fixed for consumer
cause I require this text to be present in the acceptance_tests
module 😬
@@ -0,0 +1,5 @@ | |||
= Spring Cloud Task | |||
|
|||
We're wrapping `TaskExecutionListener`s and `CommandLineRunner` and `ApplicationRunner`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a TaskExecutionListener to the sample to see if I could see the output in Zipkin. I didn't see it appear. But I probably messed something up. Here are my additions: cppwfs@465a2ea
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll add the info to the README
done via 1396cf7 |
related issue spring-cloud/spring-cloud-sleuth#1903