Skip to content

Commit c060fc3

Browse files
BigCat20196msyyc
andauthored
[Release issue status] Update in a new way (Azure#22802)
* add assignee * update function * fix bug * delete old code * fix bug * fix bug * fix pipeline id * Update scripts/release_issue_status/utils.py Co-authored-by: Yuchao Yan <[email protected]> * Update scripts/release_issue_status/utils.py Co-authored-by: Yuchao Yan <[email protected]> * Update scripts/release_issue_status/utils.py * Update scripts/release_issue_status/utils.py Co-authored-by: Yuchao Yan <[email protected]>
1 parent ef91a8d commit c060fc3

File tree

3 files changed

+14
-40
lines changed

3 files changed

+14
-40
lines changed

scripts/release_issue_status/main.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -170,10 +170,9 @@ def auto_reply(item, request_repo, rest_repo, sdk_repo, duplicated_issue, python
170170
logging.info(e)
171171
raise
172172
try:
173-
logging.info(python_piplines)
174173
pipeline_url = get_pipeline_url(python_piplines, output_folder)
175174
rg.begin_reply_generate(item=item, rest_repo=rest_repo, readme_link=readme_link,
176-
sdk_repo=sdk_repo, pipeline_url=pipeline_url, assigner_repo=assigner_repo)
175+
pipeline_url=pipeline_url)
177176
if 'Configured' in item.labels:
178177
item.issue_object.remove_from_labels('Configured')
179178
except Exception as e:

scripts/release_issue_status/reply_generator.py

Lines changed: 4 additions & 33 deletions
Original file line numberDiff line numberDiff line change
@@ -52,51 +52,22 @@ def get_latest_pr_from_readme(rest_repo, link_dict):
5252
return latest_pr_number_int[-1]
5353

5454

55-
def reply_owner(assigner_issue, reply_content):
56-
assigner_issue.create_comment(reply_content)
57-
user_tips = 'Tips: If you have special needs for release date or other things, please let us know. Otherwise we will release it ASAP after your check.'
58-
assigner_issue.create_comment(user_tips)
59-
60-
61-
62-
def get_reply_and_sdk_number_from_readme(rest_repo, link_dict, item):
63-
commits = rest_repo.get_commits(path=link_dict['resource_manager'])
64-
latest_commit = commits[0]
65-
check_run_id = latest_commit.get_check_runs(check_name='SDK azure-sdk-for-python-track2')[0].id
66-
latest_pr_number = latest_commit.get_pulls()[0].number
67-
details = rest_repo.get_check_run(check_run_id).output.text
68-
sdk_link_number = re.findall(r'/azure-sdk-for-python/pull/(\d*)">Release SDK Changes</a>', details)[0]
69-
changelog = '<details open><summary><b> python-track2</b>' + \
70-
re.search(rf'</code><b>track2_{item.package}(.)+?</pre></li>', details, re.DOTALL).group() \
71-
+ '</details>'
72-
info_model = 'hi @{} Please check the package whether works well and the changelog info ' \
73-
'is as below:\n{}\n' \
74-
'\n* (If you are not a Python User, you can mainly check whether the changelog meets your requirements)\n' \
75-
'\n* (The version of the package is only a temporary version for testing)\n' \
76-
'\nhttps://github.com/Azure/azure-rest-api-specs/pull/{}\n' \
77-
.format(issue_object_rg.user.login, changelog, str(latest_pr_number))
78-
return info_model, sdk_link_number
79-
80-
81-
def begin_reply_generate(item, rest_repo, readme_link, sdk_repo, pipeline_url, assigner_repo):
55+
def begin_reply_generate(item, rest_repo, readme_link, pipeline_url):
8256
global issue_object_rg
8357
issue_object_rg = item.issue_object
8458
link_dict = get_links(readme_link)
8559
labels = item.labels
8660
whether_change_readme = readme_comparison(rest_repo, link_dict, labels)
8761

8862
if not whether_change_readme:
89-
reply_content, sdk_link_number = get_reply_and_sdk_number_from_readme(rest_repo, link_dict, item)
9063
res_run = run_pipeline(issue_link=issue_object_rg.html_url,
91-
sdk_issue_object=sdk_repo.get_pull(int(sdk_link_number)),
92-
pipeline_url=pipeline_url
93-
)
64+
pipeline_url=pipeline_url,
65+
spec_readme=readme_link
66+
)
9467
if res_run:
9568
logging.info(f'{issue_object_rg.number} run pipeline successfully')
9669
else:
9770
logging.info(f'{issue_object_rg.number} run pipeline fail')
98-
assigner_issue = assigner_repo.get_issue(number=issue_object_rg.number)
99-
reply_owner(assigner_issue, reply_content)
10071
issue_object_rg.add_to_labels('auto-ask-check')
10172
else:
10273
logging.info('issue {} need config readme'.format(issue_object_rg.number))

scripts/release_issue_status/utils.py

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ def get_pipeline_url(python_piplines, output_folder):
148148

149149

150150
# Run sdk-auto-release(main) to generate SDK
151-
def run_pipeline(issue_link, sdk_issue_object, pipeline_url):
151+
def run_pipeline(issue_link, pipeline_url, spec_readme):
152152
paramaters = {
153153
"stages_to_skip": [],
154154
"resources": {
@@ -160,15 +160,19 @@ def run_pipeline(issue_link, sdk_issue_object, pipeline_url):
160160
},
161161
"variables": {
162162
"BASE_BRANCH": {
163-
"value": f"{sdk_issue_object.head.label}",
163+
"value": "",
164164
"isSecret": False
165165
},
166166
"ISSUE_LINK": {
167-
"value": f"{issue_link}",
167+
"value": issue_link,
168168
"isSecret": False
169169
},
170170
"PIPELINE_LINK": {
171-
"value": f"{pipeline_url}",
171+
"value": pipeline_url,
172+
"isSecret": False
173+
},
174+
"SPEC_README":{
175+
"value": spec_readme,
172176
"isSecret": False
173177
}
174178
}
@@ -181,7 +185,7 @@ def run_pipeline(issue_link, sdk_issue_object, pipeline_url):
181185
credentials = BasicAuthentication('', personal_access_token)
182186
run_parameters = models.RunPipelineParameters(**paramaters)
183187
client = PipelinesClient(base_url=organization_url, creds=credentials)
184-
result = client.run_pipeline(project='internal',pipeline_id=2500,run_parameters=run_parameters)
188+
result = client.run_pipeline(project='internal', pipeline_id=2500, run_parameters=run_parameters)
185189
if result.state == 'inProgress':
186190
return True
187191
else:

0 commit comments

Comments
 (0)