Skip to content

Allow DescribePartition request if UpdateRow permission is granted #4057

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

qyryq
Copy link
Collaborator

@qyryq qyryq commented Apr 23, 2024

No description provided.

qyryq added 3 commits April 22, 2024 08:21
Add Access field to TSchemeCacheNavigate::TEntry.
Fix (?) parameter to SetErrorAndClear - should be inverted.
Add DescribePartitionPermissions test to ydb_topic/ut/describe_topic_ut.cpp.
Copy link

github-actions bot commented Apr 23, 2024

2024-04-23 20:38:49 UTC Pre-commit check for 203769f has started.
2024-04-23 20:38:52 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-04-23 21:01:05 UTC Build successful.

Copy link

github-actions bot commented Apr 23, 2024

2024-04-23 20:38:53 UTC Pre-commit check for 203769f has started.
2024-04-23 20:38:55 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-04-23 21:02:02 UTC Build successful.
2024-04-23 21:03:47 UTC Tests are running...
🔴 2024-04-23 22:18:49 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
13611 11702 0 24 1860 25

Copy link

github-actions bot commented Apr 23, 2024

2024-04-23 20:39:00 UTC Pre-commit check for 203769f has started.
2024-04-23 20:39:03 UTC Build linux-x86_64-release-asan is running...
🟢 2024-04-23 21:02:24 UTC Build successful.
2024-04-23 21:04:10 UTC Tests are running...
🔴 2024-04-23 22:50:53 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
9510 9366 0 44 81 19

Copy link

github-actions bot commented Apr 24, 2024

2024-04-24 06:53:41 UTC Pre-commit check for 3479e07 has started.
2024-04-24 06:53:43 UTC Build linux-x86_64-release-asan is running...
🟢 2024-04-24 07:06:15 UTC Build successful.
2024-04-24 07:07:59 UTC Tests are running...
🔴 2024-04-24 07:43:46 UTC Test run completed, no test results found for commit fdbc79b. Please check build logs.
2024-04-24 07:43:49 UTC Check cancelled

Copy link

github-actions bot commented Apr 24, 2024

2024-04-24 06:53:43 UTC Pre-commit check for 3479e07 has started.
2024-04-24 06:53:46 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-04-24 07:07:12 UTC Build successful.
2024-04-24 07:08:57 UTC Tests are running...
🔴 2024-04-24 07:43:47 UTC Test run completed, no test results found for commit fdbc79b. Please check build logs.
2024-04-24 07:43:50 UTC Check cancelled

Copy link

github-actions bot commented Apr 24, 2024

2024-04-24 06:53:48 UTC Pre-commit check for 3479e07 has started.
2024-04-24 06:53:50 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-04-24 07:06:28 UTC Build successful.

Copy link

github-actions bot commented Apr 24, 2024

2024-04-24 07:47:16 UTC Pre-commit check for 3d99ab3 has started.
2024-04-24 07:47:18 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-04-24 07:51:42 UTC Build successful.

Copy link

github-actions bot commented Apr 24, 2024

2024-04-24 07:48:03 UTC Pre-commit check for 3d99ab3 has started.
2024-04-24 07:48:06 UTC Build linux-x86_64-release-asan is running...
🟢 2024-04-24 07:52:21 UTC Build successful.
2024-04-24 07:54:29 UTC Tests are running...
🔴 2024-04-24 09:41:34 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
9502 9356 0 40 93 13

Copy link

github-actions bot commented Apr 24, 2024

2024-04-24 07:48:24 UTC Pre-commit check for 3d99ab3 has started.
2024-04-24 07:48:27 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-04-24 07:52:01 UTC Build successful.
2024-04-24 07:53:53 UTC Tests are running...
🔴 2024-04-24 09:13:17 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
13607 11721 0 9 1860 17

Copy link

github-actions bot commented Apr 24, 2024

2024-04-24 13:47:46 UTC Pre-commit check for 19b3c1b has started.
2024-04-24 13:47:49 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-04-24 13:51:48 UTC Build successful.

Copy link

github-actions bot commented Apr 24, 2024

2024-04-24 13:48:17 UTC Pre-commit check for 19b3c1b has started.
2024-04-24 13:48:20 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-04-24 13:52:30 UTC Build successful.
2024-04-24 13:54:18 UTC Tests are running...
🔴 2024-04-24 15:07:10 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
13608 11725 0 6 1861 16

Copy link

github-actions bot commented Apr 24, 2024

2024-04-24 13:47:55 UTC Pre-commit check for 19b3c1b has started.
2024-04-24 13:47:57 UTC Build linux-x86_64-release-asan is running...
🟢 2024-04-24 13:52:19 UTC Build successful.
2024-04-24 13:54:02 UTC Tests are running...
🔴 2024-04-24 15:33:48 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
9507 9377 0 30 88 12

@qyryq qyryq force-pushed the persqueue-permit-describe-partition-if-has-write-permission branch from 51ac84a to 7ca2632 Compare April 25, 2024 07:17
Copy link

github-actions bot commented Apr 25, 2024

2024-04-25 10:00:09 UTC Pre-commit check for 618c7b0 has started.
2024-04-25 10:00:12 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-04-25 10:01:19 UTC Build successful.
2024-04-25 10:03:06 UTC Tests are running...
🔴 2024-04-25 11:13:04 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
13621 11716 0 8 1869 28

Copy link

github-actions bot commented Apr 25, 2024

2024-04-25 09:57:21 UTC Pre-commit check for 618c7b0 has started.
2024-04-25 09:57:22 UTC Build linux-x86_64-release-asan is running...
🟢 2024-04-25 09:58:21 UTC Build successful.
2024-04-25 09:59:57 UTC Tests are running...
🔴 2024-04-25 11:40:45 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
9528 9352 0 36 109 31

Copy link

github-actions bot commented Apr 25, 2024

2024-04-25 07:20:13 UTC Pre-commit check for 618c7b0 has started.
2024-04-25 07:20:15 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-04-25 07:22:26 UTC Build successful.

@@ -296,12 +296,11 @@ namespace {
continue;
}

const ui32 access = GetAccess(entry);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you remove this function?

@qyryq qyryq merged commit f78e8f8 into ydb-platform:main Apr 25, 2024
3 of 5 checks passed
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