brpc icon indicating copy to clipboard operation
brpc copied to clipboard

Macos workflow

Open guodongxiaren opened this issue 3 years ago • 1 comments

will close #1842

目前有两个小问题。一个是bazel无法构建,需要rework bazel的那个PR合入。#1657 另外一个是macos上test_bvar的UT失败,这个已经提了issue介绍 : #1897

guodongxiaren avatar Aug 20 '22 19:08 guodongxiaren

https://github.com/apache/incubator-brpc/pull/1657 was merged.

zyearn avatar Aug 31 '22 13:08 zyearn

单元测试在mac上也有失败case。 @zyearn

guodongxiaren avatar Oct 25 '22 17:10 guodongxiaren

[ RUN      ] RecorderTest.latency_recorder_qps_accuracy
bvar_recorder_unittest.cpp:240: Failure
Expected: (0.1) > (read(lr1, 10/2.0)), actual: 0.1 vs 0.191

看起来是slow test的问题,把expectation value改一下?

zyearn avatar Oct 25 '22 17:10 zyearn

@zyearn 改了以后还有其他地方失败。。。。

guodongxiaren avatar Oct 26 '22 15:10 guodongxiaren

看了下主要是两个地方:

I1026 15:17:31  5891 bthread_butex_unittest.cpp:61] Joined thread 4294969088 at 232338us [0]
Expected: (labs(elp - (th - (bthread_t*)arg + 1) * 100000L)) <= (15000L), actual: 131917 vs 15000
bthread_butex_unittest.cpp:59: Failure
Expected: (labs(elp - (th - (bthread_t*)arg + 1) * 100000L)) <= (15000L), actual: 65739 vs 15000

再改一改试试?

zyearn avatar Oct 26 '22 19:10 zyearn

目前有很多case是关于时间判断的,但macos执行过慢,可能会造成这些case非常不稳定,要不先跳过mac上的ut执行?先把mac编译加到CI里 后面可能需要系统地理一下这些case

wwbmmm avatar Oct 27 '22 02:10 wwbmmm

目前有很多case是关于时间判断的,但macos执行过慢,可能会造成这些case非常不稳定,要不先跳过mac上的ut执行?先把mac编译加到CI里

后面可能需要系统地理一下这些case

可以。我晚点改一下

guodongxiaren avatar Oct 27 '22 02:10 guodongxiaren

把uinitest和mac 的bazel编译都删掉了 @zyearn @wwbmmm

guodongxiaren avatar Oct 28 '22 04:10 guodongxiaren