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

hotkeys crush on describe #3462

Merged

Conversation

StekPerepolnen
Copy link
Collaborator

@StekPerepolnen StekPerepolnen commented Apr 4, 2024

#3411
hotkeys: added HasPathDescription() check to prevent crush case

Copy link

github-actions bot commented Apr 4, 2024

2024-04-04 01:19:52 UTC Pre-commit check for 0b6032a has started.
2024-04-04 01:19:55 UTC Build linux-x86_64-release-clang14 is running...
🟢 2024-04-04 01:23:17 UTC Build successful.

Copy link

github-actions bot commented Apr 4, 2024

2024-04-04 01:19:53 UTC Pre-commit check for 0b6032a has started.
2024-04-04 01:19:55 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-04-04 01:23:03 UTC Build successful.
2024-04-04 01:24:46 UTC Tests are running...
🔴 2024-04-04 02:33:35 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
9669 9588 0 20 46 15

Copy link

github-actions bot commented Apr 4, 2024

2024-04-04 01:19:54 UTC Pre-commit check for 0b6032a has started.
2024-04-04 01:19:56 UTC Build linux-x86_64-release-asan is running...
🟢 2024-04-04 01:23:30 UTC Build successful.
2024-04-04 01:25:16 UTC Tests are running...
🔴 2024-04-04 02:55:06 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
8668 8461 0 38 146 23

@StekPerepolnen StekPerepolnen merged commit adb83de into ydb-platform:main Apr 4, 2024
6 of 8 checks passed
@StekPerepolnen StekPerepolnen deleted the hotkeys-crush-on-describe branch April 4, 2024 17:40
@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