Skip to content

[ratelimiter] fix not_allowed message if no admins on cluster in configuration #2537

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

Conversation

shmel1k
Copy link
Member

@shmel1k shmel1k commented Mar 7, 2024

Fixed case when no administrators are configured via administration_allowed_sids option.

Changelog entry

...

Changelog category

  • Bugfix

Additional information

...

Copy link

github-actions bot commented Mar 7, 2024

2024-03-07 08:31:28 UTC Pre-commit check for 6a18928 has started.
2024-03-07 08:31:31 UTC Build linux-x86_64-release-cmake14 is running...
🟢 2024-03-07 08:39:18 UTC Build successful.

Copy link

github-actions bot commented Mar 7, 2024

2024-03-07 08:31:29 UTC Pre-commit check for 6a18928 has started.
2024-03-07 08:31:32 UTC Build linux-x86_64-release-asan is running...
🟢 2024-03-07 08:33:17 UTC Build successful.
2024-03-07 08:33:31 UTC Tests are running...
🔴 2024-03-07 10:12:58 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
14943 14783 0 25 106 29

Copy link

github-actions bot commented Mar 7, 2024

2024-03-07 08:31:32 UTC Pre-commit check for 6a18928 has started.
2024-03-07 08:31:34 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-03-07 08:33:20 UTC Build successful.
2024-03-07 08:33:32 UTC Tests are running...
🔴 2024-03-07 10:11:04 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
68251 57310 0 5 10918 18

@shmel1k shmel1k merged commit 942831f into ydb-platform:main Mar 7, 2024
shmel1k added a commit to shmel1k/ydb that referenced this pull request Mar 7, 2024
@shnikd shnikd mentioned this pull request Mar 26, 2024
This was referenced Mar 29, 2024
@shnikd shnikd mentioned this pull request Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants