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

operator: add additional info for operator (#2993) #3009

Merged
merged 4 commits into from
Oct 20, 2020

Conversation

ti-srebot
Copy link
Contributor

cherry-pick #2993 to release-4.0


Signed-off-by: lhy1024 [email protected]

What problem does this PR solve?

When we check the log about create and finish operator, we don't know info about it. Especially, leader score and region score.

What is changed and how it works?

Check List

Tests

  • Unit test

Release note

  • add additional info for operator

@ti-srebot ti-srebot added status/LGT1 Indicates that a PR has LGTM 1. and removed status/LGT2 Indicates that a PR has LGTM 2. labels Sep 22, 2020
@disksing disksing modified the milestones: v4.0.7, v4.0.8 Sep 22, 2020
Signed-off-by: lhy1024 <[email protected]>
@lhy1024 lhy1024 force-pushed the release-4.0-fd4e434ff7dd branch from 71679af to 08f7f23 Compare September 24, 2020 07:27
Copy link
Member

@HunDunDM HunDunDM left a comment

Choose a reason for hiding this comment

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

LGTM

@ti-srebot ti-srebot added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Sep 24, 2020
@disksing disksing added the status/wait-launch-window Wait for the release launch window. label Sep 24, 2020
@nolouch nolouch removed the status/wait-launch-window Wait for the release launch window. label Oct 19, 2020
@nolouch
Copy link
Contributor

nolouch commented Oct 19, 2020

/merge

@ti-srebot ti-srebot added the status/can-merge Indicates a PR has been approved by a committer. label Oct 19, 2020
@ti-srebot
Copy link
Contributor Author

/run-all-tests

@ti-srebot
Copy link
Contributor Author

@ti-srebot merge failed.

@lhy1024
Copy link
Contributor

lhy1024 commented Oct 20, 2020

/merge

@ti-srebot
Copy link
Contributor Author

/run-all-tests

@ti-srebot
Copy link
Contributor Author

@ti-srebot merge failed.

@lhy1024
Copy link
Contributor

lhy1024 commented Oct 20, 2020

/run-integration-compatibility-test

@nolouch nolouch merged commit a698d06 into tikv:release-4.0 Oct 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/log Log. component/scheduler Scheduler logic. status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants