Skip to content

Conversation

@galnat
Copy link
Collaborator

@galnat galnat commented Feb 4, 2025

Changelog entry

Added github action for validate PR body
Added github actions for generate changelog increment

Changelog category

  • Not for changelog (changelog entry is not required)

Additional information

Added one workflow for validate PR body.
Added two workflows for generate pr to changelog.md - one triggered by cron, another manually.
https://st.yandex-team.ru/KIKIMR-22457
https://st.yandex-team.ru/KIKIMR-22458

@github-actions
Copy link

github-actions bot commented Feb 4, 2025

2025-02-04 18:47:40 UTC Pre-commit check linux-x86_64-release-asan for 5ded659 has started.
2025-02-04 18:47:55 UTC Artifacts will be uploaded here
2025-02-04 18:50:20 UTC ya make is running...
🟢 2025-02-04 18:50:25 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-04 18:50:34 UTC Build successful.

@galnat galnat requested a review from maximyurchuk February 4, 2025 19:27
@maximyurchuk maximyurchuk added the rebase-and-check Rebase PR with the current base branch and check label Feb 5, 2025
@github-actions github-actions bot removed the rebase-and-check Rebase PR with the current base branch and check label Feb 5, 2025
@github-actions
Copy link

github-actions bot commented Feb 5, 2025

2025-02-05 09:25:50 UTC Pre-commit check linux-x86_64-release-asan for de0bbaa has started.
2025-02-05 09:25:54 UTC Artifacts will be uploaded here
2025-02-05 09:28:25 UTC ya make is running...
🟢 2025-02-05 09:28:31 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-05 09:28:37 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 5, 2025

2025-02-05 09:28:28 UTC Pre-commit check linux-x86_64-relwithdebinfo for de0bbaa has started.
2025-02-05 09:28:40 UTC Artifacts will be uploaded here
2025-02-05 09:31:13 UTC ya make is running...
🟢 2025-02-05 09:31:19 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-05 09:31:25 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 9, 2025

2025-02-09 17:12:33 UTC Pre-commit check linux-x86_64-relwithdebinfo for 58894d4 has started.
2025-02-09 17:13:27 UTC Artifacts will be uploaded here
2025-02-09 17:16:06 UTC ya make is running...
🟢 2025-02-09 17:16:11 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-09 17:16:20 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 9, 2025

2025-02-09 17:12:36 UTC Pre-commit check linux-x86_64-release-asan for 58894d4 has started.
2025-02-09 17:16:23 UTC Artifacts will be uploaded here
2025-02-09 17:18:58 UTC ya make is running...
🟢 2025-02-09 17:19:04 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-09 17:19:11 UTC Build successful.

import requests

import os
sys.path.append(os.path.abspath(os.path.join(os.path.dirname(__file__), "../validate_pr_description")))
Copy link
Collaborator

Choose a reason for hiding this comment

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

может перенесем всё это в ydb/ci директорию, что бы не делать так тут?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Окей, но сейчас на ревью еще один скрипт. Я бы тогда в отдельном PR перенесла их в ydb/ci, хорошо?

base_branch = sys.argv[3]
suffix = sys.argv[4]

GITHUB_TOKEN = subprocess.run(["git", "config", "--get", "github.token"], capture_output=True, text=True).stdout.strip()
Copy link
Collaborator

Choose a reason for hiding this comment

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

мне не очень нравится идея передавать токены через git config. как будто бы можно смотреть переменную окружения UPDATE_REPO_TOKEN и не выставлять в экшене этот параметр.

print("PR description is valid.")
return True

except Exception as e:
Copy link
Collaborator

Choose a reason for hiding this comment

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

мне не очень нравятся такие большие try-catch c широкими исключениями, может тут лучше явно падать где-то, ведь это что-то совсем нештатное получается? или если штатное, то может в validate_pr_description_from_file обернуть этот вызов в try-catch?

@github-actions
Copy link

github-actions bot commented Feb 12, 2025

2025-02-12 11:36:12 UTC Pre-commit check linux-x86_64-release-asan for 6717310 has started.
2025-02-12 11:36:17 UTC Artifacts will be uploaded here
2025-02-12 11:38:45 UTC ya make is running...
🟢 2025-02-12 11:38:51 UTC Tests successful.

Test history | Ya make output | Test bloat

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

2025-02-12 11:38:56 UTC Check cancelled

@github-actions
Copy link

github-actions bot commented Feb 12, 2025

2025-02-12 11:37:03 UTC Pre-commit check linux-x86_64-relwithdebinfo for 6717310 has started.
2025-02-12 11:37:54 UTC Artifacts will be uploaded here
2025-02-12 11:38:58 UTC Check cancelled

@github-actions
Copy link

github-actions bot commented Feb 12, 2025

2025-02-12 11:39:39 UTC Pre-commit check linux-x86_64-release-asan for 16eca8d has started.
2025-02-12 11:39:42 UTC Artifacts will be uploaded here
2025-02-12 11:42:08 UTC ya make is running...
🟢 2025-02-12 11:42:14 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-12 11:42:20 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 12, 2025

2025-02-12 11:40:29 UTC Pre-commit check linux-x86_64-relwithdebinfo for 16eca8d has started.
2025-02-12 11:40:42 UTC Artifacts will be uploaded here
2025-02-12 11:42:57 UTC ya make is running...
🟢 2025-02-12 11:43:03 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-12 11:43:09 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 12, 2025

