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

UCS/UCT: Add class macro to generate new/delete function names. #31

Merged
merged 1 commit into from
Nov 26, 2014

Conversation

yosefe
Copy link
Contributor

@yosefe yosefe commented Nov 25, 2014

Fix according to Pasha's comment on the conf call.

@mellanox-github
Copy link
Contributor

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
http://bgate.mellanox.com:8000/jenkins-secure/job/gh-ucx-pr/56/

Build Log
last 50 lines

[...truncated 2054 lines...]
[ RUN      ] test_class.failure
[       OK ] test_class.failure (0 ms)
[----------] 3 tests from test_class (0 ms total)

[----------] 1 test from test_uct_perf
[ RUN      ] test_uct_perf.envelope
[     INFO ] rc_mlx5/mlx5_0:1     put latency : 0.629 usec
[     INFO ] rc_mlx5/mlx5_0:1     put msgrate : 5.956 Mpps
/scrap/jenkins/jobs/gh-ucx-pr/workspace-2/contrib/../test/gtest/uct/test_uct_perf.cc:296: Failure
Expected: (value) >= (test->min), actual: 5.9564460436215052 vs 6
[  FAILED  ] test_uct_perf.envelope (1008 ms)
[----------] 1 test from test_uct_perf (1008 ms total)

[----------] 3 tests from test_uct
[ RUN      ] test_uct.query_resources
[     INFO ] 0: rc_mlx5 on mlx5_0:1 at 6502.32 MB/sec
[       OK ] test_uct.query_resources (15 ms)
[ RUN      ] test_uct.open_iface
[       OK ] test_uct.open_iface (28 ms)
[ RUN      ] test_uct.connect_ep
[       OK ] test_uct.connect_ep (162 ms)
[----------] 3 tests from test_uct (205 ms total)

[----------] Global test environment tear-down
[==========] 72 tests from 20 test cases ran. (5301 ms total)
[  PASSED  ] 71 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] test_uct_perf.envelope

 1 FAILED TEST
make: *** [test] Error 1
make: Leaving directory `/scrap/jenkins/jobs/gh-ucx-pr/workspace-2/build-test/test/gtest'
Build step 'Execute shell' marked build as failure
Publish TAP Results is waiting for a checkpoint on gh-ucx-pr #55
TAP Reports Processing: START
Looking for TAP results report in workspace using pattern: **/*.tap
Did not find any matching files.
[Valgrind] Files to copy:
[Valgrind] Analysing valgrind results
[Valgrind] Ending the valgrind analysis.
Anchor chain: could not read file with links: /var/lib/jenkins/jobs/gh-ucx-pr/workspace-2/jenkins_sidelinks.txt (No such file or directory)
[copy-to-slave] The build is taking place on the master node, no copy back to the master will take place.
Email was triggered for: Failure - Any
Sending email for trigger: Failure - Any
Request made to compress build log
Sending email to: [email protected] [email protected] [email protected]
[BFA] Scanning build for known causes...

[BFA] Done. 0s

Test FAILed.

@mike-dubman
Copy link
Contributor

retest this please

@mellanox-github
Copy link
Contributor

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://bgate.mellanox.com:8000/jenkins-secure/job/gh-ucx-pr/60/
Test PASSed.

@mike-dubman
Copy link
Contributor

👍

mike-dubman added a commit that referenced this pull request Nov 26, 2014
UCS/UCT: Add class macro to generate new/delete function names.
@mike-dubman mike-dubman merged commit 592833d into openucx:master Nov 26, 2014
@mike-dubman mike-dubman deleted the topic/class-new-delete-names branch November 26, 2014 09:28
amaslenn pushed a commit to amaslenn/ucx that referenced this pull request Jun 2, 2020
alinask pushed a commit to alinask/ucx that referenced this pull request Sep 13, 2020
UCP: fail if the server cannot connect to all eps addrs from the client
shizhibao pushed a commit to shizhibao/ucx that referenced this pull request Jan 16, 2021
dmitrygx pushed a commit to dmitrygx/ucx that referenced this pull request Dec 1, 2021
REF/ENH: Add function for getting the address of an interface
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants