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.

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

fix: ci replication go test failed problem #2435

Merged
merged 1 commit into from
Mar 2, 2024
Merged

fix: ci replication go test failed problem #2435

merged 1 commit into from
Mar 2, 2024

Conversation

Mixficsol
Copy link
Collaborator

@Mixficsol Mixficsol commented Mar 1, 2024

由于在之前线上的 ci 中主从测试经常出现测试失败的问题,在本 PR 中我们把集成测试和主从的测试进行了拆分,对于单机的集成测试我们使用 9221 端口,对于 Pika 的主从测试,我们使用了 92319241 作为端口进行主从测试,确保环境干净,目前在 Mac 本机上测试通过率为 100% 并且稳定

Since the master-slave test often failed in the previous online ci, we split the integration test and the master-slave test in this PR. For the stand-alone integration test, we used the '9221' port. For the master-slave test of 'Pika', we used the '9221' port. We used '9231' and '9241' as ports for master-slave testing to ensure a clean environment, and the current test pass rate on 'Mac' is' 100% 'and stable

截屏2024-03-01 14 58 10

#2356

@Mixficsol Mixficsol added ☢️ Bug Something isn't working 3.5.3 labels Mar 1, 2024
@github-actions github-actions bot added the 🧹 Updates This will not be worked on label Mar 1, 2024
@Mixficsol Mixficsol added the test label Mar 1, 2024
@baerwang baerwang merged commit 60718b1 into OpenAtomFoundation:unstable Mar 2, 2024
12 checks passed
@Mixficsol Mixficsol deleted the dev branch March 2, 2024 15:02
Copy link
Collaborator

@luky116 luky116 left a comment

Choose a reason for hiding this comment

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

LGTM

luky116 pushed a commit to luky116/pika that referenced this pull request Mar 14, 2024
Co-authored-by: wuxianrong <wuxianrong@360.cn>
luky116 pushed a commit to luky116/pika that referenced this pull request Mar 14, 2024
Co-authored-by: wuxianrong <wuxianrong@360.cn>
luky116 pushed a commit that referenced this pull request Mar 18, 2024
Co-authored-by: wuxianrong <wuxianrong@360.cn>
bigdaronlee163 pushed a commit to bigdaronlee163/pika that referenced this pull request Jun 8, 2024
Co-authored-by: wuxianrong <wuxianrong@360.cn>
cheniujh pushed a commit to cheniujh/pika that referenced this pull request Sep 24, 2024
Co-authored-by: wuxianrong <wuxianrong@360.cn>
cheniujh pushed a commit to cheniujh/pika that referenced this pull request Sep 24, 2024
Co-authored-by: wuxianrong <wuxianrong@360.cn>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.5.3 ☢️ Bug Something isn't working test 🧹 Updates This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants