-
Notifications
You must be signed in to change notification settings - Fork 404
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
Issue 5008 sporadic failures due used tcp ports #7008
base: master
Are you sure you want to change the base?
Issue 5008 sporadic failures due used tcp ports #7008
Conversation
…in use Signed-off-by: Kuldeep Kumar <kuldeep.kumar3@dell.com>
Signed-off-by: Kuldeep Kumar <kuldeep.kumar3@dell.com>
Signed-off-by: Kuldeep Kumar <kuldeep.kumar3@dell.com>
Signed-off-by: Kuldeep Kumar <kuldeep.kumar3@dell.com>
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.
LGTM
...l/src/main/java/io/pravega/segmentstore/storage/impl/bookkeeper/BookKeeperServiceRunner.java
Outdated
Show resolved
Hide resolved
...l/src/main/java/io/pravega/segmentstore/storage/impl/bookkeeper/BookKeeperServiceRunner.java
Outdated
Show resolved
Hide resolved
...l/src/main/java/io/pravega/segmentstore/storage/impl/bookkeeper/BookKeeperServiceRunner.java
Outdated
Show resolved
Hide resolved
Could you test this by checking if you can make the test start on a already occupied port and then see if the retry logic works? |
...l/src/main/java/io/pravega/segmentstore/storage/impl/bookkeeper/BookKeeperServiceRunner.java
Show resolved
Hide resolved
...l/src/main/java/io/pravega/segmentstore/storage/impl/bookkeeper/BookKeeperServiceRunner.java
Outdated
Show resolved
Hide resolved
...l/src/main/java/io/pravega/segmentstore/storage/impl/bookkeeper/BookKeeperServiceRunner.java
Outdated
Show resolved
Hide resolved
...l/src/main/java/io/pravega/segmentstore/storage/impl/bookkeeper/BookKeeperServiceRunner.java
Outdated
Show resolved
Hide resolved
...l/src/main/java/io/pravega/segmentstore/storage/impl/bookkeeper/BookKeeperServiceRunner.java
Outdated
Show resolved
Hide resolved
...l/src/main/java/io/pravega/segmentstore/storage/impl/bookkeeper/BookKeeperServiceRunner.java
Outdated
Show resolved
Hide resolved
...l/src/main/java/io/pravega/segmentstore/storage/impl/bookkeeper/BookKeeperServiceRunner.java
Outdated
Show resolved
Hide resolved
...l/src/main/java/io/pravega/segmentstore/storage/impl/bookkeeper/BookKeeperServiceRunner.java
Show resolved
Hide resolved
Signed-off-by: Kuldeep Kumar <kuldeep.kumar3@dell.com>
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #7008 +/- ##
============================================
- Coverage 86.33% 86.32% -0.01%
+ Complexity 16048 16002 -46
============================================
Files 1031 1030 -1
Lines 59815 59543 -272
Branches 6071 6024 -47
============================================
- Hits 51639 51402 -237
+ Misses 4998 4991 -7
+ Partials 3178 3150 -28
... and 29 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
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 understand the approach, but we really need that verify this works.
bookieServerAndResources = runBookie(bkPort); | ||
retry = false; | ||
} catch (BindException e) { | ||
if ( ++count >= retries ) { |
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.
Remove extra head and tail spaces.
...l/src/main/java/io/pravega/segmentstore/storage/impl/bookkeeper/BookKeeperServiceRunner.java
Outdated
Show resolved
Hide resolved
...l/src/main/java/io/pravega/segmentstore/storage/impl/bookkeeper/BookKeeperServiceRunner.java
Show resolved
Hide resolved
...l/src/main/java/io/pravega/segmentstore/storage/impl/bookkeeper/BookKeeperServiceRunner.java
Show resolved
Hide resolved
…t/common/TestUtils.java -> common/src/main/java/io/pravega/common/util/CommonUtils.java Signed-off-by: Kuldeep Kumar <kuldeep.kumar3@dell.com>
…ps://github.com/kuldeepk3/pravega into issue-5008-sporadic-failures-due-used-tcp-ports
Signed-off-by: Kuldeep Kumar <kuldeep.kumar3@dell.com>
Signed-off-by: Kuldeep Kumar <kuldeep.kumar3@dell.com>
Signed-off-by: Kuldeep Kumar <kuldeep.kumar3@dell.com>
Signed-off-by: Kuldeep Kumar <kuldeep.kumar3@dell.com>
Change log description
This pr addresses
Sporadic failures due used tcp ports
in case there is failure while starting bookeeper and goes for a retry.Purpose of the change
Fixes #5008
What the code does
Adding retry in case bookkeeper fails to start with
address already in use
. This change will retry the creation of bookkeeper in case it fails to start withBindException
.How to verify it
Execute the unit test and there should be no failure with exception:
java.net.BindException: Address already in use