Skip to content
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.

Sign up for GitHub

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

Jump to bottom

failure in partition_moving_test_rpunit #2615

Closed
twmb opened this issue Oct 11, 2021 · 1 comment · Fixed by #2630
Closed

failure in partition_moving_test_rpunit #2615

twmb opened this issue Oct 11, 2021 · 1 comment · Fixed by #2630
Assignees
@dotnwat @mmaslankaprv
Labels
area/raft ci-failure kind/bug Something isn't working

Comments

@twmb
Copy link
Contributor

twmb commented Oct 11, 2021

Potentially the seastar reactor, based on the backtraces at the bottom?
https://buildkite.com/vectorized/redpanda/builds/3207#a992bef0-cd9c-46b4-b00a-07e9270ad887
@twmb
Copy link
Contributor Author

twmb commented Oct 11, 2021 •
edited
Loading

Maybe something during shutdown:

(inlined by) void seastar::futurize<seastar::future >::satisfy_with_result_of<seastar::future::then_impl_nrvo<cluster::partition_manager::remove(model::ntp const&)::$_2, seastar::future >(cluster::partition_manager::remove(model::ntp const&)::$_2&&)::{lambda(seastar::internal::promise_base_with_type&&, cluster::partition_manager::remove(model::ntp const&)::$_2&, seastar::future_stateseastar::internal::monostate&&)#1}::operator()(seastar::internal::promise_base_with_type&&, cluster::partition_manager::remove(model::ntp const&)::$_2&, seastar::future_stateseastar::internal::monostate&&) const::{lambda()#1}>(seastar::internal::promise_base_with_type&&, cluster::partition_manager::remove(model::ntp const&)::$_2&&) at /vectorized/include/seastar/core/future.hh:2120

jcsp added a commit to jcsp/redpanda that referenced this issue Oct 12, 2021
@jcsp
cluster: disable partition_moving_test
99aec62 
This is failing too often with a segfault.

Related: redpanda-data#2615

Signed-off-by: John Spray <[email protected]>
@jcsp jcsp mentioned this issue Oct 12, 2021
Merged
@jcsp jcsp added area/raft kind/bug Something isn't working labels Oct 12, 2021
@mmaslankaprv mmaslankaprv self-assigned this Oct 12, 2021
mmaslankaprv added a commit to mmaslankaprv/redpanda that referenced this issue Oct 13, 2021
@mmaslankaprv
c/partition_manager: fixed stopping partitions during shutdown
ebe9d65 
Previously partitions weren't fully stopped when partition manager was
being shut down. This lead to race condition in which partition might be
shutdown twice.

Fixes: redpanda-data#2615

Signed-off-by: Michal Maslanka <[email protected]>
@mmaslankaprv mmaslankaprv mentioned this issue Oct 13, 2021
Merged
mmaslankaprv added a commit to mmaslankaprv/redpanda that referenced this issue Oct 13, 2021
@mmaslankaprv
c/partition_manager: fixed stopping partitions during shutdown
64f7bf3 
Previously partitions weren't fully stopped when partition manager was
being shut down. This lead to race condition in which partition might be
shutdown twice.

Fixes: redpanda-data#2615

Signed-off-by: Michal Maslanka <[email protected]>
mmaslankaprv added a commit to mmaslankaprv/redpanda that referenced this issue Oct 13, 2021
@mmaslankaprv
c/partition_manager: fixed stopping partitions during shutdown
95db017 
Previously partitions weren't fully stopped when partition manager was
being shut down. This lead to race condition in which partition might be
shutdown twice.

Fixes: redpanda-data#2615

Signed-off-by: Michal Maslanka <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dotnwat dotnwat

@mmaslankaprv mmaslankaprv

Labels
area/raft ci-failure kind/bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixed partition_movement_rpunit tests failing in CI mmaslankaprv/redpanda
4 participants
@dotnwat @jcsp @twmb @mmaslankaprv