8000 [BUILD] add loongarch info by loong-hy · Pull Request #3052 · open-telemetry/opentelemetry-cpp · GitHub
[go: up one dir, main page]
Skip to content

[BUILD] add loongarch info #3052

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
Merged
merged 4 commits into from
Sep 18, 2024
Merged

[BUILD] add loongarch info #3052

merged 4 commits into from
Sep 18, 2024

Conversation

loong-hy
Copy link
Contributor
@loong-hy loong-hy commented Sep 5, 2024

about loongarch： https://docs.kernel.org/arch/loongarch/introduction.html

`
......
Start 413: exporter.OstreamLogExporter.DefaultLogRecordToCout
413/421 Test #413: exporter.OstreamLogExporter.DefaultLogRecordToCout ................................................... Passed 0.00 sec
Start 414: exporter.OStreamLogRecordExporter.SimpleLogToCout
414/421 Test #414: exporter.OStreamLogRecordExporter.SimpleLogToCout .................................................... Passed 0.00 sec
Start 415: exporter.OStreamLogRecordExporter.LogWithStringAttributesToCerr
415/421 Test #415: exporter.OStreamLogRecordExporter.LogWithStringAttributesToCerr ...................................... Passed 0.00 sec
Start 416: exporter.OStreamLogRecordExporter.LogWithVariantTypesToClog
416/421 Test #416: exporter.OStreamLogRecordExporter.LogWithVariantTypesToClog .......................................... Passed 0.00 sec
Start 417: exporter.OStreamLogRecordExporter.IntegrationTest
417/421 Test #417: exporter.OStreamLogRecordExporter.IntegrationTest .................................................... Passed 0.00 sec
Start 418: exporter.OStreamLogRecordExporter.IntegrationTestWithEventId
418/421 Test #418: exporter.OStreamLogRecordExporter.IntegrationTestWithEventId ......................................... Passed 0.00 sec
Start 419: exporter.OStreamLogRecordExporter.Factory
419/421 Test #419: exporter.OStreamLogRecordExporter.Factory ............................................................ Passed 0.00 sec
Start 420: exporter.InMemorySpanData.AddRecordable
420/421 Test #420: exporter.InMemorySpanData.AddRecordable .............................................................. Passed 0.00 sec
Start 421: exporter.InMemorySpanExporter.ExportBatch
421/421 Test #421: exporter.InMemorySpanExporter.ExportBatch ............................................................ Passed 0.00 sec

100% tests passed, 0 tests failed out of 421

Total Test time (real) = 11.78 sec

`

@loong-hy loong-hy requested a review from a team September 5, 2024 08:30
@linux-foundation-easycla Linux Foundation: EasyCLA
Copy link
linux-foundation-easycla bot commented Sep 5, 2024 •
edited
Loading

CLA Signed


The committers listed above are authorized under a signed CLA.

@marcalff marcalff changed the title add loongarch info [BUILD] add loongarch info Sep 5, 2024
@marcalff
Copy link
Member
marcalff commented Sep 5, 2024

@loong-hy

Thanks for the patch.

Please sign the CLA so it can be merged.

@marcalff marcalff added the pr:waiting-on-cla Waiting on CLA label Sep 5, 2024
@codecov Codecov
Copy link
codecov bot commented Sep 5, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.78%. Comparing base (497eaf4) to head (9035790).
Report is 131 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3052      +/-   ##
==========================================
+ Coverage   87.12%   87.78%   +0.66%     
==========================================
  Files         200      195       -5     
  Lines        6109     5953     -156     
==========================================
- Hits         5322     5225      -97     
+ Misses        787      728      -59

see 125 files with indirect coverage changes

ThomsonTan
ThomsonTan reviewed Sep 5, 2024
View reviewed changes
@lalitb
Copy link
Member
lalitb commented Sep 6, 2024

LGTM once the CLA is signed.

marcalff
marcalff approved these changes Sep 6, 2024
View reviewed changes
Copy link
Member
@marcalff marcalff left a comment

Choose a reason for hiding this comment

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

LGTM once the CLA is signed.

owent
owent approved these changes 8000 Sep 9, 2024
View reviewed changes
@ThomsonTan
Copy link
Contributor

@loong-hy thanks for the contribution. Could you please take look at signing the CLA so we can merge the PR?

@marcalff
Copy link
Member

/easycla

@loong-hy
Copy link
Contributor Author

@loong-hy thanks for the contribution. Could you please take look at signing the CLA so we can merge the PR?

Already signed. Thank you.
8000

@marcalff marcalff merged commit 2314e06 into open-telemetry:main Sep 18, 2024
52 checks passed
@marcalff marcalff removed the pr:waiting-on-cla Waiting on CLA label Sep 18, 2024
malkia added a commit to malkia/opentelemetry-cpp that referenced this pull request Sep 18, 2024
@malkia
Merge pull request #70 from open-telemetry/main
77870dc 
[BUILD] add loongarch info (open-telemetry#3052)
@BrewTestBot BrewTestBot mentioned this pull request Oct 8, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ThomsonTan ThomsonTan ThomsonTan approved these changes

@lalitb lalitb lalitb approved these changes

@owent owent owent approved these changes

@marcalff marcalff marcalff approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@loong-hy @marcalff @lalitb @ThomsonTan @owent
0