2025-02-12 12:08:59 UTC Pre-commit check linux-x86_64-relwithdebinfo for 2057679 has started.
2025-02-12 12:09:11 UTC Artifacts will be uploaded here
2025-02-12 12:11:33 UTC ya make is running...
🟢 2025-02-12 12:11:39 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-12 12:11:44 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 12, 2025

2025-02-12 12:08:59 UTC Pre-commit check linux-x86_64-release-asan for 2057679 has started.
2025-02-12 12:09:13 UTC Artifacts will be uploaded here
2025-02-12 12:11:36 UTC ya make is running...
🟢 2025-02-12 12:11:41 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-12 12:11:49 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 12, 2025

2025-02-12 12:14:36 UTC Pre-commit check linux-x86_64-relwithdebinfo for 0b29b7d has started.
2025-02-12 12:14:49 UTC Artifacts will be uploaded here
2025-02-12 12:17:15 UTC ya make is running...
🟢 2025-02-12 12:17:21 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-12 12:17:29 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 12, 2025

2025-02-12 12:36:17 UTC Pre-commit check linux-x86_64-release-asan for 14590ff has started.
2025-02-12 12:36:29 UTC Artifacts will be uploaded here
2025-02-12 12:38:54 UTC ya make is running...
🟢 2025-02-12 12:39:00 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-12 12:39:07 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 12, 2025

2025-02-12 12:36:26 UTC Pre-commit check linux-x86_64-relwithdebinfo for 14590ff has started.
2025-02-12 12:36:39 UTC Artifacts will be uploaded here
2025-02-12 12:39:05 UTC ya make is running...
🟢 2025-02-12 12:39:11 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-12 12:39:21 UTC Build successful.

maximyurchuk
maximyurchuk previously approved these changes Feb 14, 2025
@github-actions
Copy link

github-actions bot commented Feb 16, 2025

2025-02-16 08:17:44 UTC Pre-commit check linux-x86_64-release-asan for 399d264 has started.
2025-02-16 08:18:09 UTC Artifacts will be uploaded here
2025-02-16 08:20:34 UTC Check cancelled

@github-actions
Copy link

github-actions bot commented Feb 16, 2025

2025-02-16 08:22:07 UTC Pre-commit check linux-x86_64-relwithdebinfo for 2f77aff has started.
2025-02-16 08:22:33 UTC Artifacts will be uploaded here
2025-02-16 08:24:49 UTC ya make is running...
🟢 2025-02-16 08:24:54 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-16 08:25:00 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 16, 2025

2025-02-16 08:22:17 UTC Pre-commit check linux-x86_64-release-asan for 2f77aff has started.
2025-02-16 08:22:23 UTC Artifacts will be uploaded here
2025-02-16 08:24:46 UTC ya make is running...
🟢 2025-02-16 08:24:52 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-16 08:24:58 UTC Build successful.

Update tasks templates.
Fix bug with token.
Check pr template as is.
@github-actions
Copy link

github-actions bot commented Feb 16, 2025

2025-02-16 11:29:11 UTC Pre-commit check linux-x86_64-relwithdebinfo for ebfe997 has started.
2025-02-16 11:29:15 UTC Artifacts will be uploaded here
2025-02-16 11:31:34 UTC ya make is running...
🟢 2025-02-16 11:31:39 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-16 11:31:45 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 16, 2025

2025-02-16 11:31:29 UTC Pre-commit check linux-x86_64-release-asan for ebfe997 has started.
2025-02-16 11:31:42 UTC Artifacts will be uploaded here
2025-02-16 11:34:05 UTC ya make is running...
🟢 2025-02-16 11:34:10 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-16 11:34:16 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 16, 2025

2025-02-16 13:05:59 UTC Pre-commit check linux-x86_64-relwithdebinfo for 51395f0 has started.
2025-02-16 13:06:42 UTC Artifacts will be uploaded here
2025-02-16 13:09:04 UTC ya make is running...
🟢 2025-02-16 13:09:10 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-16 13:09:17 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 16, 2025

2025-02-16 13:06:04 UTC Pre-commit check linux-x86_64-release-asan for 51395f0 has started.
2025-02-16 13:07:35 UTC Artifacts will be uploaded here
2025-02-16 13:10:01 UTC ya make is running...
🟢 2025-02-16 13:10:07 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-16 13:10:13 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 16, 2025

2025-02-16 18:34:04 UTC Pre-commit check linux-x86_64-release-asan for 4679bb8 has started.
2025-02-16 18:34:49 UTC Artifacts will be uploaded here
2025-02-16 18:37:44 UTC ya make is running...
🟢 2025-02-16 18:37:50 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-16 18:37:56 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Feb 16, 2025

2025-02-16 18:35:37 UTC Pre-commit check linux-x86_64-relwithdebinfo for 4679bb8 has started.
2025-02-16 18:36:00 UTC Artifacts will be uploaded here
2025-02-16 18:38:17 UTC ya make is running...
🟢 2025-02-16 18:38:22 UTC Tests successful.

Test history | Ya make output | Test bloat

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

🟢 2025-02-16 18:38:31 UTC Build successful.

@galnat galnat merged commit 19ac1f8 into ydb-platform:main Feb 17, 2025
12 checks passed
lberserq pushed a commit to lberserq/ydb that referenced this pull request Mar 3, 2025
Added github action for validate PR body
Added github actions for generate changelog increment
blinkov pushed a commit that referenced this pull request Mar 21, 2025
Added github action for validate PR body
Added github actions for generate changelog increment
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