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.

Sign up for GitHub

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

Jump to bottom

pref:worker模块下载接口读取文件流出错应该整个接口重试 #6404

Closed
carlyin0801 opened this issue Mar 24, 2022 · 0 comments
Closed

pref:worker模块下载接口读取文件流出错应该整个接口重试 #6404

carlyin0801 opened this issue Mar 24, 2022 · 0 comments
Assignees
@yjieliang
Labels
area/ci/backend CI 后端issue done Production environment in tencent has been deploy for gray UAT environment in tencent has been deploy for test Test environment in tencent has been deploy grayed uat环境测试通过/test passed for uat stage help wanted Extra attention is needed kind/feat/tech 技术性特性 streams/done stream 生产部署成功 streams/for gray stream 灰度环境部署成功 tested 测试环境通过/test patest passed for test stage

Comments

@carlyin0801
Copy link
Collaborator

问题：AbstractBuildResourceApi类的download方法在读取文件流出错时只重试了处理response的逻辑，实际上没有真正重试下载接口
措施：读取文件流出错应该整个接口重试
@carlyin0801 carlyin0801 added help wanted Extra attention is needed kind/feat/tech 技术性特性 area/ci/backend CI 后端issue labels Mar 24, 2022
yjieliang added a commit to yjieliang/bk-ci that referenced this issue Mar 25, 2022
@yjieliang
pref:worker模块下载接口读取文件流出错应该整个接口重试 TencentBlueKing#6404
5c1ee68
@yjieliang yjieliang mentioned this issue Mar 25, 2022
Closed
yjieliang added a commit to yjieliang/bk-ci that referenced this issue Mar 25, 2022
@yjieliang
pref:worker模块下载接口读取文件流出错应该整个接口重试 TencentBlueKing#6404
e9dae86
yjieliang added a commit to yjieliang/bk-ci that referenced this issue Mar 25, 2022
@yjieliang
pref:worker模块下载接口读取文件流出错应该整个接口重试 TencentBlueKing#6404
1d39648
yjieliang added a commit to yjieliang/bk-ci that referenced this issue Mar 28, 2022
@yjieliang
pref:worker模块下载接口读取文件流出错应该整个接口重试 TencentBlueKing#6404
8f7a27c
@yjieliang yjieliang mentioned this issue Mar 28, 2022
Merged
@yjieliang yjieliang added the for test Test environment in tencent has been deploy label Mar 28, 2022
yjieliang added a commit to yjieliang/bk-ci that referenced this issue Mar 30, 2022
@yjieliang
pref:worker模块下载接口读取文件流出错应该整个接口重试 TencentBlueKing#6404
1d159d5
@yjieliang yjieliang added the tested 测试环境通过/test patest passed for test stage label Mar 30, 2022
irwinsun added a commit that referenced this issue Apr 6, 2022
@irwinsun
Merge pull request #6434 from yjieliang/pref_6404
30e1090 
pref:worker模块下载接口读取文件流出错应该整个接口重试 #6404
@bkci-bot bkci-bot added for gray UAT environment in tencent has been deploy streams/for gray stream 灰度环境部署成功 labels Apr 7, 2022
@yjieliang yjieliang added the grayed uat环境测试通过/test passed for uat stage label Apr 14, 2022
@bkci-bot bkci-bot added the done Production environment in tencent has been deploy label Apr 14, 2022
@bkci-bot bkci-bot added the streams/done stream 生产部署成功 label Apr 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yjieliang yjieliang

Labels
area/ci/backend CI 后端issue done Production environment in tencent has been deploy for gray UAT environment in tencent has been deploy for test Test environment in tencent has been deploy grayed uat环境测试通过/test passed for uat stage help wanted Extra attention is needed kind/feat/tech 技术性特性 streams/done stream 生产部署成功 streams/for gray stream 灰度环境部署成功 tested 测试环境通过/test patest passed for test stage
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@carlyin0801 @bkci-bot @yjieliang