You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@streampipes.apache.org by "stefanobaghino (via GitHub)" <gi...@apache.org> on 2025/12/12 14:14:45 UTC
[PR] Enable `retryOnStatusCodeFailure` when calling `resetStreamPipes`
stefanobaghino opened a new pull request, #4053:
URL: https://github.com/apache/streampipes/pull/4053
### Purpose
I was looking at Develocity and noticed there's [a test that seem to be quite prone to flake](https://develocity.apache.org/scans/tests?search.buildToolType=npm&search.rootProjectNames=apache-streampipes&search.startTimeMax=1765580399999&search.startTimeMin=1764889200000&search.tasks=cypress&search.timeZoneId=Europe%2FZurich&tests.container=cypress%2Ftests%2Fconnect%2FenhancedAdapterDeletion.smoke.spec.ts&tests.test=Test%20Enhanced%20Adapter%20Deletion%20%C2%BB%20Test%20Admin%20Should%20Be%20Able%20to%20Delete%20Adapter%20and%20Not%20Owned%20Associated%20Pipelines). I was indeed able to reproduce the issue locally ([the reset endpoint returns a 500](https://develocity.apache.org/s/hfqwooeg6e66c/tests/process/apache-streampipes@cypress/details/cypress%2Ftests%2Fconnect%2FenhancedAdapterDeletion.smoke.spec.ts/Test%20Enhanced%20Adapter%20Deletion%20%C2%BB%20Test%20Admin%20Should%20Be%20Able%20to%20Delete%20Adapter%20and%20Not%20Owned%20Associated%20Pipelines?focused-exception-line=0-
0&top-execution=1)) and noticed that a simple `retryOnStatusCodeFailure` seems to help reduce the flakiness. Hope this helps! 🙂
---
PR introduces (a) breaking change(s): no
PR introduces (a) deprecation(s): no
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Enable `retryOnStatusCodeFailure` when calling `resetStreamPipes`
Posted by "tenthe (via GitHub)" <gi...@apache.org>.
tenthe merged PR #4053:
URL: https://github.com/apache/streampipes/pull/4053
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Enable `retryOnStatusCodeFailure` when calling `resetStreamPipes`
Posted by "stefanobaghino (via GitHub)" <gi...@apache.org>.
stefanobaghino commented on PR #4053:
URL: https://github.com/apache/streampipes/pull/4053#issuecomment-3646674383
FYI I just spotted a few extra tests where the same problem seems to pop up:
* https://develocity.apache.org/s/hfqwooeg6e66c/tests/process/apache-streampipes@cypress/details/cypress%2Ftests%2Fconnect%2Fcompact%2FaddCompactAdapter.spec.ts/Add%20Compact%20Adapters%20%C2%BB%20Add%20file%20stream%20adapter%20via%20the%20compact%20API.%20Start%20Adapter%20with%20Pipeline?top-execution=1
* https://develocity.apache.org/s/rk4wq3ydivq24/tests/process/apache-streampipes@cypress/details/cypress%2Ftests%2FpipelineElement%2FAllPipelineElements.spec.ts/Test%20Processor%20booleanInverter1%20%C2%BB%20Perform%20Test?top-execution=1
* https://develocity.apache.org/s/rh4727aevzclo/tests/process/apache-streampipes@cypress/details/cypress%2Ftests%2FpipelineElement%2FPipelineElementDocumentation.spec.ts/Validate%20that%20the%20markdown%20documentation%20for%20pipeline%20elements%20works%20%C2%BB%20Perform%20Test?top-execution=1
* https://develocity.apache.org/s/t3xoi74wx566s/tests/process/apache-streampipes@cypress/details/cypress%2Ftests%2Fconnect%2FeditAdapterTransformationRulesAreKept.spec.ts/Test%20Adapter%20Transformation%20Rules%20are%20properly%20stored%20%C2%BB%20Test%20configuration%20of%20adapter%20fields%20?top-execution=1
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Enable `retryOnStatusCodeFailure` when calling `resetStreamPipes`
Posted by "stefanobaghino (via GitHub)" <gi...@apache.org>.
stefanobaghino commented on PR #4053:
URL: https://github.com/apache/streampipes/pull/4053#issuecomment-3653790098
Thanks for having a look! Anything else I can do to help get this merged? 🙂
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Enable `retryOnStatusCodeFailure` when calling `resetStreamPipes`
Posted by "tenthe (via GitHub)" <gi...@apache.org>.
tenthe commented on PR #4053:
URL: https://github.com/apache/streampipes/pull/4053#issuecomment-3654123765
Hi @stefanobaghino,
thanks a lot for reporting this issue and for providing the PR!
I hope this will reduce the flaky tests.
If you notice any other issues, please let us know. Your feedback is highly appreciated.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@streampipes.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org