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

*: fix and update OWNERS file (#44377) #44387

Conversation

ti-chi-bot
Copy link
Member

This is an automated cherry-pick of #44377

What problem does this PR solve?

Issue Number: close #44385

Problem Summary: wrong filed reviewer in OWNERS file.

What is changed and how it works?

Check List

Tests

  • No code

Side effects

  • None

Release note

Please refer to Release Notes Language Style Guide to write a quality release note.

None

Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
@ti-chi-bot ti-chi-bot mentioned this pull request Jun 4, 2023
1 task
@ti-chi-bot ti-chi-bot added release-note-none Denotes a PR that doesn't merit a release note. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. type/cherry-pick-for-release-6.5 This PR is cherry-picked to release-6.5 from a source PR. labels Jun 4, 2023
@ti-chi-bot ti-chi-bot self-assigned this Jun 4, 2023
@wuhuizuo
Copy link
Contributor

wuhuizuo commented Jun 5, 2023

/retest

Signed-off-by: Weizhen Wang <wangweizhen@pingcap.com>
@ti-chi-bot
Copy link

ti-chi-bot bot commented Jun 5, 2023

[LGTM Timeline notifier]

Timeline:

  • 2023-06-05 02:50:08.734194455 +0000 UTC m=+230195.007167239: ✖️🔁 reset by hawkingrei.

@ti-chi-bot
Copy link

ti-chi-bot bot commented Jun 5, 2023

New changes are detected. LGTM label has been removed.

@hawkingrei
Copy link
Member

/approve

@ti-chi-bot
Copy link

ti-chi-bot bot commented Jun 5, 2023

@hawkingrei: adding LGTM is restricted to approvers and reviewers in OWNERS files.

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.

@hawkingrei
Copy link
Member

/approve

@wuhuizuo
Copy link
Contributor

wuhuizuo commented Jun 5, 2023

/rebase

@wuhuizuo
Copy link
Contributor

wuhuizuo commented Jun 5, 2023

rebase

@ti-chi-bot ti-chi-bot bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 5, 2023
@ti-chi-bot
Copy link

ti-chi-bot bot commented Jun 5, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hawkingrei

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

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Copy link

@ti-chi-bot ti-chi-bot bot left a comment

Choose a reason for hiding this comment

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

@ti-chi-bot: 1 invalid OWNERS file

In response to this:

This is an automated cherry-pick of #44377

What problem does this PR solve?

Issue Number: close #44385

Problem Summary: wrong filed reviewer in OWNERS file.

What is changed and how it works?

Check List

Tests

  • No code

Side effects

  • None

Release note

Please refer to Release Notes Language Style Guide to write a quality release note.

None

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.

@@ -1,5 +1,107 @@
# See the OWNERS docs at https://go.k8s.io/owners
Copy link

Choose a reason for hiding this comment

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

Cannot parse file: error converting YAML to JSON: yaml: line 105: could not find expected ':'.

OWNERS Outdated Show resolved Hide resolved
@ti-chi-bot ti-chi-bot bot removed do-not-merge/invalid-owners-file needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels Jun 5, 2023
@wuhuizuo wuhuizuo closed this Jun 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved cherry-pick-approved Cherry pick PR approved by release team. release-note-none Denotes a PR that doesn't merit a release note. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. type/cherry-pick-for-release-6.5 This PR is cherry-picked to release-6.5 from a source PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants