Skip to content
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

24-3: Add created_by to Operations API (#6463) #8708

Merged
merged 1 commit into from
Sep 4, 2024

Conversation

pixcc
Copy link
Member

@pixcc pixcc commented Sep 4, 2024

Changelog entry

Add UserSID (Security ID) who created long-running operation (export, import).

Changelog category

  • New feature

Additional information

$ ydb export s3 --user root
┌───────────────────────────────────────────┬───────┬─────────┬───────────┬───────────┬────────┐
│ id                                        │ ready │ status  │ progress  │ endpoint  │ bucket │
├───────────────────────────────────────────┼───────┼─────────┼───────────┼───────────┼────────┤
│ ydb://export/6?id=844424953136969&kind=s3 │ false │ SUCCESS │ Preparing │ localhost │ 1      │
├╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴┴╴╴╴╴╴╴╴┴╴╴╴╴╴╴╴╴╴┴╴╴╴╴╴╴╴╴╴╴╴┴╴╴╴╴╴╴╴╴╴╴╴┴╴╴╴╴╴╴╴╴┤
│ StorageClass: NOT_SET                                                                        │
│ Items:                                                                                       │
│   - source: /Root/serverless/stock                                                           │
│     destination: 1                                                                           │
│ Description:                                                                                 │
│ Number of retries: 10                                                                        │
│ Created by: root                                                                             │
│ Create time: 2024-07-10T17:59:13Z                                                            │
└──────────────────────────────────────────────────────────────────────────────────────────────┘

$ ydb export s3
┌───────────────────────────────────────────┬───────┬─────────┬───────────┬───────────┬────────┐
│ id                                        │ ready │ status  │ progress  │ endpoint  │ bucket │
├───────────────────────────────────────────┼───────┼─────────┼───────────┼───────────┼────────┤
│ ydb://export/6?id=562949976271314&kind=s3 │ false │ SUCCESS │ Preparing │ localhost │ 1      │
├╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴╴┴╴╴╴╴╴╴╴┴╴╴╴╴╴╴╴╴╴┴╴╴╴╴╴╴╴╴╴╴╴┴╴╴╴╴╴╴╴╴╴╴╴┴╴╴╴╴╴╴╴╴┤
│ StorageClass: NOT_SET                                                                        │
│ Items:                                                                                       │
│   - source: /Root/serverless/stock                                                           │
│     destination: 1                                                                           │
│ Description:                                                                                 │
│ Number of retries: 10                                                                        │
│ Create time: 2024-07-09T12:05:51Z                                                            │
└──────────────────────────────────────────────────────────────────────────────────────────────┘

@pixcc pixcc requested a review from CyberROFL September 4, 2024 08:42
@pixcc pixcc requested a review from a team as a code owner September 4, 2024 08:42
@pixcc pixcc self-assigned this Sep 4, 2024
Copy link

github-actions bot commented Sep 4, 2024

2024-09-04 08:44:34 UTC Pre-commit check for ced4e24 has started.
2024-09-04 08:47:35 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-09-04 09:54:07 UTC Build successful.
2024-09-04 09:54:21 UTC Tests are running...
🔴 2024-09-04 11:58:03 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
76855 63401 0 2 13422 30

🟢 2024-09-04 11:58:48 UTC ydbd size 8.2 GiB changed* by +50.2 KiB, which is < 100.0 KiB vs stable-24-3: OK

ydbd size dash stable-24-3: ae19a36 merge: ced4e24 diff diff %
ydbd size 8 760 268 792 Bytes 8 760 320 248 Bytes +50.2 KiB +0.001%
ydbd stripped size 478 958 024 Bytes 478 960 104 Bytes +2.0 KiB +0.000%

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

Copy link

github-actions bot commented Sep 4, 2024

2024-09-04 08:45:05 UTC Pre-commit check for ced4e24 has started.
2024-09-04 08:48:32 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-09-04 09:33:00 UTC Build successful.

Copy link

github-actions bot commented Sep 4, 2024

2024-09-04 08:45:56 UTC Pre-commit check for ced4e24 has started.
2024-09-04 08:48:55 UTC Build linux-x86_64-release-asan is running...
🟢 2024-09-04 09:39:46 UTC Build successful.
2024-09-04 09:40:03 UTC Tests are running...
🔴 2024-09-04 11:36:31 UTC Some tests failed, follow the links below.

Test history | Test log

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
15761 15658 0 25 48 30

🟢 2024-09-04 11:37:09 UTC ydbd size 5.5 GiB changed* by +64.4 KiB, which is < 100.0 KiB vs stable-24-3: OK

ydbd size dash stable-24-3: ae19a36 merge: ced4e24 diff diff %
ydbd size 5 933 776 560 Bytes 5 933 842 488 Bytes +64.4 KiB +0.001%
ydbd stripped size 1 487 710 688 Bytes 1 487 721 312 Bytes +10.4 KiB +0.001%

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

@pixcc pixcc merged commit 2764b01 into ydb-platform:stable-24-3 Sep 4, 2024
8 of 12 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.

3 participants