Skip to content

Queue tests fail #115

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
ylobankov opened this issue Dec 2, 2021 · 0 comments · Fixed by #140
Closed

Queue tests fail #115

ylobankov opened this issue Dec 2, 2021 · 0 comments · Fixed by #140
Assignees
Labels
bug Something isn't working code health Improve code readability, simplify maintenance and so on

Comments

@ylobankov
Copy link

OS: Ubuntu 20.04
Tarantool: 2.8.2-0-gfc96d10f5

For now, all queue tests fail.

$ cd queue/
$
$ mkdir snap xlog
$
$ tarantoolctl rocks install queue 1.1.0
$
$ tarantool ./config.lua > tarantool.log 2>&1 &
[1] 988812
$
$ cat tarantool.log
Deprecated option snap_dir, please use memtx_dir instead
2021-12-02 08:51:16.580 [988046] main/103/config.lua C> Tarantool 2.8.2-0-gfc96d10f5
2021-12-02 08:51:16.580 [988046] main/103/config.lua C> log level 5
2021-12-02 08:51:16.580 [988046] main/103/config.lua I> wal/engine cleanup is paused
2021-12-02 08:51:16.580 [988046] main/103/config.lua I> mapping 268435456 bytes for memtx tuple arena...
2021-12-02 08:51:16.580 [988046] main/103/config.lua I> Actual slab_alloc_factor calculated on the basis of desired slab_alloc_factor = 1.044274
2021-12-02 08:51:16.580 [988046] main/103/config.lua I> mapping 134217728 bytes for vinyl tuple arena...
2021-12-02 08:51:16.582 [988046] main/103/config.lua I> instance uuid a227e131-f9eb-4e7f-8b80-a4536bca71f7
2021-12-02 08:51:16.582 [988046] main/103/config.lua I> tx_binary: stopped
2021-12-02 08:51:16.582 [988046] main/103/config.lua I> tx_binary: bound to 0.0.0.0:3013
2021-12-02 08:51:16.582 [988046] main/103/config.lua I> initializing an empty data directory
2021-12-02 08:51:16.592 [988046] main/103/config.lua I> assigned id 1 to replica a227e131-f9eb-4e7f-8b80-a4536bca71f7
2021-12-02 08:51:16.592 [988046] main/103/config.lua I> cluster uuid c32cf3a6-70b5-485c-b9e5-913ad7300151
2021-12-02 08:51:16.592 [988046] snapshot/101/main I> saving snapshot `snap/00000000000000000000.snap.inprogress'
2021-12-02 08:51:16.607 [988046] snapshot/101/main I> done
2021-12-02 08:51:16.607 [988046] main/103/config.lua I> ready to accept requests
2021-12-02 08:51:16.607 [988046] main/104/gc I> wal/engine cleanup is resumed
2021-12-02 08:51:16.607 [988046] main/103/config.lua I> set 'log_level' configuration option to 5
2021-12-02 08:51:16.607 [988046] main/105/checkpoint_daemon I> scheduled next checkpoint for Thu Dec  2 10:12:11 2021
2021-12-02 08:51:16.608 [988046] main/103/config.lua I> set 'listen' configuration option to "3013"
2021-12-02 08:51:16.608 [988046] main/103/config.lua I> set 'log_format' configuration option to "plain"
2021-12-02 08:51:16.618 [988046] main schema.lua:2498 E> ER_NO_SUCH_SPACE: Space '_queue_taken' does not exist
2021-12-02 08:51:16.618 [988046] main F> fatal error, exiting the event loop

According to this, _queue_taken is deprecated. So let's try to use _queue_taken_2. Open the config.lua file and change box.schema.user.grant('test', 'read,write', 'space', '_queue_taken') to box.schema.user.grant('test', 'read,write', 'space', '_queue_taken_2').

$ ps aux | awk '/tarantool/{print $2}' | head -1 | xargs kill
$
$ rm -rf snap xlog && mkdir snap xlog
$
$ tarantool ./config.lua > tarantool.log 2>&1 &
[1] 988893
$
$ cat tarantool.log
Deprecated option snap_dir, please use memtx_dir instead
2021-12-02 09:10:55.567 [988893] main/103/config.lua C> Tarantool 2.8.2-0-gfc96d10f5
2021-12-02 09:10:55.567 [988893] main/103/config.lua C> log level 5
2021-12-02 09:10:55.567 [988893] main/103/config.lua I> wal/engine cleanup is paused
2021-12-02 09:10:55.567 [988893] main/103/config.lua I> mapping 268435456 bytes for memtx tuple arena...
2021-12-02 09:10:55.567 [988893] main/103/config.lua I> Actual slab_alloc_factor calculated on the basis of desired slab_alloc_factor = 1.044274
2021-12-02 09:10:55.567 [988893] main/103/config.lua I> mapping 134217728 bytes for vinyl tuple arena...
2021-12-02 09:10:55.568 [988893] main/103/config.lua I> instance uuid 8fead1ef-bff9-4e30-a857-c47ce4833944
2021-12-02 09:10:55.568 [988893] main/103/config.lua I> tx_binary: stopped
2021-12-02 09:10:55.569 [988893] main/103/config.lua I> tx_binary: bound to 0.0.0.0:3013
2021-12-02 09:10:55.569 [988893] main/103/config.lua I> initializing an empty data directory
2021-12-02 09:10:55.579 [988893] main/103/config.lua I> assigned id 1 to replica 8fead1ef-bff9-4e30-a857-c47ce4833944
2021-12-02 09:10:55.579 [988893] main/103/config.lua I> cluster uuid 1f211e77-88c4-4a38-9875-b4105ee1f45c
2021-12-02 09:10:55.580 [988893] snapshot/101/main I> saving snapshot `snap/00000000000000000000.snap.inprogress'
2021-12-02 09:10:55.589 [988893] snapshot/101/main I> done
2021-12-02 09:10:55.590 [988893] main/103/config.lua I> ready to accept requests
2021-12-02 09:10:55.590 [988893] main/104/gc I> wal/engine cleanup is resumed
2021-12-02 09:10:55.590 [988893] main/103/config.lua I> set 'log_level' configuration option to 5
2021-12-02 09:10:55.590 [988893] main/105/checkpoint_daemon I> scheduled next checkpoint for Thu Dec  2 10:52:58 2021
2021-12-02 09:10:55.590 [988893] main/103/config.lua I> set 'listen' configuration option to "3013"
2021-12-02 09:10:55.590 [988893] main/103/config.lua I> set 'log_format' configuration option to "plain"
2021-12-02 09:10:55.599 [988893] main C> entering the event loop
$
$ go clean -testcache && go test -v
=== RUN   TestFifoQueue
--- FAIL: TestFifoQueue (0.00s)
    queue_test.go:40: Failed to create queue: Create access to space 'test_queue' is denied for user 'test' (0x2a)
=== RUN   TestFifoQueue_GetExist_Statistic
--- FAIL: TestFifoQueue_GetExist_Statistic (0.00s)
    queue_test.go:65: Failed to create queue: Create access to space 'test_queue' is denied for user 'test' (0x2a)
=== RUN   TestFifoQueue_Put
--- FAIL: TestFifoQueue_Put (0.00s)
    queue_test.go:116: Failed to create queue: Create access to space 'test_queue' is denied for user 'test' (0x2a)
=== RUN   TestFifoQueue_Take
--- FAIL: TestFifoQueue_Take (0.03s)
    queue_test.go:159: Failed to create queue: Create access to space 'test_queue' is denied for user 'test' (0x2a)
=== RUN   TestFifoQueue_TakeTyped
--- FAIL: TestFifoQueue_TakeTyped (0.00s)
    queue_test.go:255: Failed to create queue: Create access to space 'test_queue' is denied for user 'test' (0x2a)
=== RUN   TestFifoQueue_Peek
--- FAIL: TestFifoQueue_Peek (0.00s)
    queue_test.go:332: Failed to create queue: Create access to space 'test_queue' is denied for user 'test' (0x2a)
=== RUN   TestFifoQueue_Bury_Kick
--- FAIL: TestFifoQueue_Bury_Kick (0.00s)
    queue_test.go:391: Failed to create queue: Create access to space 'test_queue' is denied for user 'test' (0x2a)
=== RUN   TestFifoQueue_Delete
--- FAIL: TestFifoQueue_Delete (0.00s)
    queue_test.go:478: Failed to create queue: Create access to space 'test_queue' is denied for user 'test' (0x2a)
=== RUN   TestFifoQueue_Release
--- FAIL: TestFifoQueue_Release (0.00s)
    queue_test.go:553: Failed to create queue: Create access to space 'test_queue' is denied for user 'test' (0x2a)
=== RUN   TestTtlQueue
--- FAIL: TestTtlQueue (0.00s)
    queue_test.go:643: Failed to create queue: Create access to space 'test_queue' is denied for user 'test' (0x2a)
=== RUN   TestTtlQueue_Put
--- FAIL: TestTtlQueue_Put (0.00s)
    queue_test.go:700: Failed to create queue: Create access to space 'test_queue' is denied for user 'test' (0x2a)
=== RUN   TestUtube_Put
--- FAIL: TestUtube_Put (0.00s)
    queue_test.go:772: Failed to create queue: Create access to space 'test_utube' is denied for user 'test' (0x2a)
FAIL
exit status 1
FAIL	github.com/tarantool/go-tarantool/queue	0.063s
@ylobankov ylobankov added bug Something isn't working teamE labels Dec 2, 2021
@Totktonada Totktonada added code health Improve code readability, simplify maintenance and so on 2sp labels Dec 13, 2021
DifferentialOrange added a commit that referenced this issue Jan 17, 2022
Before this patch, it was required to set up test tarantool processes
manually (and also handle their dependencies, like making working dir).
You can see an example in CI scripts.

This patch introduces go helpers for starting a tarantool process and
installing rock requirements with tarantoolctl. Helpers are based on
`os/exec` calls. Retries to connect test tarantool instance handled
explicitly, see #136.

Setup scripts are reworked to use environment variables to configure
`box.cfg`. Listen port is set in the end of script so it is possible
to connect only if every other thing was set up already.

Every test is reworked to start a tarantool process (or processes) in
TestMain before test run. Now it is possible to run a test with plain
`go test`. Queue tests changes may be broken because it is impossible
to verify before #115 is fixed.

Closes #107
vr009 added a commit that referenced this issue Jan 26, 2022
Queue tests firstly failed on using _queue_taken which was
deprecated in tarantool/queue. Switch to _queue_taken_2 fixed it.

Additionally there were troubles with grants for test user.
Additional grants fixed this problem.

The test TestUtube_Put was affected by replacing timeout in
the call of TakeTimeout() with timeout of connection structure.
For clarification of this effect see annotation for function
TakeTimeout(timeout time.Duration). That was fixed by making
connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Queue tests firstly failed on using _queue_taken which was
deprecated in tarantool/queue. Switch to _queue_taken_2 fixed it.

Additionally there were troubles with grants for test user.
Additional grants fixed this problem.

The test TestUtube_Put was affected by replacing timeout in
the call of TakeTimeout() with timeout of connection structure.
For clarification of this effect see annotation for function
TakeTimeout(timeout time.Duration). That was fixed by making
connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Queue tests firstly failed on using _queue_taken which was
deprecated in tarantool/queue. Switch to _queue_taken_2 fixed it.

Additionally there were troubles with grants for test user.
Additional grants fixed this problem.

The test TestUtube_Put was affected by replacing timeout in
the call of TakeTimeout() with timeout of connection structure.
For clarification of this effect see annotation for function
TakeTimeout(timeout time.Duration). That was fixed by making
connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Queue tests firstly failed on using _queue_taken which was
deprecated in tarantool/queue. Switch to _queue_taken_2 fixed it.

Additionally there were troubles with grants for test user.
Additional grants fixed this problem.

The test TestUtube_Put was affected by replacing timeout in
the call of TakeTimeout() with timeout of connection structure.
For clarification of this effect see annotation for function
TakeTimeout(timeout time.Duration). That was fixed by making
connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Queue tests firstly failed on using _queue_taken which was
deprecated in tarantool/queue. Switch to _queue_taken_2 fixed it.

Additionally there were troubles with grants for test user.
Additional grants fixed this problem.

The test TestUtube_Put was affected by replacing timeout in
the call of TakeTimeout() with timeout of connection structure.
For clarification of this effect see annotation for function
TakeTimeout(timeout time.Duration). That was fixed by making
connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in
the call of `TakeTimeout()` with timeout of connection structure.
For clarification of this effect see annotation for function
`TakeTimeout()` [2]. That was fixed by making connection timeout
longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
vr009 added a commit that referenced this issue Jan 26, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
DifferentialOrange added a commit that referenced this issue Jan 31, 2022
Before this patch, it was required to set up test tarantool processes
manually (and also handle their dependencies, like making working dir).
You can see an example in CI scripts.

This patch introduces go helpers for starting a tarantool process and
installing rock requirements with tarantoolctl. Helpers are based on
`os/exec` calls. Retries to connect test tarantool instance handled
explicitly, see #136.

Setup scripts are reworked to use environment variables to configure
`box.cfg`. Listen port is set in the end of script so it is possible
to connect only if every other thing was set up already.

Every test is reworked to start a tarantool process (or processes) in
TestMain before test run. Now it is possible to run a test with plain
`go test`. Queue tests changes may be broken because it is impossible
to verify before #115 is fixed.

Closes #107
vr009 added a commit that referenced this issue Jan 31, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
vr009 added a commit that referenced this issue Jan 31, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
DifferentialOrange added a commit that referenced this issue Jan 31, 2022
Before this patch, it was required to set up test tarantool processes
manually (and also handle their dependencies, like making working dir).
You can see an example in CI scripts.

This patch introduces go helpers for starting a tarantool process and
installing rock requirements with tarantoolctl. Helpers are based on
`os/exec` calls. Retries to connect test tarantool instance handled
explicitly, see #136.

Setup scripts are reworked to use environment variables to configure
`box.cfg`. Listen port is set in the end of script so it is possible
to connect only if every other thing was set up already.

Every test is reworked to start a tarantool process (or processes) in
TestMain before test run. Now it is possible to run a test with plain
`go test`. Queue tests changes may be broken because it is impossible
to verify before #115 is fixed.

Closes #107
DifferentialOrange added a commit that referenced this issue Jan 31, 2022
Before this patch, it was required to set up test tarantool processes
manually (and also handle their dependencies, like making working dir).
You can see an example in CI scripts.

This patch introduces go helpers for starting a tarantool process and
installing rock requirements with tarantoolctl. Helpers are based on
`os/exec` calls. Retries to connect test tarantool instance handled
explicitly, see #136.

Setup scripts are reworked to use environment variables to configure
`box.cfg`. Listen port is set in the end of script so it is possible
to connect only if every other thing was set up already.

Every test is reworked to start a tarantool process (or processes) in
TestMain before test run. Now it is possible to run a test with plain
`go test`. Queue tests changes may be broken because it is impossible
to verify before #115 is fixed.

Closes #107
DifferentialOrange added a commit that referenced this issue Feb 1, 2022
Before this patch, it was required to set up test tarantool processes
manually (and also handle their dependencies, like making working dir).
You can see an example in CI scripts.

This patch introduces go helpers for starting a tarantool process and
installing rock requirements with tarantoolctl. Helpers are based on
`os/exec` calls. Retries to connect test tarantool instance handled
explicitly, see #136.

Setup scripts are reworked to use environment variables to configure
`box.cfg`. Listen port is set in the end of script so it is possible
to connect only if every other thing was set up already.

Every test is reworked to start a tarantool process (or processes) in
TestMain before test run. Now it is possible to run a test with plain
`go test`. Queue tests changes may be broken because it is impossible
to verify before #115 is fixed.

Closes #107
DifferentialOrange added a commit that referenced this issue Feb 1, 2022
Before this patch, it was required to set up test tarantool processes
manually (and also handle their dependencies, like making working dir).
You can see an example in CI scripts.

This patch introduces go helpers for starting a tarantool process and
installing rock requirements with tarantoolctl. Helpers are based on
`os/exec` calls. Retries to connect test tarantool instance handled
explicitly, see #136.

Setup scripts are reworked to use environment variables to configure
`box.cfg`. Listen port is set in the end of script so it is possible
to connect only if every other thing was set up already.

Every test is reworked to start a tarantool process (or processes) in
TestMain before test run. Now it is possible to run a test with plain
`go test`. Queue tests changes may be broken because it is impossible
to verify before #115 is fixed.

Closes #107
vr009 added a commit that referenced this issue Feb 1, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
@vr009 vr009 closed this as completed in #140 Feb 1, 2022
vr009 added a commit that referenced this issue Feb 1, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
ligurio pushed a commit that referenced this issue Mar 7, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
ligurio pushed a commit that referenced this issue Mar 30, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
ligurio pushed a commit that referenced this issue Apr 7, 2022
Before this patch, queue tests failed due to impossibility to use _queue_taken,
which was deprecated in tarantool/queue [1]. Switch to _queue_taken_2 fixed it.

Create access and some other actions were denied for test user.
Additional grants fixed this problem.

The test `TestUtube_Put` was affected by replacing timeout in the call
of `TakeTimeout()` with timeout of connection structure. For clarification
of this effect see annotation for function `TakeTimeout()` [2]. That was
fixed by making connection timeout longer than the timeout in the function call.

The step in testing.yml was uncommented for enabling queue tests in CI.

1. https://github.com/tarantool/queue/tree/7d05b6db5b0d596db4281d53b57811e156234a0d#fields-of-the-_queue_consumers-space
2. https://github.com/tarantool/go-tarantool/blob/master/queue/queue.go#L35

Fixes #115
oleg-jukovec added a commit that referenced this issue Jun 1, 2022
Overview

