-
Notifications
You must be signed in to change notification settings - Fork 2k
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
[BUG] OpenSearch 2.19.0 with installed repository-s3 plugin cannot be stopped #17363
Comments
faced exactly the same just after the upgrade (running on AlmaLinux9 and installing using the RPM) |
Can confirm on RHEL8 with tarball installation and added s3 plugin |
Looks like the same stacktrace from netty/netty#14743 @reta will the netty upgrade to 4.1.118.Final resolve this issue? |
Thanks @cwperks
Yes
Yes @spapadop @HenryTheSir @olfuerniss could you folks please confirm if the issue is not happening when |
@reta Yes |
yeap I double-confirm, everything seems happy without |
The 2.19.0 didn't get Netty 4.1.118 (sadly), listing possible options here:
@andrross any other options I am missing? thank you |
Yeah, it looks like a 2.19.1 patch release is probably the way to go here. FYI @getsaurabh02 It's also worth thinking about how we missed this in testing. |
Thank you @reta and @andrross for sharing the details. I also agree that patching 2.19 with fix and releasing 2.19.1 is the best option currently. Option 2 of building the Now, since the issue occurs while stoping a running OpenSearch instance, I will like to discuss the urgency of the patch release. Based on the current release schedule, the Also, what are the functional and usability issues which OS users might run into due to this bug. Adding more folks into the discussion to arrive on the patch release plan - @cwperks @msfroh @Pallavi-AWS @dblock On the miss for tests, @andrross @reta should we open an issue and follow up to see how we can add more Integ and Smoke Test for build and distrubtion |
tagging correct @peterzhuamazon :) |
Since distribution pull the plugin from s3 it is easier to make a quick fix there. Thanks. |
I think we could not update the published binaries without the version bump, right?
We definitely need to figure out how to test that. |
Yes! Maven does not allow overwriting the same version binaries. Recommend to do a patch instead to avoid having different artifacts on different platforms. |
I'm not sure if its possible with the current tests in the repository-s3 module, but ISM has interesting test cleanup logic that checks for running threads and I have seen it catch issues like this in the past. |
AFAIK, all OpenSearch based tests check for thread leaks, but for this issue, none of the tests have caught it sadly (affected plugins included transport-netty4, transport-reactor-netty4 and s3-repository) |
Facing the same problem. Since we use scripts to start/stop OS, this is a blocker for us to update. |
2.19.1 Release Issue can be tracked here |
Same problem
|
Adding @krisfreedain @nateynateynate @kolchfa-aws that 2.19.1 is coming next week. |
Tested the first RC of 2.19.1 to confirm if the issue is resolved. LGTM! . Find the RC details here for testing.
|
Hi @olfuerniss @spapadop @HenryTheSir @moesterheld @shpatserman @reta could you please help confirm the above Release Candidates posted by @Divyaasm solves the issues? We will go ahead with a official release on 2025/02/27. Thanks! |
Yes, the issue is resolved, thanks @peterzhuamazon |
@peterzhuamazon Sorry, can't test it as it is. I do get an
|
That is expected since it is a RC version not official release: opensearch-project/opensearch-build#5152 (comment) Could you try to manually download the zip and install as well? Thanks. |
LGTM :-)
Check installed plugins
AND ... now it can be stopped with Control+C. Thanks!!! |
Thanks @olfuerniss then we are good to release on Thursday! |
Closing this one, thanks folks! |
Why is this issue not mentioned in the release notes? |
The issue is caused by Netty and has zero changes on OpenSearch side besides dependency update - which is in release notes. |
I understand this point of view. But the effect on the user is different. The users are affected by this bug and they have no reason why this dependency update should help them. The release notes are not (in my opinion) a change log for other developers, they are an important information for the users. When I ran into that bug the release notes should give me a hint that this update would help. But currently they need to try or to find this GH issue. |
Describe the bug
When the repository-s3 plugin is installed, it is no longer possible to stop a running OpenSearch instance. Happens since 2.19.0.
Related component
Plugins
To Reproduce
=> It does not stop, shows the following exception and hangs (must be killed)
Expected behavior
OpenSearch should not hang on stop when the repository-s3 plugin is installed.
Additional Details
Plugins
Host/Environment (please complete the following information):
The text was updated successfully, but these errors were encountered: