Skip to content

pool: get status of an instance via WATCH_ONCE #380

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
oleg-jukovec opened this issue Feb 6, 2024 · 0 comments · Fixed by #386
Closed

pool: get status of an instance via WATCH_ONCE #380

oleg-jukovec opened this issue Feb 6, 2024 · 0 comments · Fixed by #386
Assignees
Labels
2sp feature A new functionality teamE

Comments

@oleg-jukovec
Copy link
Collaborator

oleg-jukovec commented Feb 6, 2024

At now we make evals on the server side to get an instance status. It requires execute access for a user.
We could use WATCH_ONCE instead for servers that support it. It will allow us to require fewer rights from the user.

@oleg-jukovec oleg-jukovec added teamE feature A new functionality 2sp labels Feb 6, 2024
DerekBum added a commit that referenced this issue Mar 5, 2024
Starting from Tarantool version >= 3.0.0 `WatchOnce` requset is
supported. So we can get instance status using this request instead
of calling `box.info`.

This way user can add instances to the pool without the `execute` role.

Closes #380
DerekBum added a commit that referenced this issue Mar 5, 2024
Starting from Tarantool version >= 3.0.0 `WatchOnce` requset is
supported. So we can get instance status using this request instead
of calling `box.info`.

This way user can add instances to the pool without the `execute` role.

Closes #380
DerekBum added a commit that referenced this issue Mar 5, 2024
Starting from Tarantool version >= 3.0.0 `WatchOnce` requset is
supported. So we can get instance status using this request instead
of calling `box.info`.

This way user can add instances to the pool without the `execute` role.

Closes #380
DerekBum added a commit that referenced this issue Mar 5, 2024
Starting from Tarantool version >= 3.0.0 `WatchOnce` requset is
supported. So we can get instance status using this request instead
of calling `box.info`.

This way user can add instances to the pool without the `execute` role.

Closes #380
DerekBum added a commit that referenced this issue Mar 5, 2024
Starting from Tarantool version >= 3.0.0 `WatchOnce` requset is
supported. So we can get instance status using this request instead
of calling `box.info`.

This way user can add instances to the ConnectionPool without the
`execute` access.

Closes #380
oleg-jukovec pushed a commit that referenced this issue Mar 6, 2024
Starting from Tarantool version >= 3.0.0 `WatchOnce` requset is
supported. So we can get instance status using this request instead
of calling `box.info`.

This way user can add instances to the ConnectionPool without the
`execute` access.

Closes #380
oleg-jukovec added a commit that referenced this issue Mar 6, 2024
Overview

    The small release improves the ConnectionPool. The ConnectionPool
    is no longer required execute access for `box.info` from a user
    for Tarantool >= 3.0.0.

Breaking changes

    There are no breaking changes in the release.

New features

    `execute` access for `box.info` is no longer required for
    ConnectionPool for a Tarantool version >= 3.0.0 (#380).

Bugfixes

    `ConnectionPool.Remove()` does not notify a `ConnectionHandler`
    after an instance is already removed from the pool (#385).
oleg-jukovec added a commit that referenced this issue Mar 6, 2024
Overview

    The small release improves the ConnectionPool. The ConnectionPool
    is no longer required execute access for `box.info` from a user
    for Tarantool >= 3.0.0.

Breaking changes

    There are no breaking changes in the release.

New features

    `execute` access for `box.info` is no longer required for
    ConnectionPool for a Tarantool version >= 3.0.0 (#380).

Bugfixes

    `ConnectionPool.Remove()` does not notify a `ConnectionHandler`
    after an instance is already removed from the pool (#385).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2sp feature A new functionality teamE
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants