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.

Sign up for GitHub

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

Jump to bottom

chore(prow/config): do not watching migrated or archived repositories #765

Merged
merged 2 commits into from
Feb 2, 2023
Merged

chore(prow/config): do not watching migrated or archived repositories #765

merged 2 commits into from
Feb 2, 2023

Conversation

wuhuizuo
Copy link
Contributor

@wuhuizuo wuhuizuo commented Jan 9, 2023 •
edited
Loading

The repositories were archived or migrated for more than six months:

  • pingcap/docs-appdev
  • pingcap/dumpling
  • pingcap/dm
  • pingcap/br
  • pingcap/parser

The bot should not watching them any more.
The PR will save api calling to GitHub.

@ti-chi-bot
Copy link
Member

ti-chi-bot commented Jan 9, 2023 •
edited
Loading

[REVIEW NOTIFICATION]

This pull request has been approved by:

  • purelind

To complete the pull request process, please ask the reviewers in the list to review by filling /cc @reviewer in the comment.
After your PR has acquired the required number of LGTMs, you can assign this pull request to the committer in the list by filling /assign @committer in the comment to help you merge this pull request.

The full list of commands accepted by this bot can be found here.

Reviewer can indicate their review by submitting an approval review.
Reviewer can cancel approval by submitting a request changes review.

@ti-chi-bot
Copy link
Member

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@ti-chi-bot ti-chi-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 9, 2023
@ti-chi-bot ti-chi-bot requested review from purelind and a user January 9, 2023 04:33
@ti-chi-bot ti-chi-bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jan 9, 2023
@ti-chi-bot
Copy link
Member

@wuhuizuo: PR needs rebase.

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.

@ti-chi-bot ti-chi-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 12, 2023
@wuhuizuo wuhuizuo force-pushed the feature/delete-useless-configs-for-pingcap-dumpling-repo branch from 7f93066 to 81c6ee7 Compare February 2, 2023 07:53
@wuhuizuo wuhuizuo force-pushed the feature/delete-useless-configs-for-pingcap-dumpling-repo branch from 81c6ee7 to ff141de Compare February 2, 2023 07:55
@ti-chi-bot ti-chi-bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 2, 2023
@wuhuizuo
chore(prow/config): delete configs for archived or migrated repositories
dbc8a3d 
- pingcap/docs-appdev
- pingcap/dm
- pingcap/br
- pingcap/parser

Signed-off-by: wuhuizuo <wuhuizuo@126.com>
@ti-chi-bot ti-chi-bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 2, 2023
@wuhuizuo wuhuizuo marked this pull request as ready for review February 2, 2023 08:50
@ti-chi-bot ti-chi-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 2, 2023
@wuhuizuo wuhuizuo changed the title chore(prow/config): delete configs for repo pingcap/dumpling chore(prow/config): do not watching migrated or archived repositories Feb 2, 2023
@ti-chi-bot ti-chi-bot added the status/LGT1 Indicates that a PR has LGTM 1. label Feb 2, 2023
@wuhuizuo
Copy link
Contributor Author

wuhuizuo commented Feb 2, 2023

/merge

@ti-chi-bot
Copy link
Member

This pull request has been accepted and is ready to merge.

Commit hash: dbc8a3d

@ti-chi-bot ti-chi-bot added the status/can-merge Indicates a PR has been approved by a committer. label Feb 2, 2023
@ti-chi-bot ti-chi-bot merged commit 956059f into main Feb 2, 2023
@ti-chi-bot ti-chi-bot deleted the feature/delete-useless-configs-for-pingcap-dumpling-repo branch February 2, 2023 13:41
@ti-chi-bot
Copy link
Member

@wuhuizuo: Updated the following 3 configmaps:

  • external-plugins-config configmap in namespace prow at cluster default using the following files:
    • key external_plugins_config.yaml using file prow/config/external_plugins_config.yaml
  • plugins configmap in namespace prow at cluster default using the following files:
    • key plugins.yaml using file prow/config/plugins.yaml
  • config configmap in namespace prow at cluster default using the following files:
    • key config.yaml using file prow/config/config.yaml

In response to this:

The repositories were archived or migrated for more than six months:

  • pingcap/docs-appdev
  • pingcap/dumpling
  • pingcap/dm
  • pingcap/br
  • pingcap/parser

The bot should not watching them any more.
The PR will save api calling to GitHub.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@purelind purelind purelind approved these changes

Assignees
No one assigned
Labels
size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. status/can-merge Indicates a PR has been approved by a committer. status/LGT1 Indicates that a PR has LGTM 1.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wuhuizuo @ti-chi-bot @purelind