mirror of
https://github.com/ClickHouse/ClickHouse.git
synced 2024-11-22 15:42:02 +00:00
Merge pull request #40177 from ClickHouse/cherry-pick-logging
Improve assignment and logging for cherry-pick and backport steps
This commit is contained in:
commit
9f3b2fafcc
@ -206,8 +206,13 @@ Merge it only if you intend to backport changes to the target branch, otherwise
|
|||||||
)
|
)
|
||||||
self.cherrypick_pr.add_to_labels(Labels.LABEL_CHERRYPICK)
|
self.cherrypick_pr.add_to_labels(Labels.LABEL_CHERRYPICK)
|
||||||
self.cherrypick_pr.add_to_labels(Labels.LABEL_DO_NOT_TEST)
|
self.cherrypick_pr.add_to_labels(Labels.LABEL_DO_NOT_TEST)
|
||||||
if self.pr.assignee is not None:
|
if self.pr.assignees:
|
||||||
self.cherrypick_pr.add_to_assignees(self.pr.assignee)
|
logging.info(
|
||||||
|
"Assing to assignees of the original PR: %s",
|
||||||
|
", ".join(user.login for user in self.pr.assignees),
|
||||||
|
)
|
||||||
|
self.cherrypick_pr.add_to_assignees(self.pr.assignees)
|
||||||
|
logging.info("Assign to the author of the original PR: %s", self.pr.user.login)
|
||||||
self.cherrypick_pr.add_to_assignees(self.pr.user)
|
self.cherrypick_pr.add_to_assignees(self.pr.user)
|
||||||
|
|
||||||
def create_backport(self):
|
def create_backport(self):
|
||||||
@ -239,8 +244,13 @@ Merge it only if you intend to backport changes to the target branch, otherwise
|
|||||||
head=self.backport_branch,
|
head=self.backport_branch,
|
||||||
)
|
)
|
||||||
self.backport_pr.add_to_labels(Labels.LABEL_BACKPORT)
|
self.backport_pr.add_to_labels(Labels.LABEL_BACKPORT)
|
||||||
if self.pr.assignee is not None:
|
if self.pr.assignees:
|
||||||
self.cherrypick_pr.add_to_assignees(self.pr.assignee)
|
logging.info(
|
||||||
|
"Assing to assignees of the original PR: %s",
|
||||||
|
", ".join(user.login for user in self.pr.assignees),
|
||||||
|
)
|
||||||
|
self.cherrypick_pr.add_to_assignees(self.pr.assignees)
|
||||||
|
logging.info("Assign to the author of the original PR: %s", self.pr.user.login)
|
||||||
self.backport_pr.add_to_assignees(self.pr.user)
|
self.backport_pr.add_to_assignees(self.pr.user)
|
||||||
|
|
||||||
@property
|
@property
|
||||||
|
Loading…
Reference in New Issue
Block a user