Release branch diff with upstream trunk #13
Conversation
|
/test post-test-infra-test-all |
|
@wuhuizuo: The specified target(s) for
Use In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
/assign lijie0123 |
|
/cc @lijie0123 ignore, only for test |
|
/cc lijie0123 |
feat: add push-tide.sh for tide release fix: patch missing arg fix: set prefix fix: patch env fix: remove edge prefix fix fix
* Revert "add: build tide workflow" This reverts commit 7c8b726. * feat(prow/config): add template function `replace` for jenkins controller * refactor(prow/tide): move static methods to CodeReviewCommon - and fix tests * refactor(prow/tide): provide longer commit list by increase last limit in graphql - from 4 to 100, it should be enough, also it make changes be easier. - fix unit tests of prow/tide Signed-off-by: wuhuizuo <[email protected]> * Revert "refactor(prow/tide): move static methods to CodeReviewCommon" This reverts comment style parts of commit e725958. Signed-off-by: wuhuizuo <[email protected]> * Revert "Merge branch 'feat/add-template-function-replace' into fix/tide-custom-template" This reverts comment changes in commit ab59468, reversing changes made to 439f619. Signed-off-by: wuhuizuo <[email protected]> Signed-off-by: wuhuizuo <[email protected]>
Signed-off-by: wuhuizuo <[email protected]>
- test `.ExtractConetent` for title. Signed-off-by: wuhuizuo <[email protected]> Signed-off-by: wuhuizuo <[email protected]>
- The plugin will add `lgtm` label in the pull request when at least `ReviewerCount` people agreed with `/lgtm` or review approves. - Any valid reviewer againsted, it will reset the accumulated votes and remove the `lgtm` label if existed, the againsted vote will not be accumulated in the next lgtm voting. ## TODO - [x] fix dup `lgtm` comment or review approves from same reviewer. - [x] comment when some body delete his `/lgtm` or `/lgtm cancel` comments for auditing. Signed-off-by: wuhuizuo <[email protected]> --------- Signed-off-by: wuhuizuo <[email protected]>
Signed-off-by: wuhuizuo <[email protected]>
Signed-off-by: wuhuizuo <[email protected]> Signed-off-by: wuhuizuo <[email protected]>
Signed-off-by: wuhuizuo <[email protected]> Signed-off-by: wuhuizuo <[email protected]>
Signed-off-by: wuhuizuo <[email protected]> Signed-off-by: wuhuizuo <[email protected]>
- delete the `need-more-*-lgtm` label. - update timeline comment. Signed-off-by: wuhuizuo <[email protected]> Signed-off-by: wuhuizuo <[email protected]>
## Why `lgtm` plugin's dup judging logic was not consider Github app bot name, it will be `<name>[bot]` , but common user's login can not be this format. Signed-off-by: wuhuizuo <[email protected]>
…REPOS` (#29) Some repos(which have not finished CI jobs refactoring jobs) current need tars plugin to refresh the PR. Signed-off-by: wuhuizuo <[email protected]>
It implement pr flow tasks with ChatGPT(currently we develop it with GPT3.5 turbo and GTP4 models --------- Signed-off-by: wuhuizuo <[email protected]>
) - skip by author - skip by branch - skip by pr label Signed-off-by: wuhuizuo <[email protected]> Signed-off-by: wuhuizuo <[email protected]>
#32) Fixed: kubernetes#7690 Ref: - PingCAP-QE/ee-ops#621 - Make it be fixed without massive refactoring, and not change the behavior and approve notifies. --------- Signed-off-by: wuhuizuo <[email protected]>
Signed-off-by: wuhuizuo <[email protected]>
ignore these formats: - `kubernetes#1334` - `abc kubernetes#123` - `abc#124` - `abc#https://org/repo/issue/1234` - `abc#https://org/repo/issue/1234` include those formats: - `Fix kubernetes#123` - `ref org/repo#1234` - `close https://github.com/org/repo/issue/1234` Fix ti-community-infra/tichi#1175 Signed-off-by: wuhuizuo <[email protected]>
Signed-off-by: wuhuizuo <[email protected]>
Signed-off-by: wuhuizuo <[email protected]>
Signed-off-by: wuhuizuo <[email protected]>
wuhuizuo
force-pushed
the
release
branch
2 times, most recently
from
5e9958e to
d6ed2a2
Compare
`NoParentOwners` should only worked when none people got.
For an example:
```
# file sub/OWNERS
options:
no_parent_owners: true
filters:
"\\.yaml":
approvers: [alice]
```
But only the file `func_test.go` changed in the PR, it should use the parent OWNERS file.
Signed-off-by: wuhuizuo <[email protected]>
Signed-off-by: wuhuizuo <[email protected]>
Signed-off-by: wuhuizuo <[email protected]> Signed-off-by: wuhuizuo <[email protected]>
… no parent inherit (#39) In this case, reviewers only include sub dir's approvers and reviewers and parent level reviewers but not include approvers in parent folder. Signed-off-by: wuhuizuo <[email protected]> Signed-off-by: wuhuizuo <[email protected]>
Signed-off-by: wuhuizuo <[email protected]> --------- Signed-off-by: wuhuizuo <[email protected]>
|
Issues go stale after 90d of inactivity. |
|
/remove-lifecycle stale |
|
/test ? |
|
@wuhuizuo: No presubmit jobs available for ti-community-infra/test-infra@master In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
/test all |
|
@wuhuizuo: No presubmit jobs available for ti-community-infra/test-infra@master In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Fix statusreconciler trigger all prs unexpectedly if f a run_if_changed job has its regexp changed, we should check PRs against the new regexp and only trigger it if matched the new regexp. Signed-off-by: purelind <[email protected]>
|
Issues go stale after 90d of inactivity. |
2 similar comments
|
Issues go stale after 90d of inactivity. |
|
Issues go stale after 90d of inactivity. |
Should not merge it, only for review