Skip to content

[已重新提PR,待Review] perf: Handle exception responses to avoid partially erroneous logic and logs#5163

Closed
wangliang181230 wants to merge 17 commits into1Panel-dev:v2from
wangliang181230:PR/fix-request-error
Closed

[已重新提PR,待Review] perf: Handle exception responses to avoid partially erroneous logic and logs#5163
wangliang181230 wants to merge 17 commits into1Panel-dev:v2from
wangliang181230:PR/fix-request-error

Conversation

@wangliang181230
Copy link
Copy Markdown
Contributor

@wangliang181230 wangliang181230 commented Apr 24, 2026

What this PR does / why we need it?

Handle exception responses to avoid partially erroneous logic and logs

Summary of your change

处理异常响应,避免部分错误的逻辑和日志。

Please indicate you've done the following:

  • Made sure tests are passing and test coverage is added if needed.
  • Made sure commit message follow the rule of Conventional Commits specification.
  • Considered the docs impact and opened a new docs issue or PR with docs changes if needed.

@f2c-ci-robot
Copy link
Copy Markdown

f2c-ci-robot Bot commented Apr 24, 2026

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@f2c-ci-robot
Copy link
Copy Markdown

f2c-ci-robot Bot commented Apr 24, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@wangliang181230 wangliang181230 changed the title perf: Handle failed requests perf: Handle abnormal response Apr 24, 2026
@wangliang181230 wangliang181230 changed the title perf: Handle abnormal response perf: Handle exception responses to avoid partially erroneous logic and logs Apr 27, 2026
@shaohuzhang1
Copy link
Copy Markdown
Contributor

感谢你的贡献!为了保证提交历史的整洁与可追溯性,项目要求每个 Pull Request 只包含一个 commit

当前 PR 包含了多个 commit,暂时无法合并。请按以下方式修整后重新提交:

  1. 在本地分支执行 git rebase -i main(或目标分支),将多个 commit 压缩为一个;
  2. 然后 git push --force-with-lease 更新此 PR;
  3. 或者你也可以新开一个 PR,只附带一个干净的 commit。

我们期待你的更新,感谢理解和配合!

@wangliang181230
Copy link
Copy Markdown
Contributor Author

已经 rebase 了

@wangliang181230 wangliang181230 changed the title perf: Handle exception responses to avoid partially erroneous logic and logs [已重新提PR,待Review] perf: Handle exception responses to avoid partially erroneous logic and logs May 6, 2026
@wangliang181230
Copy link
Copy Markdown
Contributor Author

see #5221

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants