Skip to content

Commit 0673cc0

Browse files
authored
issue owner fix (#36391)
1 parent b43861c commit 0673cc0

File tree

5 files changed

+10
-3
lines changed

5 files changed

+10
-3
lines changed

scripts/auto_release/PythonSdkLiveTest.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,7 @@ jobs:
9191
export DEBUG_SDK_BRANCH=$(DEBUG_SDK_BRANCH)
9292
export TARGET_DATE=$(TARGET_DATE)
9393
export TEST_FOLDER=$(TEST_FOLDER)
94+
export ISSUE_OWNER=$(ISSUE_OWNER)
9495
9596
# run
9697
cd azure-sdk-for-python

scripts/auto_release/main.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,7 @@ def __init__(self):
111111
self.spec_repo = os.getenv('SPEC_REPO', '')
112112
self.target_date = os.getenv('TARGET_DATE', '')
113113
self.test_folder = os.getenv('TEST_FOLDER', '')
114+
self.issue_owner = os.getenv('ISSUE_OWNER', '')
114115

115116
self.package_name = '' # 'dns' of 'sdk/compute/azure-mgmt-dns'
116117
self.whole_package_name = '' # 'azure-mgmt-dns'
@@ -522,7 +523,7 @@ def ask_check_policy(self):
522523
# comment to ask for check from users
523524
issue_number = int(self.issue_link.split('/')[-1])
524525
api = GhApi(owner='Azure', repo='sdk-release-request', token=self.bot_token)
525-
author = api.issues.get(issue_number=issue_number).user.login
526+
author = self.issue_owner or api.issues.get(issue_number=issue_number).user.login
526527
body = f'Hi @{author}, please check whether CHANGELOG for this release meet requirements:\n' \
527528
f'```\n' \
528529
f'CHANGELOG:\n' \

scripts/release_helper/common.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -428,7 +428,7 @@ def output_md(idx: int, item: IssueProcess):
428428
idx,
429429
item.issue_package.issue.html_url.split('/')[-1],
430430
item.issue_package.issue.html_url,
431-
item.issue_package.issue.user.login,
431+
item.owner,
432432
item.package_name,
433433
item.assignee,
434434
' '.join(item.bot_advice),

scripts/release_helper/python.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,7 @@ def auto_reply(self) -> None:
8585
python_tag=self.python_tag,
8686
rest_repo_hash=self.rest_repo_hash,
8787
target_date=self.target_date,
88+
issue_owner=self.owner,
8889
)
8990
if res_run:
9091
self.log(f'{issue_number} run pipeline successfully')

scripts/release_helper/utils.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ def get_python_release_pipeline(output_folder):
7373

7474

7575
# Run sdk-auto-release(main) to generate SDK
76-
def run_pipeline(issue_link, pipeline_url, spec_readme, python_tag="", rest_repo_hash="", target_date=""):
76+
def run_pipeline(issue_link, pipeline_url, spec_readme, python_tag="", rest_repo_hash="", target_date="", issue_owner=""):
7777
paramaters = {
7878
"stages_to_skip": [],
7979
"resources": {
@@ -111,6 +111,10 @@ def run_pipeline(issue_link, pipeline_url, spec_readme, python_tag="", rest_repo
111111
"TARGET_DATE": {
112112
"value": target_date,
113113
"isSecret": False
114+
},
115+
"ISSUE_OWNER": {
116+
"value": issue_owner,
117+
"isSecret": False
114118
}
115119
}
116120
}

0 commit comments

Comments
 (0)