-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Flaky Test - tailsamplingprocessor/TestConcurrentTraceArrival #10205
Comments
Hi @jpkrohling is this issue still open? |
I believe it is! Do you want to take this one? |
Yes, can assign to me @jpkrohling |
@dgoscn, assigned to you |
Pinging code owners: @jpkrohling. See Adding Labels via Comments if you do not have permissions to add labels yourself. |
This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
@dgoscn, are you still available to work on this one? |
@jpkrohling I will have to let for someone else. If no one else assign, I want to go back to work on this as soon as possible |
Hi @djaglowski How are you? Happy new year. Do you know if this scenario still happening? The log for the events are expired. |
@dgoscn I was looking through the processor and was wondering if have you been able to replicate these flaky test locally? I ran the tests for a while on my machine and did not see any failures. Running it on an Intel MBP |
Hey @bryan-aguilar. Thanks for the message, this sounds really good. I didn't made the tests again. But, sure I will do. Until the moment, I believe if you do not face the error, we can advance based on some advice from @jpkrohling . |
Hello, I am currently review this test. Upon running 20 times the GitHub Action on my personal branch, there were no test failures. Here is the file Tailsamplingprocessor-FlakyTest-#10205.xlsx If everything seems fine, I will proceed with pulling request. |
Please go ahead, thanks! |
**Description:** <Describe what has changed.> <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> Run 20 times the GitHub Action on my personal branch, there were no test failures. See the files: [Tailsamplingprocessor-FlakyTest-.10205.xlsx](https://github.com/open-telemetry/opentelemetry-collector-contrib/files/11911095/Tailsamplingprocessor-FlakyTest-.10205.xlsx) **Link to tracking Issue:** <Issue number if applicable> [10205](#10205) **Testing:** tailsamplingprocessor/processor_test.go --------- Co-authored-by: Alex Boten <[email protected]> Co-authored-by: zeno-splunk <[email protected]>
Thanks! |
Observed here: https://github.com/open-telemetry/opentelemetry-collector-contrib/runs/6555994409?check_suite_focus=true
The text was updated successfully, but these errors were encountered: