已合并
fix: Fix missing deployment_id parameter in DeployResult of LocalSubprocessDeployer.stop() #17
fix: Fix missing deployment_id parameter in DeployResult of LocalSubprocessDeployer.stop() #17
已合并
陈卉创建于 3月20日
陈卉
3月20日

What type of PR is this?

/kind

Self-checklist:(请自检,在[ ]内打上x,我们将检视你的完成情况,否则会导致pr无法合入)

    • [x ] 接口:是否涉及对外接口变更,相应变更已得到接口评审组织的通过,API对应的注释信息已经刷新正确

这个PR的主要工作
问题现象:
调用停止部署接口时返回 500 错误,日志提示DeployResult.init() missing 1 required positional argument: 'deployment_id',进程无法正常停止。
问题原因:
LocalSubprocessDeployer的stop()方法返回DeployResult时,未传入必填的deployment_id参数,导致构造函数报错。
修复方案:
在stop()方法的成功和异常返回中,补充deployment_id参数。

likedislike
Pull Request已成功合入, 合并人@王明琦
(感谢 陈卉 的贡献)
openJiuwen-bot成员
3月20日 评论:

欢迎来到 openJiuwen 社区

Hey @chen-hui-zhe-xi , 感谢你对社区的贡献.

机器人使用手册

有关指令的使用,可以点击 此处 查看详情。开发人员可以在每个PR或Issue下方评论特定指令来触发机器人任务。

联系指引

如果您有任何疑问,请联系仓库维护者: @ZYQ5333, @zhuxushen, @iamcandiceguo, @Fancy1234567, @huguangzheng, @openLiBing-bot .

likedislike
OopenJiuwen-bot成员
3月20日 添加了label:openJiuwen-cla/yes
openJiuwen-bot成员
3月20日 评论:

CLA 签署成功

@chen-hui-zhe-xi, 感谢您的代码贡献。 所有的代码贡献者都已完成了 CLA 签署。 👍

likedislike
陈陈卉
3月20日 修改了pull request 的描述
陈陈卉
3月20日 修改了pull request 的描述
王明琦王明琦成员
3月20日 将wangmingqi设为测试人
王明琦王明琦成员
3月20日 将 wangmingqi 设为合并人
王明琦王明琦成员
3月20日 将wangmingqi设为评审人
王明琦王明琦成员
3月20日 通过了评审
王明琦王明琦成员
3月20日 合入了pull request