This release adds a number of features and updates documentation.

Breaking changes

	There are no breaking changes in the release.

New features

	Coveralls support (#149).

	Reusable testing workflow (integration testing with latest Tarantool) (#123).

	Simple CI based on GitHub actions (#114).

	Support UUID type in msgpack (#90).

	Go modules support (#91).

	queue-utube handling (#85).

	Master discovery (#113).

	SQL support (#62).

    Handle everything with `go test` (#115).

	Update API documentation: comments and examples (#123).

Bugfixes

	Reset buffer if its average use size smaller than quater of capacity (#95).

	Fix queue tests (#107).

	Make test case consistent with comments (#105).
oleg-jukovec added a commit that referenced this issue Jun 1, 2022
Overview

This release adds a number of features and updates documentation.

Breaking changes

	There are no breaking changes in the release.

New features

	Coveralls support (#149).

	Reusable testing workflow (integration testing with latest
	Tarantool) (#123).

	Simple CI based on GitHub actions (#114).

	Support UUID type in msgpack (#90).

	Go modules support (#91).

	queue-utube handling (#85).

	Master discovery (#113).

	SQL support (#62).

    Handle everything with `go test` (#115).

	Update API documentation: comments and examples (#123).

Bugfixes

	Reset buffer if its average use size smaller than quater of
	capacity (#95).

	Fix queue tests (#107).

	Make test case consistent with comments (#105).
oleg-jukovec added a commit that referenced this issue Jun 1, 2022
Overview

This release adds a number of features and updates documentation.

Breaking changes

	There are no breaking changes in the release.

New features

	Coveralls support (#149).

	Reusable testing workflow (integration testing with latest
	Tarantool) (#123).

	Simple CI based on GitHub actions (#114).

	Support UUID type in msgpack (#90).

	Go modules support (#91).

	queue-utube handling (#85).

	Master discovery (#113).

	SQL support (#62).

	Handle everything with `go test` (#115).

	Update API documentation: comments and examples (#123).

Bugfixes

	Reset buffer if its average use size smaller than quater of
	capacity (#95).

	Fix queue tests (#107).

	Make test case consistent with comments (#105).
oleg-jukovec added a commit that referenced this issue Jun 1, 2022
Overview

This release adds a number of features and updates documentation.

Breaking changes

	There are no breaking changes in the release.

New features

	Coveralls support (#149).

	Reusable testing workflow (integration testing with latest
	Tarantool) (#112).

	Simple CI based on GitHub actions (#114).

	Support UUID type in msgpack (#90).

	Go modules support (#91).

	queue-utube handling (#85).

	Master discovery (#113).

	SQL support (#62).

	Handle everything with `go test` (#115).

	Update API documentation: comments and examples (#123).

Bugfixes

	Reset buffer if its average use size smaller than quater of
	capacity (#95).

	Fix queue tests (#107).

	Make test case consistent with comments (#105).
oleg-jukovec added a commit that referenced this issue Jun 2, 2022
Overview

This release adds a number of features. Also it significantly improves
testing, CI and documentation.

Breaking changes

    There are no breaking changes in the release.

New features

    Support UUID type in msgpack (#90).

    queue-utube handling (#85).

    Master discovery (#113).

    SQL support (#62).

Bugfixes

    Reset buffer if its average use size smaller than quater of
    capacity (#95).

Testing

    Coveralls support (#149).

    Reusable testing workflow (integration testing with latest
    Tarantool) (#112).

    Simple CI based on GitHub actions (#114).

    Handle everything with `go test` (#115).

    Fix queue tests (#107).

    Make test case consistent with comments (#105).

Other

    Go modules support (#91).

    Update API documentation: comments and examples (#123).
oleg-jukovec added a commit that referenced this issue Jun 2, 2022
Overview

This release adds a number of features. Also it significantly improves
testing, CI and documentation.

Breaking changes

    There are no breaking changes in the release.

New features

    Support UUID type in msgpack (#90).

    queue-utube handling (#85).

    Master discovery (#113).

    SQL support (#62).

Bugfixes

    Reset buffer if its average use size smaller than quater of
    capacity (#95).

Testing

    Coveralls support (#149).

    Reusable testing workflow (integration testing with latest
    Tarantool) (#112).

    Simple CI based on GitHub actions (#114).

    Handle everything with `go test` (#115).

    Fix queue tests (#107).

    Make test case consistent with comments (#105).

Other

    Go modules support (#91).

    Update API documentation: comments and examples (#123).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working code health Improve code readability, simplify maintenance and so on
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants