Skip to content

test: modify cases for error message update #45400

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
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

test: modify cases for error message update #45400

wants to merge 1 commit into from
+6 −8

Conversation

@NicoYuan1986
Copy link
Contributor

@NicoYuan1986 NicoYuan1986 commented Nov 7, 2025

No description provided.

@NicoYuan1986
test: modify cases for error message update
45287a3 
Signed-off-by: nico <cheng.yuan@zilliz.com>
@sre-ci-robot
Copy link
Contributor

sre-ci-robot commented Nov 7, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: NicoYuan1986
To complete the pull request process, please assign yanliang567 after the PR has been reviewed.
You can assign the PR to them by writing /assign @yanliang567 in a comment when ready.

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

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

@sre-ci-robot sre-ci-robot added the size/S Denotes a PR that changes 10-29 lines. label Nov 7, 2025
@mergify mergify bot added dco-passed DCO check passed. kind/test labels Nov 7, 2025
@sre-ci-robot
Copy link
Contributor

sre-ci-robot commented Nov 7, 2025

[ci-v2-notice]
Notice: We are gradually rolling out the new ci-v2 system.

  • Legacy CI jobs remain unaffected, you can just ignore ci-v2 if you don't want to run it.
  • Additional "ci-v2/*" checkers will run for this PR to ensure the new ci-v2 system is working as expected.
  • For tests that exist in both v1 and v2, passing in either system is considered PASS.

To rerun ci-v2 checks, comment with:

  • /ci-rerun-code-check // for ci-v2/code-check
  • /ci-rerun-build // for ci-v2/build
  • /ci-rerun-ut-integration // for ci-v2/ut-integration
  • /ci-rerun-ut-go // for ci-v2/ut-go
  • /ci-rerun-ut-cpp // for ci-v2/ut-cpp
  • /ci-rerun-ut // for all ci-v2/ut-integration, ci-v2/ut-go, ci-v2/ut-cpp
  • /ci-rerun-e2e-arm // for ci-v2/e2e-arm

If you have any questions or requests, please contact @zhikunyao.

@mergify
Copy link
Contributor

mergify bot commented Nov 7, 2025

@NicoYuan1986 go-sdk check failed, comment rerun go-sdk can trigger the job again.

@mergify mergify bot added the ci-passed label Nov 7, 2025
yanliang567
yanliang567 reviewed Nov 8, 2025
View reviewed changes
tests/python_client/milvus_client/test_milvus_client_collection.py
error = {ct.err_code: 65535, ct.err_msg: f"duplicated new collection name {collection_name} in database "
f"default with other collection name or alias"}
error = {ct.err_code: 65535, ct.err_msg: f"collection name or database name should be different"}
self.rename_collection(client, collection_name, collection_name,
Copy link
Contributor

@yanliang567 yanliang567 Nov 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this new meg making sense?

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

Reviewers

@yanliang567 yanliang567 yanliang567 left review comments

@wangting0128 wangting0128 Awaiting requested review from wangting0128

@zhuwenxing zhuwenxing Awaiting requested review from zhuwenxing

Assignees

No one assigned

Labels

area/test ci-passed dco-passed DCO check passed. kind/test sig/testing size/S Denotes a PR that changes 10-29 lines.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@NicoYuan1986 @sre-ci-robot @yanliang567