Skip to content

Conversation

@ijon
Copy link
Collaborator

@ijon ijon commented Mar 24, 2025

Add more tests on database admin actions in tenant database for mode enable_strict_user_management+domain_login_only.
Proper functional tests, which were impossible to do for that mode until recently.

Depends on:

@ijon ijon requested a review from a team as a code owner March 24, 2025 10:03
@github-actions
Copy link

github-actions bot commented Mar 24, 2025

🟢 2025-03-25 11:03:16 UTC The validation of the Pull Request description is successful.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@ijon ijon force-pushed the security-more-tests-for-admin-actions branch from cf98ae9 to 0da21e7 Compare March 24, 2025 17:18
@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

CyberROFL
CyberROFL previously approved these changes Mar 25, 2025
@ijon ijon force-pushed the security-more-tests-for-admin-actions branch from 0da21e7 to bd5780d Compare March 25, 2025 11:00
@github-actions
Copy link

github-actions bot commented Mar 25, 2025

2025-03-25 11:04:04 UTC Pre-commit check linux-x86_64-relwithdebinfo for 77639ad has started.
2025-03-25 11:04:20 UTC Artifacts will be uploaded here
2025-03-25 11:06:42 UTC ya make is running...
🟢 2025-03-25 11:15:53 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
140 132 0 0 2 6

🟢 2025-03-25 11:16:03 UTC Build successful.
🟡 2025-03-25 11:16:15 UTC ydbd size 2.2 GiB changed* by +139.4 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 70be00c merge: 77639ad diff diff %
ydbd size 2 321 084 280 Bytes 2 321 227 056 Bytes +139.4 KiB +0.006%
ydbd stripped size 485 261 312 Bytes 485 297 344 Bytes +35.2 KiB +0.007%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@github-actions
Copy link

github-actions bot commented Mar 25, 2025

2025-03-25 11:04:32 UTC Pre-commit check linux-x86_64-release-asan for 77639ad has started.
2025-03-25 11:04:48 UTC Artifacts will be uploaded here
2025-03-25 11:07:06 UTC ya make is running...
🟢 2025-03-25 11:08:49 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
13 13 0 0 0 0

🟢 2025-03-25 11:08:59 UTC Build successful.
🟡 2025-03-25 11:09:12 UTC ydbd size 3.8 GiB changed* by +226.0 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 70be00c merge: 77639ad diff diff %
ydbd size 4 083 249 072 Bytes 4 083 480 544 Bytes +226.0 KiB +0.006%
ydbd stripped size 1 410 852 488 Bytes 1 410 958 120 Bytes +103.2 KiB +0.007%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@ijon ijon merged commit 575de06 into ydb-platform:main Mar 25, 2025
14 checks passed
@ijon ijon deleted the security-more-tests-for-admin-actions branch March 25, 2025 11:33
ijon added a commit to ijon/ydb that referenced this pull request Mar 25, 2025
Add more tests on database admin actions in tenant database for mode `enable_strict_user_management`+`domain_login_only`.
Proper functional tests, which were impossible to do for that mode until recently.
ijon added a commit to ijon/ydb that referenced this pull request Mar 26, 2025
Add more tests on database admin actions in tenant database for mode `enable_strict_user_management`+`domain_login_only`.
Proper functional tests, which were impossible to do for that mode until recently.
lberserq pushed a commit to lberserq/ydb that referenced this pull request Mar 28, 2025
Add more tests on database admin actions in tenant database for mode `enable_strict_user_management`+`domain_login_only`.
Proper functional tests, which were impossible to do for that mode until recently.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants