Possible misleading `prer` commit history
- Ngôn ngữ chính
- Starlark
- Star
- 208
- Fork
- 45
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
`prer` implicitly assumes that is is run in strict order on the commits in the tracked branch. If commits execute in parallel (as is the case if its run in Github Actions) there is no guarantee that these will complete in order. This can lead to generated release notes missing commits from which they do contain changes, or referencing commits from which the changes are not included.
Let's assume `prer` is run in a GithubAction workflow with two steps:
```
bazel test //...
prer
```
Two commits which modify different services: `A-B`, and begin workflow execution. For the sake of example, assume the test step takes twice as long for `A` as it does for `B`. `B` completes, renders manifests, see changes and pushes to the PR branch all of the changes it produced (from both A and B). `A` completes, renders manifests, sees no changes and does not push to the PR branch. In this example changes from both `A` and `B` are included but the notes only contain a reference to `B`.
Taking the same example, but this time assuming that `A` and `B` modify the same service. In this case when `A` completes it will see changes (since B's changes include A's but A's dont include B's) and commit them. In this case, the release notes will reference both A and B (though in reverse order) however the changes in the output will only be build as of A. The changes from B will ultimately be included by the next commit (which plays out like the example above).
This scenario can be checked by `prer`. When it looks at the git log as of its current ref it can decide:
* if this commit is ahead of the commit for which im running - abort
* if this commit is behind the commit for which im running - include a log of all changes since then, not just the log for the commit im running.
Another possible outcome is that both run and checkout the accumulation branch as of the same commit. This will lead one of the executions to fail when it tries to push this branch. This could be addressed with a retry (which includes the logic described above)
Hướng dẫn đóng góp
Hướng nghiên cứu
Bắt đầu tại điểm vào `prer` và theo dõi cách Git ref hiện tại, lịch sử commit và nhánh tích lũy được đọc trong một lần chạy GitHub Actions. Tái hiện các kịch bản được mô tả với thứ tự không đúng và cùng một nhánh. Hoàn tất khi các lần thực thi đồng thời không còn tạo ra release notes gây hiểu nhầm hoặc làm mất các thay đổi đã tạo.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- git, github-actions
- Lĩnh vực
- ci-cd, release
- Loại issue
- Lỗi
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 35/100