This is an archive of the discontinued LLVM Phabricator instance.

Differential D143878

Remove submit perf results
ClosedPublic

Authored by yinyangsx on Feb 12 2023, 11:01 PM.

Details

Reviewers
gkistanova
Commits
rZORG562ccebc5263: Remove submit perf results
Summary

Stop submitting results to LNT server because recently LNT server became unreliable with accepting runs

Signed-off-by: Yin Yang <yin.yang@intel.com>

Diff Detail

Event Timeline

yinyangsx created this revision.Feb 12 2023, 11:01 PM
Herald added a reviewer: gkistanova. · View Herald TranscriptFeb 12 2023, 11:01 PM
Herald added a project: Restricted Project. · View Herald Transcript
yinyangsx requested review of this revision.Feb 12 2023, 11:01 PM
Harbormaster completed remote builds in B213342: Diff 496844.Feb 12 2023, 11:01 PM
gkistanova added a comment.Feb 13 2023, 7:19 PM

Could you update the summary with the reason of why LNT submit gets removed, please?

yinyangsx added a comment.Feb 15 2023, 8:54 AM

Hi @gkistanova, We just switched to cmake from autoconfig to build and run the tests. Then we encounter a error when submitting the test results to lnt server,
2023-02-09 05:18:19: submitting result to 'http://lnt.llvm.org/submitRun'
error: b'<html>\r\n<head><title>502 Bad Gateway</title></head>\r\n<body bgcolor="white">\r\n<center><h1>502 Bad Gateway</h1></center>\r\n<hr><center>nginx/1.6.2</center>\r\n</body>\r\n</html>\r\n'

Then, we checked the all builders in builder.py, we can not find any active builder is using lnt server. So we decided to remove lnt server line.

yinyangsx added a comment.Feb 15 2023, 8:56 AM

I am new with LLVM Buildbot, what's the impacts of no lnt server configuration?

gkistanova added a comment.Feb 21 2023, 10:36 AM

That's fine, you do not have to submit perf results to LNT if you do not want to.
Just add something to the description which would provide the reasoning why you stop doing so. Saying something like "because recently LNT server became unreliable with accepting runs", or whatever you things the reason is, is Ok.

yinyangsx edited the summary of this revision. (Show Details)Feb 21 2023, 5:56 PM

Hi @gkistanova, I added reason to the description, could you please approved and lend this change?

This revision was not accepted when it landed; it landed in state Needs Review.Feb 25 2023, 12:15 AM
Closed by commit rZORG562ccebc5263: Remove submit perf results (authored by yinyangsx, committed by gkistanova). · Explain Why
This revision was automatically updated to reflect the committed changes.
gkistanova added a commit: rZORG562ccebc5263: Remove submit perf results.

Revision Contents

PathSize
buildbot/
osuosl/
master/
config/
4 lines

Diff 500382

buildbot/osuosl/master/config/builders.py

Show First 20 Lines • Show All 876 Lines • ▼ Show 20 Lines 'factory' : ClangBuilder.getClangCMakeBuildFactory(
useTwoStage=False, useTwoStage=False,
runTestSuite=True, runTestSuite=True,
testsuite_flags=['--cflag', '-march=cascadelake', '--threads=1', '--build-threads=32', '--use-perf=all', testsuite_flags=['--cflag', '-march=cascadelake', '--threads=1', '--build-threads=32', '--use-perf=all',
'--benchmarking-only', '--exec-multisample=4', '--exclude-stat-from-submission=compile'], '--benchmarking-only', '--exec-multisample=4', '--exclude-stat-from-submission=compile'],
env={'PATH':'/usr/bin/ccache:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin'}, env={'PATH':'/usr/bin/ccache:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin'},
extra_cmake_args=[ extra_cmake_args=[
"-DCMAKE_C_FLAGS='-march=cascadelake'", "-DCMAKE_C_FLAGS='-march=cascadelake'",
"-DCMAKE_CXX_FLAGS='-march=cascadelake'", "-DCMAKE_CXX_FLAGS='-march=cascadelake'",
"-DLLVM_TARGETS_TO_BUILD='X86'"], "-DLLVM_TARGETS_TO_BUILD='X86'"])},
submitURL='http://lnt.llvm.org/submitRun',
testerName='LNT-Cascadelake-AVX2-O1')},
{'name' : "llvm-avr-linux", {'name' : "llvm-avr-linux",
'tags' : ["clang"], 'tags' : ["clang"],
'workernames' : ["avr-build-01"], 'workernames' : ["avr-build-01"],
'builddir': "llvm-avr-linux", 'builddir': "llvm-avr-linux",
'factory' : ClangBuilder.getClangCMakeBuildFactory( 'factory' : ClangBuilder.getClangCMakeBuildFactory(
jobs=8, jobs=8,
clean=False, clean=False,
▲ Show 20 Lines • Show All 1,860 Lines • Show Last 20 Lines