Skip to content

Single instance testing isn't working #92

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

Closed
Totktonada opened this issue Oct 28, 2021 · 0 comments · Fixed by #93
Closed

Single instance testing isn't working #92

Totktonada opened this issue Oct 28, 2021 · 0 comments · Fixed by #93
Assignees
Labels
bug Something isn't working

Comments

@Totktonada
Copy link
Member

What I see on the 'jepsen-single-instance' job in tarantool (postprocessed logs):

2021-10-27T09:05:54.9580285Z 
:workload :bank-lua,
clojure.lang.ExceptionInfo: throw+: {:type :empty-grudge, :grudge {"89.208.228.127" #{}}, :node "89.208.228.127"}
clojure.lang.ExceptionInfo: throw+: {:type :empty-grudge, :grudge {"89.208.228.127" #{}}, :node "89.208.228.127"}
Everything looks good! ヽ(‘ー`)ノ 
2021-10-27T09:18:01.2961242Z 

2021-10-27T09:18:01.4252200Z 
:workload :bank-multitable-lua,
clojure.lang.ExceptionInfo: throw+: {:type :empty-grudge, :grudge {"89.208.228.127" #{}}, :node "89.208.228.127"}
clojure.lang.ExceptionInfo: throw+: {:type :empty-grudge, :grudge {"89.208.228.127" #{}}, :node "89.208.228.127"} 
Everything looks good! ヽ(‘ー`)ノ 
2021-10-27T09:20:11.8104307Z 

2021-10-27T09:20:11.9301155Z 
:workload :set,
java.sql.SQLException: Failed to execute SQL: INSERT INTO sets (value) VALUES (?), params: [0]
Caused by: org.tarantool.TarantoolException: The synchronous transaction queue doesn't belong to any instance
<..skipped repeated messages..>
clojure.lang.ExceptionInfo: throw+: {:type :empty-grudge, :grudge {"89.208.228.127" #{}}, :node "89.208.228.127"}
<..skipped repeated messages..>
Analysis invalid! (ノಥ益ಥ)ノ ┻━┻
2021-10-27T09:22:47.8019469Z 

2021-10-27T09:22:48.7684703Z 
:workload :register,
java.sql.SQLException: Failed to execute SQL: SELECT _UPSERT(0, 2, 'REGISTER'), params: []
Caused by: org.tarantool.TarantoolException: The synchronous transaction queue doesn't belong to any instance
<..skipped repeated messages..> 
Analysis invalid! (ノಥ益ಥ)ノ ┻━┻ 
2021-10-27T09:26:21.5315705Z

Full logs: logs.txt.

My findings:

  • The grudge exception fail a nemesis process and is considered as a warning. Not insterested for us here.
  • 'bank-lua' and 'bank-multitable-lua' work, 'set' and 'register' fails.
  • The failed tests uses synchronous spaces.
  • After some tarantool's commit, we should call box.ctl.promote() on a new leader, when RAFT is not used for leader election. It seems, even when we have just one instance. See Implement memtx transaction isolation levels for Vinyl tarantool#5522 (comment) (from this comment and below).

It seems, we should add box.ctl.promote() after box.cfg() for single instance configuration.

@Totktonada Totktonada added the bug Something isn't working label Oct 28, 2021
Totktonada added a commit that referenced this issue Oct 28, 2021
Otherwise 'set' and 'register' tests, which uses synchronous spaces,
fail with the 'The synchronous transaction queue doesn't belong to any
instance' error.

Fixes #92
Totktonada added a commit that referenced this issue Oct 29, 2021
Otherwise 'set' and 'register' tests, which uses synchronous spaces,
fail with the 'The synchronous transaction queue doesn't belong to any
instance' error.

Fixes #92
Totktonada added a commit that referenced this issue Oct 29, 2021
Otherwise 'set' and 'register' tests, which use synchronous spaces, fail
with the 'The synchronous transaction queue doesn't belong to any
instance' error.

Those tests did pass before the change [1] in tarantool, which was made
in the scope of the issue [2]. The commit land into release branches as
2.7.2-160-g7310288fa, 2.8.1-168-g806123567, 2.9.0-221-g362e9a668.

[1]: tarantool/tarantool@362e9a6
[2]: tarantool/tarantool#6034

Fixes #92
@Totktonada Totktonada self-assigned this Nov 1, 2021
ligurio pushed a commit that referenced this issue Nov 2, 2021
Otherwise 'set' and 'register' tests, which use synchronous spaces, fail
with the 'The synchronous transaction queue doesn't belong to any
instance' error.

Those tests did pass before the change [1] in tarantool, which was made
in the scope of the issue [2]. The commit land into release branches as
2.7.2-160-g7310288fa, 2.8.1-168-g806123567, 2.9.0-221-g362e9a668.

[1]: tarantool/tarantool@362e9a6
[2]: tarantool/tarantool#6034

Fixes #92
@kyukhin kyukhin added the teamE label Nov 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants